rnk accepted this revision. rnk added subscribers: probinson, aprantl, dblaikie. rnk added a comment. This revision is now accepted and ready to land.
lgtm --- Clang's builtin headers use `__attribute__((__nodebug__))` for this purpose. Do you think we should follow this up by using artificial instead? It seems like it would be a representational improvement. @aprantl @probinson @dblaikie ================ Comment at: test/Sema/artificial.c:3 + +void __attribute__((artificial)) bar() {} // expected-warning {{'artificial' attribute only applies to inline functions}} ---------------- I think it's worth adding the `foo` function from the CodeGen test here to show we don't generate warnings when the function is inline specified. https://reviews.llvm.org/D43259 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits