[PATCH] D57571: [clang-tidy] A new OpenMP module

2019-03-22 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL356800: [clang-tidy] A new OpenMP module (authored by lebedevri, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/

[PATCH] D57571: [clang-tidy] A new OpenMP module

2019-03-21 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri updated this revision to Diff 191724. lebedev.ri added a comment. Rebased, NFC. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57571/new/ https://reviews.llvm.org/D57571 Files: clang-tidy/CMakeLists.txt clang-tidy/ClangTidyForceLinker

[PATCH] D57571: [clang-tidy] A new OpenMP module

2019-03-18 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. While I'm okay with this, I would mildly prefer it if landed with at least one check rather than entirely empty (IIRC, that's how we'd added new modules in the past). Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D5757

[PATCH] D57571: [clang-tidy] A new OpenMP module

2019-03-16 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri updated this revision to Diff 190972. lebedev.ri added a comment. Herald added a subscriber: jdoerfert. Rebased, no changes. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57571/new/ https://reviews.llvm.org/D57571 Files: clang-tidy/CMa

[PATCH] D57571: [clang-tidy] A new OpenMP module

2019-02-01 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri added a comment. In D57571#1380421 , @JonasToth wrote: > LGTM. In D57571#1380421 , @JonasToth wrote: > Committing both revision together would be good :) Yes. I can't land them since D57112

[PATCH] D57571: [clang-tidy] A new OpenMP module

2019-02-01 Thread Jonas Toth via Phabricator via cfe-commits
JonasToth accepted this revision. JonasToth added a comment. This revision is now accepted and ready to land. LGTM. Committing both revision together would be good :) Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57571/new/ https://reviews.llvm.org/

[PATCH] D57571: [clang-tidy] A new OpenMP module

2019-02-01 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri created this revision. lebedev.ri added reviewers: JonasToth, aaron.ballman, alexfh, xazax.hun, hokein. lebedev.ri added projects: clang-tools-extra, OpenMP. Herald added subscribers: openmp-commits, arphaman, guansong, rnkovacs, mgorny. Herald added a project: clang. Just the empty ske