[clang] [Clang][ExprConstant] fix constant expression did not evaluate to integer (PR #97146)

Eli Friedman via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 1 11:17:10 PDT 2024


efriedma-quic wrote:

To the extent that EvaluateAsRValue handles LValues, FastEvaluateAsRValue should also handle them, sure.  I'm suspicious of the fact that EvaluateAsRValue is doing this conversion in the first place, but I guess this pull request isn't the place to address that.

Given that, this patch LGTM

https://github.com/llvm/llvm-project/pull/97146


More information about the cfe-commits mailing list