[llvm-branch-commits] [llvm] [WIP][CodeGen][DebugInfo][RISCV] Support scalable offsets in CFI (PR #170607)
Mikhail Gudim via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Thu Dec 11 16:47:00 PST 2025
mgudim wrote:
> > @ppenzin I am in the process of posting the commits myself, this may not be entirely ready. Please close these PRs.
>
> I'm entirely sympathetic to your wanting these commits credited to you, but I've been finding these _much_ easier to understand than the existing patches.
I should clarify the situation with branches / patches:
(1) There exist 2 POC branches:
the original one: https://github.com/llvm/llvm-project/pull/90819
and rebased on a more recent, but still old `main`: https://github.com/mgudim/llvm-project/tree/rebased_save_csr_in_ra
These have one huge commit and are impossible to understand, they are not for review, they are just proof-of-concept.
(2) I have another branch: https://github.com/mgudim/llvm-project/tree/save_csr_in_ra3 where I split up all the work from (1) into small commits. This branch is still in the broken state and it is missing 3 - 4 commits.
(3) As commits are ready in (2) and can be merged into `main` I am posting them as individual PRs:
https://github.com/llvm/llvm-project/pull/168869
https://github.com/llvm/llvm-project/pull/168531
https://github.com/llvm/llvm-project/pull/166773
https://github.com/llvm/llvm-project/pull/166763
https://github.com/llvm/llvm-project/pull/164480
This commit was just taken from (2). I didn't post it myself because it's not ready yet, I still have above 5 PRs to merge.
> I really don't want to loose that until your patches are in a state where they really do explain what is being proposed.
Sure we can keep it just to save this discussion, but in terms of content this is just a commit from (2).
I am going to post this explanation on the original PR too.
https://github.com/llvm/llvm-project/pull/170607
More information about the llvm-branch-commits
mailing list