Adjustments for (upcoming) LLVM-23 and Clang's new offload driver#1219
Conversation
a6a33cf to
e5966ce
Compare
|
/run-aurora-ci |
|
/run-aurora-ci |
Yeah I got it to the stage where things were working but since LLMV is still getting commits in I didn't see the point in doing further work until LLVM side PRs are merged. |
e5966ce to
b4d2a57
Compare
Fair enough. We (PoCL) have a goal to have the chipStar working on non-forked Clang before the LLVM-23 release. Given your on-going LLVM upstream efforts (MacOS and SPIR-V BE support for HIPSPV and maybe more?), I'm concerned if we are going to miss that set goal if we need to wait them to be merged first. |
b4d2a57 to
55210ee
Compare
|
@linehill sorry I missed this comment! |
No, this PR is based on the recent state of the LLVM main branch - last time tested on a LLVM revision from the last week. |
55210ee to
61ca5a3
Compare
|
llvm/llvm-project#183991 MacOS support has been merged. We don't have to wait on the others I don't want to block your pocl progress |
This patch updates chipStar to work on non-forked, upstream LLVM-23. The recent Clang has switched to use its new offload driver by default for HIP compilation and chipStar is adjusted to it. There is a still an option to use the old while it still around in Clang. That said, chipStar doesn't yet work on the recent Clang until the following upstream patch lands: llvm/llvm-project#187655.
61ca5a3 to
fb46b44
Compare
|
Rebased.
I tested this PR on Ubuntu 24.04/x86_64 on the recent LLVM which includes this LLVM PR - all seems to be good.
If the CIs turn green, would this PR be good for landing? |
|
@linehill Can you check if your rebase includes llvm/llvm-project#190443? |
No, but I'll check it. |
|
Shouldn't affect anything. I'll make a new release candidate |
|
Thanks for the review and merge.
Checked on the LLVM featuring the linked PR locally: all seems to be good. |
btw, we can use the non-forked clang since LLVM 22.0 + Native SPIRV backend. All tests pass. |
This patch updates chipStar to work on non-forked, upstream LLVM-23. The recent Clang has switched to use its new offload driver by default for HIP compilation and chipStar is adjusted to it. There is a still an option to use the old while it still around in Clang.
That said, chipStar doesn't yet work on the recent Clang until the following upstream patch lands:
llvm/llvm-project#187655.