[PATCH] D31696: Automatically add include-what-you-use for when building in tree

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 6 17:45:14 PDT 2017


rnk added inline comments.


================
Comment at: clang/tools/.gitignore:12
+#==============================================================================#
+# The include-what-you-use project, for when building in-tree.
+include-what-you-use
----------------
LLVM doesn't currently have any .gitignore files for other projects, and they don't show up in `git status`. What's the difference here?


================
Comment at: clang/tools/CMakeLists.txt:35
+# if include-what-you-use is cloned for building in-tree, add it here.
+if (EXISTS "${CMAKE_CURRENT_SOURCE_DIR}/include-what-you-use")
+  add_clang_subdirectory(include-what-you-use)
----------------
Maybe we should do `llvm_add_implicit_projects(CLANG)` here instead?

Or do we not want clang/tools to be a project extension point? Would IWYU build fine if we added it to llvm/projects or llvm/tools? Maybe we should just recommend that.


https://reviews.llvm.org/D31696





More information about the cfe-commits mailing list