[PATCH] D58956: [WebAssembly] Remove tabs in tests (NFC)

Thomas Lively via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 7 11:16:39 PST 2019


tlively added a comment.

In D58956#1421823 <https://reviews.llvm.org/D58956#1421823>, @aardappel wrote:

> It would be awesome if we could somehow force tabs to be errors, frankly. Or is there some other automatic way to detect them? Or show them in code review?
>
> I know this changes is abandoned, but I am in favor of it.


+1. Perhaps we could all agree on a stand git pre-commit hook. I use one that doesn't let me commit unless I've run `git clang-format`, and I'm sure it could be extended to check for the absence of tabs as well.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58956/new/

https://reviews.llvm.org/D58956





More information about the llvm-commits mailing list