[PATCH] D138115: [clang] Short-circuit evaluation in ::EvaluateAsConstantExpr

2022-11-17 Thread Timm Bäder via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGec782951d7bd: [clang] Short-circuit evaluation in ::EvaluateAsConstantExpr (authored by tbaeder). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138115/new/

[PATCH] D138115: [clang] Short-circuit evaluation in ::EvaluateAsConstantExpr

2022-11-16 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138115/new/ https://reviews.llvm.org/D138115 ___ cfe-commits mailing list cfe-commi

[PATCH] D138115: [clang] Short-circuit evaluation in ::EvaluateAsConstantExpr

2022-11-16 Thread Erich Keane via Phabricator via cfe-commits
erichkeane accepted this revision. erichkeane added a comment. This revision is now accepted and ready to land. This seems to make sense to me. I'll accept, but please give others time to take a look. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D

[PATCH] D138115: [clang] Short-circuit evaluation in ::EvaluateAsConstantExpr

2022-11-16 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder created this revision. tbaeder added reviewers: aaron.ballman, erichkeane. Herald added a project: All. tbaeder requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This adds another case to `FastEvaluateAsRValue()` for boolean expressio