clangd needs them to function properly, even though they are not
strictly required for the build.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
- Build Status
Buildable 32518 Build 32517: arc lint + arc unit
Event Timeline
Comment Actions
Thanks for the patch!
Does the cmake build have this dependency? If so, where? I tried to find it for a bit but failed. Alternatively, if there's a reason why the gn build needs this but the cmake build doesn't, that works too :)
Comment Actions
The dependency is there in CMake, although it took me awhile to dig it out for first time.
It is added by add_clang_tool, see AddClang.cmake.
Obviously, we want the same dependency on other instances of add_clang_tool. The lack of this dependency can only be noticed when building a tool in a clean build directory without building clang. Worse yet, it will build just fine, but would fail to find the builtin headers at runtime.