Re: [PATCH 1/2] compiler_attributes.h: define __no_profile, add to noinstr

From: Miguel Ojeda
Date: Sat Jun 19 2021 - 07:26:57 EST


On Sat, Jun 19, 2021 at 1:30 AM Nick Desaulniers
<ndesaulniers@xxxxxxxxxx> wrote:
>
> +/*
> + * Optional: only supported since clang >= 13
> + * gcc: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80223
> + * clang: https://clang.llvm.org/docs/AttributeReference.html#no_profile
> + */

I am not sure if it is best or not to have the GCC link in order to be
consistent with the rest of the links (they are for the docs only). Do
we know if GCC going to implement it soon?

Otherwise, it looks good to me.

Cheers,
Miguel