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:14 PM.

Diff Detail

Event Timeline

alexfh updated this revision to Diff 13378.Sep 7 2014, 4:14 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 accepted this revision.Sep 8 2014, 6:10 AM
djasper edited edge metadata.

I think this is a no-brainer.

This revision is now accepted and ready to land.Sep 8 2014, 6:10 AM

The only question we might think about in the long(er) run is whether we really want ".clang-tidy" here but keep clang-tidy in clang-tools-extra.

alexfh added a comment.Sep 8 2014, 6:38 AM
In D5236#7, @djasper wrote:

The only question we might think about in the long(er) run is whether we really want ".clang-tidy" here but keep clang-tidy in clang-tools-extra.

Yes, we should think about moving clang-tidy to the main tree. But I don't think the location of clang-tidy sources matters in this context. The file in question is a tool configuration file. We also have .arcconfig and .gitignore here, but nobody asks whether it's fine that the relevant tools are developed separately ;)

alexfh closed this revision.Sep 8 2014, 6:39 AM