[PATCH] D124066: [clang-tidy] Ignore macros defined within declarations in modernize-macro-to-enum
Richard via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 20 14:07:50 PDT 2022
LegalizeAdulthood added a comment.
In D124066#3463008 <https://reviews.llvm.org/D124066#3463008>, @aaron.ballman wrote:
> This seems like a case where we might want a configuration option (maybe). [...]
In my bug report on this problem, I sketch out a plan of attack:
1. **DON'T BREAK MY CODE** -- that is this review `:)`
2. Do some analysis of macro expansion locations to determine the nearest enclosing scope at which the enum should be declared.
CHANGES SINCE LAST ACTION
More information about the cfe-commits