[PATCH] D54404: Exclude matchers which can have multiple results
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Nov 11 15:40:30 PST 2018
aaron.ballman added inline comments.
================
Comment at: lib/ASTMatchers/Dynamic/Registry.cpp:604
+ static std::vector<StringRef> excludedMatchers{
+ "allOf",
----------------
`excludedMatchers` -> `ExcludedMatchers`
================
Comment at: lib/ASTMatchers/Dynamic/Registry.cpp:604
+ static std::vector<StringRef> excludedMatchers{
+ "allOf",
----------------
aaron.ballman wrote:
> `excludedMatchers` -> `ExcludedMatchers`
Please add comments that explain why these are excluded and under what circumstances this list should be updated to add or remove items.
Also: how do we ensure this list stays properly updated? We sometimes miss updating `RegistryMaps()`, so I'm a bit concerned that adding a second list in a different location will be inviting trouble.
Repository:
rC Clang
https://reviews.llvm.org/D54404
More information about the cfe-commits
mailing list