njames93 added a comment. In D122078#3407984 <https://reviews.llvm.org/D122078#3407984>, @aaron.ballman wrote:
> Okie dokie, thanks for weighing in! This LGTM (feel free to add the release > note when landing). If @alexfh has some technical concerns with > `hasAncestor()`, we can always revert and address them once we know more. I feel like this is another issue which could be solved with traversal scopes. `isInEvaluationContext` This traversal scope wouldn't traverse things like concepts, static_asserts(?), Noexcept & explicit specifiers, sizeof, decltype etc. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122078/new/ https://reviews.llvm.org/D122078 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits