[PATCH] D126783: [NFC] clang-format InstructionSimplify.cpp
Renato Golin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 1 08:50:45 PDT 2022
rengolin added a comment.
I don't know how people feel nowadays about clang-formatting an entire file.
In the past, we used to only format around local changes and hope that, in time, it would converge.
The bad side of formatting everything is that git-blame needs one more level of indirection. It's not terrible, though.
I'd ask other people around how they still feel about it. I don't personally mind, but I also don't personally work on this file often.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126783/new/
https://reviews.llvm.org/D126783
More information about the llvm-commits
mailing list