[PATCH] D124066: [clang-tidy] Ignore macros defined within declarations in modernize-macro-to-enum

[Richard via Phabricator via cfe-commits]

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). [...]
> WDYT?

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.

https://github.com/llvm/llvm-project/issues/54883


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

https://reviews.llvm.org/D124066