[PATCH] D108479: [Clang] Add __builtin_addressof_nocfi

[John McCall via Phabricator via cfe-commits]

rjmccall added a comment.

In D108479#3113055 <https://reviews.llvm.org/D108479#3113055>, @pcc wrote:

> (Adding back @rsmith, @rjmccall.)
>
> In D108479#3113035 <https://reviews.llvm.org/D108479#3113035>, @samitolvanen wrote:
>
>> In D108479#3112492 <https://reviews.llvm.org/D108479#3112492>, @rjmccall wrote:
>>
>>> You could also make this Just Work for things like C++ member functions rather than producing a member function pointer.
>>
>> I'm not sure I understand. What does Just Work mean when it comes to C++ member functions?
>
> I think he means that e.g. `__builtin_symbol_address(&Foo::bar)` should return a `void*` pointing to the address of the `Foo::bar` member function body, instead of a member function pointer for `Foo::bar`.

Yes, exactly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108479