Temporary fix for a performance problem Eli spotted. The APValue representation
is currently too inefficient to allow us to use it for array initializers, but
fortunately we usually don't yet need to evaluate such initializers.
llvm-svn: 144260
diff --git a/clang/lib/AST/ExprConstant.cpp b/clang/lib/AST/ExprConstant.cpp
index aa94990..f2e3d36 100644
--- a/clang/lib/AST/ExprConstant.cpp
+++ b/clang/lib/AST/ExprConstant.cpp
@@ -3391,6 +3391,12 @@
/// in Result. If this expression is a glvalue, an lvalue-to-rvalue conversion
/// will be applied to the result.
bool Expr::EvaluateAsRValue(EvalResult &Result, const ASTContext &Ctx) const {
+ // FIXME: Evaluating initializers for large arrays can cause performance
+ // problems, and we don't use such values yet. Once we have a more efficient
+ // array representation, this should be reinstated, and used by CodeGen.
+ if (isRValue() && getType()->isArrayType())
+ return false;
+
EvalInfo Info(Ctx, Result);
CCValue Value;