ilya-biryukov added a comment.

While we wait for Richard's response... @Fznamznon what are your thoughts on 
wrapping all `ConstantExpr` that span immediate invocations with redundant 
`ExprWithCleanups` per Richard's suggestions?
I would be comfortable LGTMing such a change even if we choose to remove some 
of this redundancy later in a follow-ups.

This bug looks scary enough to me to warrant a fix asap.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D153294/new/

https://reviews.llvm.org/D153294

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to