================ @@ -10,6 +10,7 @@ // //===----------------------------------------------------------------------===// +#include "../Basic/Targets/NVPTX.h" ---------------- sam-mccall wrote:
This header is not part of `clangBasic`'s interface, but rather its implementation (`lib/Basic` rather than `include/clang/Basic`). Sema can't depend on it - if you need to use its APIs from outside clangBasic they should be moved to a public header. The bazel build shows the problem: https://buildkite.com/llvm-project/upstream-bazel/builds/75928#018ad568-2f7c-4dda-ae90-3b4d787caad7. Breaking bazel is not itself reason to revert, but here it's flagging a real problem that CMake doesn't catch. I've reverted as 0afbcb20fd908f8bf9073697423da097be7db592 - sorry to do this so abruptly, but I can't fix this myself & such problems block downstream use of LLVM. https://github.com/llvm/llvm-project/pull/66496 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
