[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