[PATCH] D153294: [clang] Do not create ExprWithCleanups while checking immediate invocation
Ilya Biryukov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jun 23 05:41:59 PDT 2023
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
More information about the cfe-commits
mailing list