[PATCH] D57113: [clang-tidy] openmp-use-default-none - a new module and a check

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jan 27 05:25:38 PST 2019


lebedev.ri added subscribers: sbenza, klimek.
lebedev.ri added a comment.

In D57113#1372891 <https://reviews.llvm.org/D57113#1372891>, @JonasToth wrote:

> LGTM with the few language nits.
>  The new matchers look better and are more matcher style, gj :)


Thank you for the review!
I'm not quite sure who is qualified to review the base
`D57112 ASTTypeTraits` <https://reviews.llvm.org/D57112> patch though, @klimek or @sbenza it looks like?


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D57113





More information about the cfe-commits mailing list