[llvm] [NFC][AArch64] Refactor AArch64ConditionOptimizer (PR #177521)
David Spickett via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 18 05:14:16 PST 2026
DavidSpickett wrote:
A good way to do this split is to make a branch locally that has all the changes split into commits. Then you know you can get to the end point successfully. Then make a branch with the first commit and send that as a PR. Update that as required, and once it's merged, rebase your full branch onto main. Make another branch with the next new commit, and so on.
You can also send a PR that has multiple commits and reviewers can review each commit (they will get merged later anyway). However, in the case of this PR's changes, I would prefer each step to be merged with it's own message so multiple PRs is what I'd do.
https://github.com/llvm/llvm-project/pull/177521
More information about the llvm-commits
mailing list