[clang] [llvm] [clang] missing changes to the Rewriter (PR #152845)
Matheus Izvekov via llvm-commits
llvm-commits at lists.llvm.org
Sat Aug 9 10:05:48 PDT 2025
mizvekov wrote:
According to the author of the change, the feature is sort of obsolete and probably a waste of most users time to enable it.
But if we are not removing it, then it needs testing, so its better to just enable it on CI anyway.
Splitting the change would mean we'd either break all of linux pre-commit CI, and we would have to commit it and keep it in red until the second change lands, or we would need to commit this blind first without any testing, and only test it later.
I'd have a slight preference to keep both changes together.
https://github.com/llvm/llvm-project/pull/152845
More information about the llvm-commits
mailing list