[llvm] [llvm][DebugInfo][DWARF] Use formatv() instead of format() (PR #191308)
David Spickett via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 13 09:06:32 PDT 2026
https://github.com/DavidSpickett commented:
I agree with how you've split it but this PR still needs to be smaller.
I'm a few files into this and it's still way too much for a single PR. Each change is "simple" but I'm struggling to concentrate with so many very similar changes to look at. Can we get say, 5 files max? And split the tests into their own PR, I see their utility but they can land on their own.
I don't disagree with what I've read so far though, and I think my only objections are going to be some casts added or removed.
https://github.com/llvm/llvm-project/pull/191308
More information about the llvm-commits
mailing list