[PATCH] D101598: [clang][Sema] adds `[[clang::no_address]]` attribute

rsmith added a comment. In D101598#2729865 , @ldionne wrote: > Just for posterity, what we discussed is that since there is a list of > addressable functions in the standard, we should explore adding a warning to > Clang that fires whenever somebody tak

[PATCH] D101598: [clang][Sema] adds `[[clang::no_address]]` attribute

xbolva00 added a comment. In D101598#2729865 , @ldionne wrote: > Just for posterity, what we discussed is that since there is a list of > addressable functions in the standard, we should explore adding a warning to > Clang that fires whenever somebody t

[PATCH] D101598: [clang][Sema] adds `[[clang::no_address]]` attribute

ldionne added a comment. Just for posterity, what we discussed is that since there is a list of addressable functions in the standard, we should explore adding a warning to Clang that fires whenever somebody takes the address of a function in namespace `std`, except if it's an addressable funct

[PATCH] D101598: [clang][Sema] adds `[[clang::no_address]]` attribute

cjdb abandoned this revision. cjdb added a comment. A different path forward was discussed with @ldionne and @zoecarver. Will experiment this other way. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101598/new/ https://reviews.llvm.org/D101598 __

[PATCH] D101598: [clang][Sema] adds `[[clang::no_address]]` attribute

Quuxplusone added inline comments. Comment at: clang/test/SemaCXX/attr-no-address.cpp:47-48 +void address_test() { + void (*fp1)() = &std::one_overload; + // expected-error@-1{{cannot take address of 'std::one_overload' because functions in namespace 'std' are not addressable}

[PATCH] D101598: [clang][Sema] adds `[[clang::no_address]]` attribute

zoecarver added inline comments. Comment at: clang/include/clang/Basic/AttrDocs.td:5972 + Although opt-in, the attribute poisons the entire overload set, meaning that only one overload + needs the attribute per set (although such usage is discouraged). Until there is confiden

[PATCH] D101598: [clang][Sema] adds `[[clang::no_address]]` attribute

cjdb created this revision. cjdb added reviewers: rsmith, gbiv, aaron.ballman, manojgupta. Herald added a reviewer: george.burgess.iv. cjdb requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. [namespace.std]/p6 indicates that taking the address