[llvm-commits] [llvm] r61545 - in /llvm/trunk/lib/CodeGen/SelectionDAG: LegalizeTypes.h LegalizeVectorTypes.cpp

Misha Brukman brukman at gmail.com
Fri Jan 2 13:24:54 PST 2009


This isn't a commit script, but some lint tools to check existing (and new)
code to make sure it conforms to the style guide: svn up and see
llvm/utils/lint/* .

% cd llvm

% utils/lint/cpp_lint.py `find lib/ -name \*\.h` | wc -l
11175
% utils/lint/cpp_lint.py `find include/ -name \*\.h` | grep whitespace | wc
-l
11063

% utils/lint/cpp_lint.py `find lib/ -name \*\.cpp` | wc -l
35265
% utils/lint/cpp_lint.py `find lib/ -name \*\.cpp` | grep whitespace | wc -l

34881

We clearly have a ways to go, but
llvm/utils/lint/remove_trailing_whitespace.sh can fix the majority of these
issues.

2009/1/1 Bill Wendling <isanbard at gmail.com>

> And tab and >80 columns. ;-)
>
> -bw
>
> On Jan 1, 2009, at 12:19 PM, Duncan Sands <baldrick at free.fr> wrote:
>
> >> Thank you!  That's one of my pet peeves. :-)
> >> I keep seeing trailing spaces everywhere; I'm considering removing
> >> them all
> >> (via "perl -pi").
> >
> > If you do, then maybe you should also add a subversion commit script
> > that rejects
> > changes which introduce trailing spaces.
> >
> > Ciao,
> >
> > Duncan.
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20090102/b9506229/attachment.html>


More information about the llvm-commits mailing list