JonasToth marked an inline comment as done. JonasToth added inline comments.
================ Comment at: clang/lib/Analysis/ExprMutationAnalyzer.cpp:61 + // below. + auto const ConditionalOperator = conditionalOperator(anyOf( + hasTrueExpression(ignoringParens(canResolveToExpr(InnerMatcher))), ---------------- aaron.ballman wrote: > Do you also want to handle the binary conditional operator (a GNU extension > like `a ? : b`)? Yup, was a fast improvement. Thanks for pointing that out :+1: Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88088/new/ https://reviews.llvm.org/D88088 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
