diff options
| author | Daniel Dunbar <daniel@zuster.org> | 2008-08-23 18:44:10 +0000 |
|---|---|---|
| committer | Daniel Dunbar <daniel@zuster.org> | 2008-08-23 18:44:10 +0000 |
| commit | 28ff072b70e8091d1d311032ddca49406ea9bb57 (patch) | |
| tree | 399578220d3d5c1dfadeede39cb63645fe3a214e | |
| parent | d644ad61e98f5ff267ae2d551e7428b13fdd4225 (diff) | |
| download | bcm5719-llvm-28ff072b70e8091d1d311032ddca49406ea9bb57.tar.gz bcm5719-llvm-28ff072b70e8091d1d311032ddca49406ea9bb57.zip | |
Drop a dead call to isConstantExpr()
llvm-svn: 55244
| -rw-r--r-- | clang/lib/CodeGen/CGExprAgg.cpp | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/clang/lib/CodeGen/CGExprAgg.cpp b/clang/lib/CodeGen/CGExprAgg.cpp index fbba9c6c0c4..c3db624478e 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()) { |

