[clang] [Analyzer][CFG] Correctly handle rebuilt default arg and default init expression (PR #117437)

[Erich Keane via cfe-commits]

================
@@ -5570,8 +5570,10 @@ ExprResult Sema::BuildCXXDefaultArgExpr(SourceLocation 
CallLoc,
           /*SkipImmediateInvocations=*/NestedDefaultChecking))
     return ExprError();
 
+  Expr *RewrittenExpr = Init == Param->getDefaultArg() ? nullptr : Init;
----------------
erichkeane wrote:

```suggestion
  Expr *RewrittenExpr = (Init == Param->getDefaultArg() ? nullptr : Init);
```

MINOR preference to reverse this condition?  But not sure enough to suggest 
this.

https://github.com/llvm/llvm-project/pull/117437
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits