- Use llvm releases of libclc
- Force llvm12 as it is what's used for mesa/clover
- Use the llvm12 flavor of spirv-llvm-translator
Details
Details
- Reviewers
jbeich - Group Reviewers
x11 - Commits
- R11:7268d3bb1379: devel/libclc: Update to 12.0.1
Not tested
Diff Detail
Diff Detail
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
Comment Actions
Moving to libclc from llvm12 needs llvm-spirv ...
llvm-spirv dependency can be dropped:
post-patch:
# Exclude targets depending on llvm-spirv
@${REINPLACE_CMD} -e '/^[[:space:]]*spirv.*mesa/d' \
${WRKSRC}/CMakeLists.txtor after 9b3e3e4b33ca fixed with:
BUILD_DEPENDS+= llvm-spirv:devel/spirv-llvm-translator
# Can be a relative path e.g., -DLLVM_SPIRV:FILEPATH=llvm-spirv
CMAKE_ARGS+= -DLLVM_SPIRV:FILEPATH="${LOCALBASE}/bin/llvm-spirv"Caveat: spirv-llvm-translator currently uses llvm11 because intel-graphics-compiler upstream keeps reverting llvm12 support.
Comment Actions
I don't use this port, so not interested in thoroughly reviewing.
| devel/libclc/Makefile | ||
|---|---|---|
| 25 | Either qualify type by adding :FILEPATH then quote the path or use relative path e.g., -DLLVM_CONFIG=llvm-config12. Such style appears to be preferred by devel/cmake maintainer. | |
Comment Actions
After c638dc3a4dc4 libclc >= 12.* will automatically find llvm-spirv without adjusting CMAKE_ARGS if the dependency is installed for the desired LLVM version.