jdoerfert marked an inline comment as done. jdoerfert added inline comments.
================ Comment at: clang/lib/Sema/TreeTransform.h:2414 Expr *ExecConfig = nullptr) { - return getSema().BuildCallExpr(/*Scope=*/nullptr, Callee, LParenLoc, Args, - RParenLoc, ExecConfig); + ExprResult Result = getSema().BuildCallExpr( + /*Scope=*/nullptr, Callee, LParenLoc, Args, RParenLoc, ExecConfig); ---------------- erichkeane wrote: > It seems to me that this logic would better fit in 'ActOnCallExpr' if at all > possible, and RebuildCallExpr should just call ActOnCallExpr. I see there is > a warning implemented there (a strange place for that), but if this doesn't > cause conflicts I would expect that to be the behavior here. > > The logic is in `ActOnCallExpr` so if I can call that here it should work just fine. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77290/new/ https://reviews.llvm.org/D77290 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits