From 28ff072b70e8091d1d311032ddca49406ea9bb57 Mon Sep 17 00:00:00 2001 From: Daniel Dunbar Date: Sat, 23 Aug 2008 18:44:10 +0000 Subject: Drop a dead call to isConstantExpr() llvm-svn: 55244 --- clang/lib/CodeGen/CGExprAgg.cpp | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) (limited to 'clang/lib/CodeGen/CGExprAgg.cpp') diff --git a/clang/lib/CodeGen/CGExprAgg.cpp b/clang/lib/CodeGen/CGExprAgg.cpp index fbba9c6c0c40..c3db624478ec 100644 --- a/clang/lib/CodeGen/CGExprAgg.cpp +++ b/clang/lib/CodeGen/CGExprAgg.cpp @@ -371,14 +371,12 @@ void AggExprEmitter::EmitNullInitializationToLValue(LValue LV, QualType T) { } void AggExprEmitter::VisitInitListExpr(InitListExpr *E) { - if (E->isConstantExpr(CGF.getContext(), 0)) { - // FIXME: call into const expr emitter so that we can emit - // a memcpy instead of storing the individual members. - // This is purely for perf; both codepaths lead to equivalent - // (although not necessarily identical) code. - // It's worth noting that LLVM keeps on getting smarter, though, - // so it might not be worth bothering. - } + // FIXME: For constant expressions, call into const expr emitter so + // that we can emit a memcpy instead of storing the individual + // members. This is purely for perf; both codepaths lead to + // equivalent (although not necessarily identical) code. It's worth + // noting that LLVM keeps on getting smarter, though, so it might + // not be worth bothering. // Handle initialization of an array. if (E->getType()->isArrayType()) { -- cgit v1.2.3