[PATCH] D20352: Add XRay flags to Clang

Reid Kleckner via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 7 11:32:37 PDT 2016


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





More information about the cfe-commits mailing list