diff options
| author | Benjamin Kramer <benny.kra@googlemail.com> | 2011-02-24 23:26:09 +0000 |
|---|---|---|
| committer | Benjamin Kramer <benny.kra@googlemail.com> | 2011-02-24 23:26:09 +0000 |
| commit | dfdca1a14db80a508bfec8062d670a9ed9853cb1 (patch) | |
| tree | d6469f7130ae5aad03564e3966c86aadd8a49481 /llvm/lib/Transforms | |
| parent | e68a27eecd50c804f0005947c1b4300ee0cb3949 (diff) | |
| download | bcm5719-llvm-dfdca1a14db80a508bfec8062d670a9ed9853cb1.tar.gz bcm5719-llvm-dfdca1a14db80a508bfec8062d670a9ed9853cb1.zip | |
SimplifyCFG: GEPs with just one non-constant index are also cheap.
llvm-svn: 126452
Diffstat (limited to 'llvm/lib/Transforms')
| -rw-r--r-- | llvm/lib/Transforms/Utils/SimplifyCFG.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp index c6708857cb5..3968d6e8d3b 100644 --- a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp +++ b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp @@ -247,11 +247,13 @@ static bool DominatesMergePoint(Value *V, BasicBlock *BB, if (PBB->getFirstNonPHIOrDbg() != I) return false; break; - case Instruction::GetElementPtr: - // GEPs are cheap if all indices are constant. - if (!cast<GetElementPtrInst>(I)->hasAllConstantIndices()) + case Instruction::GetElementPtr: { + // GEPs are cheap if all indices are constant or if there's only one index. + GetElementPtrInst *GEP = cast<GetElementPtrInst>(I); + if (!GEP->hasAllConstantIndices() && GEP->getNumIndices() > 1) return false; break; + } case Instruction::Add: case Instruction::Sub: case Instruction::And: |
