Commit bfba621
authored
[clang-linker-wrapper][lit] Fix OpenMP SPIR-V ELF test again (#126142)
I was able to reproduce the issue with the sanitizer buildbot scripts
and confirmed this fixes it.
The issue was the quotes, `"0"` is true in Python so we incorrectly
added the `spirv-tools` feature even when the CMake variable was false.
I don't know why it didn't always fail.
Also add the var to clang's `BUILD.gn` which matches what we do for
other similar variables, however I don't think it has any effect on CI
here.
Signed-off-by: Sarnie, Nick <nick.sarnie@intel.com>1 parent 7eaaa4e commit bfba621
File tree
3 files changed
+2
-3
lines changed- clang/test
- Tooling
- llvm/utils/gn/secondary/clang/test
3 files changed
+2
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | | - | |
3 | | - | |
4 | 2 | | |
5 | 3 | | |
6 | 4 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
43 | 43 | | |
44 | 44 | | |
45 | 45 | | |
46 | | - | |
| 46 | + | |
47 | 47 | | |
48 | 48 | | |
49 | 49 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
75 | 75 | | |
76 | 76 | | |
77 | 77 | | |
| 78 | + | |
78 | 79 | | |
79 | 80 | | |
80 | 81 | | |
| |||
0 commit comments