[PATCH] D158942: [bazel] Include builtin headers with clang-tidy

Jathu Satkunarajah via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 7 19:33:50 PDT 2023


jathu added a comment.

In D158942#4633201 <https://reviews.llvm.org/D158942#4633201>, @chapuni wrote:

> FYI, I was working for introducing `clang-resources` in D141755 <https://reviews.llvm.org/D141755>. Could we unify `clang-resources` in the `clang`?

This will not work since it needs to be located in the parent folder of where the binary is built — so it must exist in `utils/bazel/llvm-project-overlay/clang-tools-extra/lib/clang/{major version}/include`. FWIW also tried it be rebasing on your diff and get a runtime errors (headers not found)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D158942/new/

https://reviews.llvm.org/D158942



More information about the llvm-commits mailing list