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

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 1 03:48:13 PST 2019


lebedev.ri added a comment.

In D57571#1380421 <https://reviews.llvm.org/D57571#1380421>, @JonasToth wrote:

> LGTM.




In D57571#1380421 <https://reviews.llvm.org/D57571#1380421>, @JonasToth wrote:

> Committing both revision together would be good :)


Yes.
I can't land them since D57112 <https://reviews.llvm.org/D57112> has no review feedback as of yet.
Hopefully not for too long. Though i'm not sure when i can reasonably ping it,
updating the review probably counts as a ping, i think.

I have split it up so i can work on the second check, `openmp-exception-escape`.


Repository:
  rCTE Clang Tools Extra

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57571/new/

https://reviews.llvm.org/D57571





More information about the cfe-commits mailing list