[llvm-branch-commits] [clang] [Serialization] Code cleanups and polish 83233 (PR #83237)

Chuanqi Xu via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Thu Dec 5 18:22:17 PST 2024


ChuanqiXu9 wrote:

Note that I'll merge this and https://github.com/llvm/llvm-project/pull/83233 and https://github.com/llvm/llvm-project/pull/83108 into a commit and commit it directly to trunk. Since these prs are split initially to make it clear to be reviewed. But during the review, we always add changes to this PR. So the meaning of stacked PR in this series of patches is pretty questionable. So I feel it is better to merge these PRs into a single commit so that it is easier to be cherry-picked and reverted in my experience. I'll add a link to the PR page in the commit message.

Thanks every one here again.

https://github.com/llvm/llvm-project/pull/83237


More information about the llvm-branch-commits mailing list