[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