[clang] [CMake] Change GCC_INSTALL_PREFIX from warning to fatal error (PR #85891)

2024-04-23 Thread Harmen Stoppels via cfe-commits
haampie wrote: Users of `GCC_INSTALL_PREFIX` typically have GCC installed in a unique, non-standard prefix, where clang's default search just worked. Now those users have to generate config files for post-install, AND figure out how to make clang behave correctly when building runtimes. That's

[clang] [CMake] Change GCC_INSTALL_PREFIX from warning to fatal error (PR #85891)

2024-04-24 Thread Harmen Stoppels via cfe-commits
haampie wrote: I disagree, `--gcc-install-dir` is sure an improvement over `--gcc-toolchain`, but they're both weaker than the compile time option `GCC_INSTALL_PREFIX` because of runtimes. You're looking to remove `GCC_INSTALL_PREFIX`, then give a clear alternative that's equivalent. The curr

[clang] [AMDGPU] Remove detection of hip runtime for Spack (PR #133263)

2025-04-02 Thread Harmen Stoppels via cfe-commits
haampie wrote: @yxsamliu can you land this? I cannot https://github.com/llvm/llvm-project/pull/133263 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [AMDGPU] Remove detection of hip runtime for Spack (PR #133263)

2025-03-27 Thread Harmen Stoppels via cfe-commits
https://github.com/haampie edited https://github.com/llvm/llvm-project/pull/133263 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [AMDGPU] Remove detection of hip runtime for Spack (PR #133263)

2025-03-27 Thread Harmen Stoppels via cfe-commits
https://github.com/haampie edited https://github.com/llvm/llvm-project/pull/133263 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [AMDGPU] Remove detection of hip runtime for Spack (PR #133263)

2025-03-27 Thread Harmen Stoppels via cfe-commits
https://github.com/haampie edited https://github.com/llvm/llvm-project/pull/133263 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [AMDGPU] Remove detection of hip runtime for Spack (PR #133263)

2025-03-27 Thread Harmen Stoppels via cfe-commits
haampie wrote: I believe Spack sets either `HIP_PATH` or `ROCM_PATH` for clang which is considered before the auto-detection removed in this PR. https://github.com/llvm/llvm-project/pull/133263 ___ cfe-commits mailing list cfe-commits@lists.llvm.org h

[clang] [AMDGPU] Remove detection of hip runtime for Spack (PR #133263)

2025-03-27 Thread Harmen Stoppels via cfe-commits
https://github.com/haampie created https://github.com/llvm/llvm-project/pull/133263 There is special logic to detect the hip runtime when llvm is installed with Spack. It works by matching the install prefix of llvm against `llvm-amdgpu-*` followed by effectively globbing for ``` /../hip-x.y.z-