[PATCH] D54188: [Sema] Mark target of __attribute__((alias("target"))) used for C

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 9 13:44:43 PST 2019


rjmccall added inline comments.


================
Comment at: test/Sema/alias-unused.c:3
+// expected-no-diagnostics
+int f() { return 42; }
+int g() __attribute__((alias("f")));
----------------
nickdesaulniers wrote:
> rjmccall wrote:
> > Is this meant to be `static`?
> It doesn't make a difference for this test.  Would you like me to add it before submitting?
It's what you're trying to test, right?  `f()` is not otherwise an internal declaration, so of course the warning won't fire for it.


Repository:
  rC Clang

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

https://reviews.llvm.org/D54188





More information about the cfe-commits mailing list