[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