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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits