This is an archive of the discontinued LLVM Phabricator instance.

Add .clang-tidy configuration file to provide LLVM-optimized defaults for clang-tidy.
ClosedPublic

Authored by alexfh on Sep 7 2014, 4:27 PM.

Diff Detail

Event Timeline

alexfh updated this revision to Diff 13379.Sep 7 2014, 4:27 PM
alexfh retitled this revision from to Add .clang-tidy configuration file to provide LLVM-optimized defaults for clang-tidy..
alexfh updated this object.
alexfh edited the test plan for this revision. (Show Details)
alexfh added reviewers: chandlerc, djasper.
alexfh added a subscriber: Unknown Object (MLST).
djasper edited edge metadata.Sep 7 2014, 11:23 PM

Do we really need a separate file for Clang? Shouldn't D5236 suffice as Clang is always checked out into LLVM?

On the other hand, I would add files containing "Checks: '-*'" into the test/ directories of both. Mosts test won't be (and don't need to be) clang-tidy clean.

alexfh accepted this revision.Sep 8 2014, 6:42 AM
alexfh added a reviewer: alexfh.

The question has been answered on the mailing list. I assume, this is also fine to submit, as it's the same file as in D5236.

This revision is now accepted and ready to land.Sep 8 2014, 6:42 AM
alexfh closed this revision.Sep 8 2014, 6:43 AM