nickdesaulniers added inline comments.
================ Comment at: clang/lib/Sema/SemaChecking.cpp:10133-10135 + if (ImplicitMatch == ArgType::NoMatchPedantic || + ImplicitMatch == ArgType::NoMatchTypeConfusion) + Match = ImplicitMatch; ---------------- Similarly, if `ImplicitMatch` can only be updated to one of these two values within the above `const ImplicitCastExpr *ICE = dyn_cast<ImplicitCastExpr>(E)` branch, does it make sense to additionally hoist this conditional check into that branch as well? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D132568/new/ https://reviews.llvm.org/D132568 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
