================ @@ -34,13 +34,13 @@ static void BuildParentMap(MapTy& M, Stmt* S, case Stmt::PseudoObjectExprClass: { PseudoObjectExpr *POE = cast<PseudoObjectExpr>(S); - if (OVMode == OV_Opaque && M[POE->getSyntacticForm()]) + if (OVMode == OV_Opaque && M.contains(POE->getSyntacticForm())) ---------------- jayfoad wrote:
Could put this and the next `if` inside one big `if (M.contains)` check to avoid a duplicated map lookup. https://github.com/llvm/llvm-project/pull/121736 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits