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 _______________________________________________ llvm-branch-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
