sjnewbury / rocm-bleeding-edge

Completely unofficial Gentoo ROCm support
GNU General Public License v3.0
9 stars 2 forks source link

Use of "get_llvm_prefix" function in ebuilds requires "llvm" in inherit line #1

Open mattmcadoo opened 6 months ago

mattmcadoo commented 6 months ago

Without "llvm":

` * Package: sci-libs/rocSPARSE-6.0.2:0/6.0

With "llvm" on line 9 of sci-libs/rocSPARSE/rocSPARSE-6.0.2.ebuild:

` * Package: sci-libs/rocSPARSE-6.0.2:0/6.0

-- Detecting C compiler ABI info -- Detecting C compiler ABI info - done -- Check for working C compiler: /usr/lib/llvm/18/bin/clang - skipped -- Detecting C compile features -- Detecting C compile features - done -- Configuring incomplete, errors occurred!

Getting the same kind of failure with: dev-util/roctracer/roctracer-6.0.2.ebuild sci-libs/rocBLAS/rocBLAS-5.7.1.ebuild

There's probably more but these are the prerequisites.

mattmcadoo commented 6 months ago

Github butchered the formatting of trying to keep in all in code block...

sjnewbury commented 6 months ago

I should probably make all the llvm build stuff optional. My main reason for forcing it is the lack of compiler flag support explicitly for the HIP compiler vs C++. An alternative could be to judiciously filter the flags or just make sure all the packages properly support HIPFLAGS.