llvmbot wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Brian Cain (androm3da)

<details>
<summary>Changes</summary>

Since this cache value overrides the defaults, we end up with `clang` linked to 
`clang-20`, and some `${triple}-clang*` links, but we're missing `clang++`.  
This makes for a toolchain with inconsistent behavior when used in someone's 
`$PATH`.

We'll add the default symlinks to our list so that C and C++ programs are both 
built as expected when `clang` and `clang++` are invoked.

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


1 Files Affected:

- (modified) clang/cmake/caches/hexagon-unknown-linux-musl-clang-cross.cmake 
(+3) 


``````````diff
diff --git a/clang/cmake/caches/hexagon-unknown-linux-musl-clang-cross.cmake 
b/clang/cmake/caches/hexagon-unknown-linux-musl-clang-cross.cmake
index 91bbe26b621054..fd2aeec819fc01 100644
--- a/clang/cmake/caches/hexagon-unknown-linux-musl-clang-cross.cmake
+++ b/clang/cmake/caches/hexagon-unknown-linux-musl-clang-cross.cmake
@@ -10,6 +10,9 @@ set(CLANG_LINKS_TO_CREATE
             hexagon-none-elf-clang
             hexagon-unknown-none-elf-clang++
             hexagon-unknown-none-elf-clang
+            clang++
+            clang-cl
+            clang-cpp
             CACHE STRING "")
 
 set(LLVM_INSTALL_TOOLCHAIN_ONLY ON CACHE BOOL "")

``````````

</details>


https://github.com/llvm/llvm-project/pull/123011
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to