[PATCH] D51447: [WebAssembly] clang-format (NFC)

Derek Schuff via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 29 13:05:56 PDT 2018


dschuff added a comment.

If we want to make concerted progress toward full clang-formattedness (I like the idea in general, but opinions sometimes differ on the value of that) we could do it one file at a time, to be landed before making some change to that file. That would be a sort of compromise; it would keep the changes small and manageable, but it would be a larger granularity and scope than what you'd strictly need if you were going to make a small change or just work on one component (which would mean faster progress).

If nothing else though, here's your reminder that 'git clang-format' is a great tool and can automatically format just your diff against master. So that's an easy way to ensure we at least have things formatted as we check them in.


Repository:
  rL LLVM

https://reviews.llvm.org/D51447





More information about the llvm-commits mailing list