Hi Petter,

On 2025-01-30 14:02, Petter Reinholdtsen wrote:
Did you try to set HIPCXX to clang-17, something like this?

   "HIPCXX=clang-17 cmake"

It helped me when trying to build llama.cpp.

The proper behaviour for this is an open question for me. By default, CMake doesn't consider the `-nn` suffixed executables when it is searching for a usable HIP compiler. It would be easy enough to change that in CMake, but it wasn't clear to me if it should be changed.

On one hand, forcing the user to specify `HIPCXX=clang++-17` makes build scripts fragile and contributes to HIP being more difficult to use. On the other hand, the CMake behaviour of ignoring `-nn` suffixed compilers was probably intentional for the C and CXX languages and it would be inconsistent to have the HIP language behave differently.

I personally lean slightly towards enabling CMake to explicitly consider the version-suffixed clang compilers when searching for a usable CMAKE_HIP_COMPILER, but I must admit that I am still quite uncertain.

Sincerely,
Cory  Bloor

Reply via email to