diff options
| author | Nick Sarnie <nick.sarnie@intel.com> | 2025-02-12 05:04:53 +0900 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-02-11 20:04:53 +0000 |
| commit | c2fea0d837c481c2c1f00077778f84de54c0196f (patch) | |
| tree | cc334a49e6570a15c7e8ea083aaca194f247acfb /lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.h | |
| parent | 401f4b828c374a01de6acd7fd8f4a3a4245f7595 (diff) | |
| download | llvm-c2fea0d837c481c2c1f00077778f84de54c0196f.zip llvm-c2fea0d837c481c2c1f00077778f84de54c0196f.tar.gz llvm-c2fea0d837c481c2c1f00077778f84de54c0196f.tar.bz2 | |
[clang-linker-wrapper][lit] Fix SPIR-V ELF test when spirv-tools feature is available (#126756)
My last change made the test not run when the `spirv-tools` feature is
not available, which is always the case in CI for clang tests, but it
fails if `spirv-tools` is available for the following reasons:
1) We didn't build `spirv-link` as part of the internal `SPIRV-Tools`
build, which is required by the `clang` call in `clang-linker-wrapper`,
I already fixed that
[here](https://github.com/llvm/llvm-project/pull/126319).
2) We didn't depend on the `SPIRV-Tools` CMake targets in clang tests,
so depending on what CMake targets were built before running
`check-clang`, `SPIRV-Tools` might not have been built.
3) We didn't check for `llvm-spirv` being available, which is not part
of `SPIRV-Tools` but is currently required for SPIR-V compilation.
Manually confirmed this works. This test is the bane of my existence.
---------
Signed-off-by: Sarnie, Nick <nick.sarnie@intel.com>
Diffstat (limited to 'lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.h')
0 files changed, 0 insertions, 0 deletions