[PATCH] D91908: [clang-tidy] Use compiled regex for AllowedRegexp in macro usage check

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Nov 22 05:18:39 PST 2020


njames93 accepted this revision.
njames93 added a comment.
This revision is now accepted and ready to land.

Because this is basically NFC and we already have tests for the AllowedRegexp option, there is no need to introduce new tests here.
So long as those current tests pass with this change, this should be perfectly fine.


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

https://reviews.llvm.org/D91908



More information about the cfe-commits mailing list