rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land.
Looks good from a driver perspective. Aaron, you're happy with the attribute spelling stuff, right? It looks like you can actually land this before landing the two dependent changes you mention and the tests will pass. I'd go ahead and land it to avoid the need to rebase, but it's up to you. ================ Comment at: include/clang/Basic/AttrDocs.td:2459 @@ +2458,3 @@ + +Conversely, ``__attribute__((xray_never_instrument))`` or ``[[clang:xray_never_instrument]]`` will inhibit the insertion of these instrumentation points. + ---------------- These paragraphs can be wrapped to 80 cols like the others. http://reviews.llvm.org/D20352 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
