[HLSL][CMake] Add clangd and distribution settings

[llvm-beanz]

This just adds some simple distribution settings and includes clangd in the build for distribution.

[llvmbot]

@llvm/pr-subscribers-clang

Author: Chris B (llvm-beanz)

Changes

This just adds some simple distribution settings and includes clangd in the build for distribution.

---

Full diff: https://github.com/llvm/llvm-project/pull/92011.diff

1 Files Affected:

• (modified) clang/cmake/caches/HLSL.cmake (+5-1)

diff --git a/clang/cmake/caches/HLSL.cmake b/clang/cmake/caches/HLSL.cmake
index 84850c86f12cd..9aa28625ab81e 100644
--- a/clang/cmake/caches/HLSL.cmake
+++ b/clang/cmake/caches/HLSL.cmake
@@ -8,6 +8,10 @@ set(LLVM_EXPERIMENTAL_TARGETS_TO_BUILD "DirectX;SPIRV" CACHE STRING "")
 
 # HLSL support is currently limted to clang, eventually it will expand to
 # clang-tools-extra too.
-set(LLVM_ENABLE_PROJECTS "clang" CACHE STRING "")
+set(LLVM_ENABLE_PROJECTS "clang;clang-tools-extra" CACHE STRING "")
 
 set(CLANG_ENABLE_HLSL On CACHE BOOL "")
+
+set(LLVM_DISTRIBUTION_COMPONENTS
+    "clang;hlsl-resource-headers;clangd"
+    CACHE STRING "")

[llvmbot]

@llvm/pr-subscribers-hlsl

Author: Chris B (llvm-beanz)

Changes

This just adds some simple distribution settings and includes clangd in the build for distribution.

---

Full diff: https://github.com/llvm/llvm-project/pull/92011.diff

1 Files Affected:

• (modified) clang/cmake/caches/HLSL.cmake (+5-1)

diff --git a/clang/cmake/caches/HLSL.cmake b/clang/cmake/caches/HLSL.cmake
index 84850c86f12cd..9aa28625ab81e 100644
--- a/clang/cmake/caches/HLSL.cmake
+++ b/clang/cmake/caches/HLSL.cmake
@@ -8,6 +8,10 @@ set(LLVM_EXPERIMENTAL_TARGETS_TO_BUILD "DirectX;SPIRV" CACHE STRING "")
 
 # HLSL support is currently limted to clang, eventually it will expand to
 # clang-tools-extra too.
-set(LLVM_ENABLE_PROJECTS "clang" CACHE STRING "")
+set(LLVM_ENABLE_PROJECTS "clang;clang-tools-extra" CACHE STRING "")
 
 set(CLANG_ENABLE_HLSL On CACHE BOOL "")
+
+set(LLVM_DISTRIBUTION_COMPONENTS
+    "clang;hlsl-resource-headers;clangd"
+    CACHE STRING "")

[bogner]

Including clang-tools-extra makes sense to me. Setting up the distribution may not be as universal but it seems fine too as a default in this cache file.

LGTM

[bogner]

If we're going to define a distribution components we might want clang-dxc in there as well

[llvm-beanz]

The clang symlinks are all part of the clang component, so they all get installed together. We should probably change that and have some which are part of the default target (clang, clang++), and others that aren't (like clang-dxc).