zaks.anna added inline comments.

================
Comment at: include/clang/StaticAnalyzer/Checkers/CMakeLists.txt:3
@@ +2,3 @@
+  -I ${CMAKE_CURRENT_SOURCE_DIR}/../../../
+  SOURCE ../../../../lib/StaticAnalyzer/Checkers/Checkers.td
+  TARGET ClangSACheckers)
----------------
srhines wrote:
> Anna should probably decide here, but I think it might be preferable to 
> actually move Checkers.td into include/..., since it is indeed being used 
> beyond just the lib/... directory it currently resides in (hence the original 
> problem here of a bizarre include path in clang/tools/extra).
+1 
If you use it outside of lib/StaticAnalyzer, feel free to move it.


http://reviews.llvm.org/D19393



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

Reply via email to