[PATCH] D77491: [Sema] Introduce BuiltinAttr, per-declaration builtin-ness

dmajor via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 18 09:40:44 PDT 2020


dmajor added a comment.

This commit broke Firefox builds on Mac with an error in the SDK headers. Could you please revert if a fix is not readily available?

Reproducer:

  struct objc_super {};
  extern "C" id objc_msgSendSuper(struct objc_super *super, SEL op, ...);
  extern "C" void objc_msgSendSuper_stret(struct objc_super *super, SEL op, ...);

Result:

  $ clang -c test.mm
  test.mm:3:48: error: reference to 'objc_super' is ambiguous
  extern "C" void objc_msgSendSuper_stret(struct objc_super *super, SEL op, ...);
                                                 ^
  test.mm:1:8: note: candidate found by name lookup is 'objc_super'
  struct objc_super {};
         ^
  note: candidate found by name lookup is 'objc_super'
  1 error generated.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77491



More information about the cfe-commits mailing list