================ @@ -40,8 +40,12 @@ static const Expr *ignoreTransparentExprs(const Expr *E) { switch (E->getStmtClass()) { case Stmt::OpaqueValueExprClass: - E = cast<OpaqueValueExpr>(E)->getSourceExpr(); - break; + if (const clang::Expr *SE = cast<OpaqueValueExpr>(E)->getSourceExpr()) { + E = SE; + break; + } else { + return E; + } ---------------- steakhal wrote:
```suggestion if (const auto *SE = cast<OpaqueValueExpr>(E)->getSourceExpr()) { E = SE; break; } return E; ``` Fix `else-after-break`. Use `auto` after `cast<>`. https://github.com/llvm/llvm-project/pull/79764 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits