[InstCombine] Some cleanup in optimization of redundant insertvalue instructions.

And one more test added.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@208355 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Michael Zolotukhin
2014-05-08 19:50:24 +00:00
parent 82cb28e99a
commit 6cc3e0e4e7
2 changed files with 14 additions and 4 deletions
@@ -506,13 +506,12 @@ Instruction *InstCombiner::visitInsertValueInst(InsertValueInst &I) {
// chain), check if any of the 'children' uses the same indices as the first
// instruction. In this case, the first one is redundant.
Value *V = &I;
unsigned int Depth = 0;
unsigned Depth = 0;
while (V->hasOneUse() && Depth < 10) {
User *U = V->user_back();
InsertValueInst *UserInsInst = dyn_cast<InsertValueInst>(U);
if (!UserInsInst || U->getType() != I.getType()) {
auto UserInsInst = dyn_cast<InsertValueInst>(U);
if (!UserInsInst || U->getOperand(0) != V)
break;
}
if (UserInsInst->getIndices() == FirstIndices) {
IsRedundant = true;
break;