From: Dan Gohman <gohman@apple.com>
Date: Tue, 31 Jul 2007 17:22:27 +0000 (+0000)
Subject: Use SCEVExpander::InsertCastOfTo instead of calling new IntToPtrInst
X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=a9cfed77b6bc4930ffb07cbd2877cef5c44d93b2;p=oota-llvm.git

Use SCEVExpander::InsertCastOfTo instead of calling new IntToPtrInst
directly, because the insert point used by the SCEVExpander may vary
from what LSR originally computes.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@40641 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Transforms/Scalar/LoopStrengthReduce.cpp b/lib/Transforms/Scalar/LoopStrengthReduce.cpp
index 9689c120d69..64c60ba2e63 100644
--- a/lib/Transforms/Scalar/LoopStrengthReduce.cpp
+++ b/lib/Transforms/Scalar/LoopStrengthReduce.cpp
@@ -596,10 +596,13 @@ void BasedUser::RewriteInstructionToUseNewBase(const SCEVHandle &NewBase,
       }
     }
     Value *NewVal = InsertCodeForBaseAtPosition(NewBase, Rewriter, InsertPt, L);
-    // Adjust the type back to match the Inst.
+    // Adjust the type back to match the Inst. Note that we can't use InsertPt
+    // here because the SCEVExpander may have inserted the instructions after
+    // that point, in its efforts to avoid inserting redundant expressions.
     if (isa<PointerType>(OperandValToReplace->getType())) {
-      NewVal = new IntToPtrInst(NewVal, OperandValToReplace->getType(), "cast",
-                                InsertPt);
+      NewVal = SCEVExpander::InsertCastOfTo(Instruction::IntToPtr,
+                                            NewVal,
+                                            OperandValToReplace->getType());
     }
     // Replace the use of the operand Value with the new Phi we just created.
     Inst->replaceUsesOfWith(OperandValToReplace, NewVal);
@@ -648,9 +651,13 @@ void BasedUser::RewriteInstructionToUseNewBase(const SCEVHandle &NewBase,
         Instruction *InsertPt = PN->getIncomingBlock(i)->getTerminator();
         Code = InsertCodeForBaseAtPosition(NewBase, Rewriter, InsertPt, L);
 
-        // Adjust the type back to match the PHI.
+        // Adjust the type back to match the PHI. Note that we can't use InsertPt
+        // here because the SCEVExpander may have inserted its instructions after
+        // that point, in its efforts to avoid inserting redundant expressions.
         if (isa<PointerType>(PN->getType())) {
-          Code = new IntToPtrInst(Code, PN->getType(), "cast", InsertPt);
+          Code = SCEVExpander::InsertCastOfTo(Instruction::IntToPtr,
+                                              Code,
+                                              PN->getType());
         }
       }