[PATCH] D122078: [clang-tidy] Ignore concepts in `misc-redundant-expression`

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 25 08:19:35 PDT 2022


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



More information about the cfe-commits mailing list