Index: llvm/trunk/lib/Transforms/IPO/GlobalOpt.cpp =================================================================== --- llvm/trunk/lib/Transforms/IPO/GlobalOpt.cpp +++ llvm/trunk/lib/Transforms/IPO/GlobalOpt.cpp @@ -1710,19 +1710,25 @@ assert(isa(StoreVal) && "Not a load of NewGV!"); } } - new StoreInst(StoreVal, NewGV, false, 0, - SI->getOrdering(), SI->getSyncScopeID(), SI); + StoreInst *NSI = + new StoreInst(StoreVal, NewGV, false, 0, SI->getOrdering(), + SI->getSyncScopeID(), SI); + NSI->setDebugLoc(SI->getDebugLoc()); } else { // Change the load into a load of bool then a select. LoadInst *LI = cast(UI); LoadInst *NLI = new LoadInst(NewGV, LI->getName()+".b", false, 0, LI->getOrdering(), LI->getSyncScopeID(), LI); - Value *NSI; + Instruction *NSI; if (IsOneZero) NSI = new ZExtInst(NLI, LI->getType(), "", LI); else NSI = SelectInst::Create(NLI, OtherVal, InitVal, "", LI); NSI->takeName(LI); + // Since LI is split into two instructions, NLI and NSI both inherit the + // same DebugLoc + NLI->setDebugLoc(LI->getDebugLoc()); + NSI->setDebugLoc(LI->getDebugLoc()); LI->replaceAllUsesWith(NSI); } UI->eraseFromParent(); Index: llvm/trunk/test/Transforms/GlobalOpt/shrink-global-to-bool-check-debug.ll =================================================================== --- llvm/trunk/test/Transforms/GlobalOpt/shrink-global-to-bool-check-debug.ll +++ llvm/trunk/test/Transforms/GlobalOpt/shrink-global-to-bool-check-debug.ll @@ -0,0 +1,22 @@ +;RUN: opt -S -debugify -globalopt -f %s | FileCheck %s + +@foo = internal global i32 0, align 4 + +define dso_local i32 @bar() { +entry: + store i32 5, i32* @foo, align 4 + %0 = load i32, i32* @foo, align 4 + ret i32 %0 +} + +;CHECK: @bar +;CHECK-NEXT: entry: +;CHECK-NEXT: store i1 true, i1* @foo, !dbg ![[DbgLocStore:[0-9]+]] +;CHECK-NEXT: %.b = load i1, i1* @foo, !dbg ![[DbgLocLoadSel:[0-9]+]] +;CHECK-NEXT: %0 = select i1 %.b, i32 5, i32 0, !dbg ![[DbgLocLoadSel]] +;CHECK-NEXT: call void @llvm.dbg.value({{.*}}), !dbg ![[DbgLocLoadSel]] +;CHECK-NEXT: ret i32 %0, !dbg ![[DbgLocRet:[0-9]+]] + +;CHECK: ![[DbgLocStore]] = !DILocation(line: 1, +;CHECK: ![[DbgLocLoadSel]] = !DILocation(line: 2, +;CHECK: ![[DbgLocRet]] = !DILocation(line: 3,