Skip to content

Retarget RVY LLVM to the correct branch.#463

Open
qwattash wants to merge 1 commit intomainfrom
llvm-rvy-repo-update
Open

Retarget RVY LLVM to the correct branch.#463
qwattash wants to merge 1 commit intomainfrom
llvm-rvy-repo-update

Conversation

@qwattash
Copy link
Contributor

@qwattash qwattash commented Mar 3, 2026

Use temporary override to use known-working patches until these are merged in the CHERI Alliance repository.

Use temporary override to use known-working patches until these
are merged in the CHERI Alliance repository.
@qwattash qwattash requested review from arichardson and jrtc27 March 3, 2026 13:13
@jrtc27
Copy link
Member

jrtc27 commented Mar 3, 2026

CC @heshamelmatary given this will affect Linux and seL4

@heshamelmatary
Copy link
Member

If this is the same as the PR against CA, then yeah I'd like it merged ASAP for CHERI-Linux (and I tested it).

I'll test for CHERI-seL4 still. It should remove a function or two I added that clears the tags there, and probably PCC get in hybrid mode?

@jrtc27
Copy link
Member

jrtc27 commented Mar 3, 2026

Yeah it's pointing at the branch being used for that PR. If you're happy then I'm happy.

@heshamelmatary
Copy link
Member

Yeah it's pointing at the branch being used for that PR. If you're happy then I'm happy.

There's just a slight fix in the Linux triple (to remove _purecap), I can submit a separate fix or integrate it in #451

@heshamelmatary
Copy link
Member

Currently CHERI-seL4 tests fault with this branch with a non-CHERI trap for some reason. I'll need to debug that. Could you please hold off merging this as it will break CHERI-seL4?

@qwattash
Copy link
Contributor Author

qwattash commented Mar 6, 2026

Are we happy for this to land?

@jrtc27
Copy link
Member

jrtc27 commented Mar 6, 2026

@heshamelmatary any progress on the seL4 issue?

@jrtc27
Copy link
Member

jrtc27 commented Mar 6, 2026

(Note this is going from an old LLVM 17-based Codasip branch to a newer LLVM 19-based one, so it's not so surprising there are issues...)

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