Skip to content

Commit 670cc1f

Browse files
j2kuncopybara-github
authored andcommitted
Remove NVPTX and AMDGPU from LLVM targets in llvm_configure
When llvm/llvm-project#187213 lands, removing these lines should no longer break the build PiperOrigin-RevId: 888142864
1 parent 92c26b2 commit 670cc1f

1 file changed

Lines changed: 0 additions & 8 deletions

File tree

MODULE.bazel

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -67,14 +67,6 @@ use_repo(
6767
# The subset of LLVM backend targets that should be compiled
6868
_LLVM_TARGETS = [
6969
"X86",
70-
# The bazel dependency graph for mlir-opt fails to load (at the analysis
71-
# step) without the NVPTX and AMDGPU targets in this list, because, e.g.,
72-
# mlir/test:TestGPU depends on the //llvm:NVPTXCodeGen target, which is not
73-
# defined unless this is included. @j2kun asked the LLVM maintiners for
74-
# tips on how to fix this, see
75-
# https://github.com/llvm/llvm-project/issues/63135
76-
"NVPTX",
77-
"AMDGPU",
7870
# Needed for Apple M1 targets, see
7971
# https://github.com/j2kun/mlir-tutorial/issues/11
8072
"AArch64",

0 commit comments

Comments
 (0)