RPCSX/llvm
1
0
Fork 0
mirror of https://github.com/RPCSX/llvm.git synced 2025-02-28 00:50:37 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

simplify/detangle some control flow.

Browse Source 
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@80476 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Chris Lattner 2009-08-30 05:00:50 +00:00
parent ccf4b34a08
commit 6e24d83167
1 changed files with 15 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
lib/Transforms/Scalar/InstructionCombining.cpp
View File

@ -11044,21 +11044,18 @@ Instruction *InstCombiner::visitGetElementPtrInst(GetElementPtrInst &GEP) {
// is a getelementptr instruction, combine the indices of the two // is a getelementptr instruction, combine the indices of the two
// getelementptr instructions into a single instruction. // getelementptr instructions into a single instruction.
// //
SmallVector<Value*, 8> SrcGEPOperands;
bool BothInBounds = cast<GEPOperator>(&GEP)->isInBounds();
if (GEPOperator *Src = dyn_cast<GEPOperator>(PtrOp)) { if (GEPOperator *Src = dyn_cast<GEPOperator>(PtrOp)) {
SmallVector<Value*, 8> SrcGEPOperands;
SrcGEPOperands.append(Src->op_begin(), Src->op_end()); SrcGEPOperands.append(Src->op_begin(), Src->op_end());
if (!Src->isInBounds())
BothInBounds = false;
}
if (!SrcGEPOperands.empty()) {
// Note that if our source is a gep chain itself that we wait for that // Note that if our source is a gep chain itself that we wait for that
// chain to be resolved before we perform this transformation. This // chain to be resolved before we perform this transformation. This
// avoids us creating a TON of code in some cases. // avoids us creating a TON of code in some cases.
// //
if (isa<GetElementPtrInst>(SrcGEPOperands[0]) && if (SrcGEPOperands.empty() ||
cast<Instruction>(SrcGEPOperands[0])->getNumOperands() == 2) (isa<GetElementPtrInst>(SrcGEPOperands[0]) &&
cast<Instruction>(SrcGEPOperands[0])->getNumOperands() == 2))
return 0; // Wait until our source is folded to completion. return 0; // Wait until our source is folded to completion.
SmallVector<Value*, 8> Indices; SmallVector<Value*, 8> Indices;
@ -11084,17 +11081,14 @@ Instruction *InstCombiner::visitGetElementPtrInst(GetElementPtrInst &GEP) {
// target's pointer size. // target's pointer size.
if (SO1->getType() != GO1->getType()) { if (SO1->getType() != GO1->getType()) {
if (Constant *SO1C = dyn_cast<Constant>(SO1)) { if (Constant *SO1C = dyn_cast<Constant>(SO1)) {
SO1 = SO1 = ConstantExpr::getIntegerCast(SO1C, GO1->getType(), true);
ConstantExpr::getIntegerCast(SO1C, GO1->getType(), true);
} else if (Constant *GO1C = dyn_cast<Constant>(GO1)) { } else if (Constant *GO1C = dyn_cast<Constant>(GO1)) {
GO1 = GO1 = ConstantExpr::getIntegerCast(GO1C, SO1->getType(), true);
ConstantExpr::getIntegerCast(GO1C, SO1->getType(), true);
} else if (TD) { } else if (TD) {
unsigned PS = TD->getPointerSizeInBits(); unsigned PS = TD->getPointerSizeInBits();
if (TD->getTypeSizeInBits(SO1->getType()) == PS) { if (TD->getTypeSizeInBits(SO1->getType()) == PS) {
// Convert GO1 to SO1's type. // Convert GO1 to SO1's type.
GO1 = InsertCastToIntPtrTy(GO1, SO1->getType(), &GEP, this); GO1 = InsertCastToIntPtrTy(GO1, SO1->getType(), &GEP, this);
} else if (TD->getTypeSizeInBits(GO1->getType()) == PS) { } else if (TD->getTypeSizeInBits(GO1->getType()) == PS) {
// Convert SO1 to GO1's type. // Convert SO1 to GO1's type.
SO1 = InsertCastToIntPtrTy(SO1, GO1->getType(), &GEP, this); SO1 = InsertCastToIntPtrTy(SO1, GO1->getType(), &GEP, this);
@ -11136,15 +11130,16 @@ Instruction *InstCombiner::visitGetElementPtrInst(GetElementPtrInst &GEP) {
GetElementPtrInst *NewGEP = GetElementPtrInst *NewGEP =
GetElementPtrInst::Create(SrcGEPOperands[0], Indices.begin(), GetElementPtrInst::Create(SrcGEPOperands[0], Indices.begin(),
Indices.end(), GEP.getName()); Indices.end(), GEP.getName());
if (BothInBounds) if (cast<GEPOperator>(&GEP)->isInBounds() && Src->isInBounds())
cast<GEPOperator>(NewGEP)->setIsInBounds(true); cast<GEPOperator>(NewGEP)->setIsInBounds(true);
return NewGEP; return NewGEP;
} }
}
} else if (Value *X = getBitCastOperand(PtrOp)) { // Is the operand a cast?
if (!isa<PointerType>(X->getType())) { if (Value *X = getBitCastOperand(PtrOp)) { // Is the operand a cast?
// Not interesting. Source pointer must be a cast from pointer. assert(isa<PointerType>(X->getType()) && "Must be cast from pointer");
} else if (HasZeroPointerIndex) {
if (HasZeroPointerIndex) {
// transform: GEP (bitcast [10 x i8]* X to [0 x i8]*), i32 0, ... // transform: GEP (bitcast [10 x i8]* X to [0 x i8]*), i32 0, ...
// into : GEP [10 x i8]* X, i32 0, ... // into : GEP [10 x i8]* X, i32 0, ...
// //