nickdesaulniers added a comment.

In D104253#2817695 <https://reviews.llvm.org/D104253#2817695>, @MaskRay wrote:

> `__attribute__((no_instrument_function))` seems specific to 
> `-finstrument-functions`.  Somehow `gcc -pg` uses it as well.
>
> The name may not be generic, so it may be odd to exclude various 
> instrumentations under this generic attribute.
>
> clang -fprofile-instr-generate (frontend coverage/PGO; AIUI only Apples uses 
> it for PGO) / clang -fprofile-generate (IR PGO; most users use this) / clang 
> -fprofile-arcs / gcc -fprofile-arcs are 4 different instrumentation 
> approaches.
> Their user-facing options are similar enough and the instrumented output has 
> similarity (so a user not wanting one instr will certainly not want other 
> instr) so I think they warrant similar function attribute.
>
> Perhaps a good idea to file a feature request on 
> https://gcc.gnu.org/bugzilla/buglist.cgi?bug_status=__open__&component=gcov-profile&list_id=304970&product=gcc
>  and check what function attributes GCC wants to use.

Done; thanks for the advice: 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80223#c6.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104253/new/

https://reviews.llvm.org/D104253

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to