6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2026-04-21 08:17:40 +00:00
Code Issues Projects Releases Wiki Activity

Don't skip debug instructions when looking for the insertion point of

Browse Source 
the cast. If we do, we can end up with

   inst1
   ---------------  < Insertion point
   dbg inst
   new inst

instead of the desired

   inst1
   new inst
   ---------------  < Insertion point
   dbg inst

Another option would be for InsertNoopCastOfTo (or its callers) to move the
insertion point and we would end up with

   inst1
   dbg inst
   new inst
   ---------------  < Insertion point

but that complicates the callers. This fixes PR12018 (and firefox's build).

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@150884 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Rafael Espindola
2012-02-18 17:22:58 +00:00
parent 31d157ae1a
commit ef4c80e07b
2 changed files with 46 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/Analysis/ScalarEvolutionExpander.cpp
+8 -5
View File
@@ -31,6 +31,12 @@ using namespace llvm;
Value *SCEVExpander::ReuseOrCreateCast(Value *V, Type *Ty,
Instruction::CastOps Op,
BasicBlock::iterator IP) {
// All new or reused instructions must strictly dominate the Builder's
// InsertPt to ensure that the expression's expansion dominates its uses.
// Assert that the requested insertion point works at least for new
// instructions.
assert(SE.DT->dominates(IP, Builder.GetInsertPoint()));
// Check to see if there is already a cast!
for (Value::use_iterator UI = V->use_begin(), E = V->use_end();
UI != E; ++UI) {
@@ -38,9 +44,7 @@ Value *SCEVExpander::ReuseOrCreateCast(Value *V, Type *Ty,
if (U->getType() == Ty)
if (CastInst *CI = dyn_cast<CastInst>(U))
if (CI->getOpcode() == Op) {
// If the cast isn't where we want it, fix it. All new or reused
// instructions must strictly dominate the Builder's InsertPt to
// ensure that the expression's expansion dominates its uses.
// If the cast isn't where we want it, fix it.
if (BasicBlock::iterator(CI) != IP
|| IP == Builder.GetInsertPoint()) {
// Create a new cast, and leave the old cast in place in case
@@ -124,8 +128,7 @@ Value *SCEVExpander::InsertNoopCastOfTo(Value *V, Type *Ty) {
BasicBlock::iterator IP = I; ++IP;
if (InvokeInst *II = dyn_cast<InvokeInst>(I))
IP = II->getNormalDest()->begin();
while (isa<PHINode>(IP) || isa<DbgInfoIntrinsic>(IP) ||
isa<LandingPadInst>(IP))
while (isa<PHINode>(IP) || isa<LandingPadInst>(IP))
++IP;
return ReuseOrCreateCast(I, Ty, Op, IP);
}