Skip to content

Adjustments for (upcoming) LLVM-23 and Clang's new offload driver#1219

Merged
pvelesko merged 1 commit into
CHIP-SPV:mainfrom
linehill:clang-new-offload-driver-take2
Apr 29, 2026
Merged

Adjustments for (upcoming) LLVM-23 and Clang's new offload driver#1219
pvelesko merged 1 commit into
CHIP-SPV:mainfrom
linehill:clang-new-offload-driver-take2

Conversation

@linehill

Copy link
Copy Markdown
Collaborator

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.

@pvelesko pvelesko force-pushed the clang-new-offload-driver-take2 branch from a6a33cf to e5966ce Compare April 6, 2026 14:23
@colleeneb

Copy link
Copy Markdown
Contributor

/run-aurora-ci

@colleeneb

Copy link
Copy Markdown
Contributor

/run-aurora-ci

@pvelesko

pvelesko commented Apr 8, 2026

Copy link
Copy Markdown
Collaborator

We already have #1194

This PR is missing CI for llvm 23 @linehill anything wrong with the existing PR?

@linehill

linehill commented Apr 8, 2026

Copy link
Copy Markdown
Collaborator Author

We already have #1194

This PR is missing CI for llvm 23 @linehill anything wrong with the existing PR?

That one haven't seen progress for a while and its changes are based on an old LLVM revision that wouldn't work on the more recent one.

@pvelesko

pvelesko commented Apr 9, 2026

Copy link
Copy Markdown
Collaborator

That one haven't seen progress for a while and its changes are based on an old LLVM revision that wouldn't work on the more recent one.

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.

@linehill

linehill commented Apr 9, 2026

Copy link
Copy Markdown
Collaborator Author

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.

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.

@linehill linehill force-pushed the clang-new-offload-driver-take2 branch from b4d2a57 to 55210ee Compare April 17, 2026 08:01
@pvelesko

Copy link
Copy Markdown
Collaborator

@linehill sorry I missed this comment!
I don't think any of my upstream LLVM PRs are blocking anything? Or am I missing something?

@linehill

Copy link
Copy Markdown
Collaborator Author

@linehill sorry I missed this comment! I don't think any of my upstream LLVM PRs are blocking anything? Or am I missing something?

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.

@linehill linehill force-pushed the clang-new-offload-driver-take2 branch from 55210ee to 61ca5a3 Compare April 23, 2026 08:35
@pvelesko

Copy link
Copy Markdown
Collaborator

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.
@linehill linehill force-pushed the clang-new-offload-driver-take2 branch from 61ca5a3 to fb46b44 Compare April 29, 2026 10:00
@linehill

Copy link
Copy Markdown
Collaborator Author

Rebased.

llvm/llvm-project#183991 MacOS support has been merged.

I tested this PR on Ubuntu 24.04/x86_64 on the recent LLVM which includes this LLVM PR - all seems to be good.

We don't have to wait on the others I don't want to block your pocl progress

If the CIs turn green, would this PR be good for landing?

@pvelesko

pvelesko commented Apr 29, 2026

Copy link
Copy Markdown
Collaborator

@linehill Can you check if your rebase includes llvm/llvm-project#190443?

@pvelesko pvelesko left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pvelesko pvelesko merged commit dd563ea into CHIP-SPV:main Apr 29, 2026
17 checks passed
@linehill

Copy link
Copy Markdown
Collaborator Author

@linehill Can you check if your rebase includes llvm/llvm-project#190443?

No, but I'll check it.

@pvelesko

Copy link
Copy Markdown
Collaborator

Shouldn't affect anything. I'll make a new release candidate

@linehill

Copy link
Copy Markdown
Collaborator Author

Thanks for the review and merge.

@linehill Can you check if your rebase includes llvm/llvm-project#190443?

No, but I'll check it.

Checked on the LLVM featuring the linked PR locally: all seems to be good.

@pvelesko

pvelesko commented May 4, 2026

Copy link
Copy Markdown
Collaborator

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.

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.

btw, we can use the non-forked clang since LLVM 22.0 + Native SPIRV backend. All tests pass.

@linehill

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants