-
Notifications
You must be signed in to change notification settings - Fork 14.3k
[HLSL] Include SPIRV in LLVM_TARGETS_TO_BUILD in the HLSL cmake cache #133287
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Since SPIRV is no longer an experimental target this wasn't actually enabling it any more.
@llvm/pr-subscribers-hlsl @llvm/pr-subscribers-clang Author: Justin Bogner (bogner) ChangesSince SPIRV is no longer an experimental target this wasn't actually enabling it any more. Full diff: https://github.com/llvm/llvm-project/pull/133287.diff 1 Files Affected:
diff --git a/clang/cmake/caches/HLSL.cmake b/clang/cmake/caches/HLSL.cmake
index ed813f60c9c69..24afcbbda2a40 100644
--- a/clang/cmake/caches/HLSL.cmake
+++ b/clang/cmake/caches/HLSL.cmake
@@ -1,13 +1,10 @@
# Including the native target is important because some of LLVM's tests fail if
# you don't.
-set(LLVM_TARGETS_TO_BUILD Native CACHE STRING "")
+set(LLVM_TARGETS_TO_BUILD "Native;SPIRV" CACHE STRING "")
-# Include the DirectX target for DXIL code generation, eventually we'll include
-# SPIR-V here too.
-set(LLVM_EXPERIMENTAL_TARGETS_TO_BUILD "DirectX;SPIRV" CACHE STRING "")
+# Include the DirectX target for DXIL code generation.
+set(LLVM_EXPERIMENTAL_TARGETS_TO_BUILD "DirectX" CACHE STRING "")
-# HLSL support is currently limted to clang, eventually it will expand to
-# clang-tools-extra too.
set(LLVM_ENABLE_PROJECTS "clang;clang-tools-extra" CACHE STRING "")
set(CLANG_ENABLE_HLSL On CACHE BOOL "")
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, not an expert on cmake thought
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/88/builds/10100 Here is the relevant piece of the build log for the reference
|
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/64/builds/2839 Here is the relevant piece of the build log for the reference
|
Since SPIRV is no longer an experimental target this wasn't actually enabling it any more.