rnk added a comment.

I think you need some driver logic to make sure the xray lists show up in .d 
files so that modifications to them trigger rebuilds in most build systems. See 
the SanitizerArgs.cpp driver logic for this.



================
Comment at: include/clang/Basic/XRayFunctionFilter.h:1
+//===--- XRayFunctionFilter.h - XRay automatic-attribution ------*- C++ 
-*-===//
+//
----------------
Do you think XRayLists.h might be a better name for this header? It's analagous 
to SanitizerBlacklist.h, but it's not a "black" list.


https://reviews.llvm.org/D30388



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to