This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] [host runtime] add .clang-tidy file
ClosedPublic

Authored by jlpeyton on Jun 4 2021, 12:40 PM.

Details

Summary

Use same checks as compiler-rt which removes checks for readability-* and llvm-header-guard style. The host runtime code does not use the same style for variables/functions/etc. as LLVM nor apparently header guards. The style checks are removed similar to the compiler-rt project.

On some of the patches, the clang-tidy annotations are so verbose they become quite distracting. This should help silent many of the irrelevant ones.

Diff Detail

Event Timeline

jlpeyton created this revision.Jun 4 2021, 12:40 PM
jlpeyton requested review of this revision.Jun 4 2021, 12:40 PM
protze.joachim accepted this revision.Jun 6 2021, 9:17 AM
protze.joachim added a subscriber: protze.joachim.

I think, this reflects what we discussed in the OpenMP in LLVM meeting. Thanks for working on this!

This revision is now accepted and ready to land.Jun 6 2021, 9:17 AM
This revision was automatically updated to reflect the committed changes.