[PATCH] D50580: [clang-tidy] Abseil: no namespace check
Jonas Toth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 16 07:16:17 PDT 2018
JonasToth added a comment.
That sounds good as well, just not in clang and best in `clang-tidy/utils`
> `ASTMatchers.h` is not a reasonable place to put `asseil`-specific matchers. We have `clang-tidy/utils/Matchers.h` for putting clang-tidy specific matchers. I'm not sure whether it is reasonable to put abseil-specific matchers there. Maybe we create a `AbseilMatcher.h` file in `clang-tidy/abseil` directory?
>
> https://reviews.llvm.org/D50580
https://reviews.llvm.org/D50580
More information about the cfe-commits
mailing list