During bottom up fast-isel, instructions emitted to materalize registers are at top of basic block and do not have debug location. This may misguide debugger while entering the basic block and sometimes debugger provides semi useful view of current location to developer by picking up previous known location as current location. Assign a sensible location to the first instruction in a basic block, if it does not have one location derived from source file, so that debugger can provide meaningful user experience to developers in edge cases.

[take 2]


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@135423 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Devang Patel
2011-07-18 20:55:23 +00:00
parent fc933c073e
commit 1360bc8eb0
4 changed files with 148 additions and 0 deletions

View File

@@ -292,6 +292,28 @@ void FastISel::recomputeInsertPt() {
++FuncInfo.InsertPt;
}
/// recomputeDebugLocForMaterializedRegs - Recompute debug location for
/// very first instruction in a basic block. All instructions emitted
/// to materialize registers do not have location information, see
/// enterLocalValueArea(), becase they may not be emited at the right
/// location.
void FastISel::recomputeDebugLocForMaterializedRegs() {
if (!getLastLocalValue())
return;
MachineInstr *First = FuncInfo.MBB->getFirstNonPHI();
if (!First->getDebugLoc().isUnknown())
return;
for (MachineBasicBlock::iterator I = FuncInfo.MBB->begin(),
E = FuncInfo.MBB->end(); I != E; ++I) {
DebugLoc DL = I->getDebugLoc();
if (!DL.isUnknown()) {
First->setDebugLoc(DL);
return;
}
}
}
FastISel::SavePoint FastISel::enterLocalValueArea() {
MachineBasicBlock::iterator OldInsertPt = FuncInfo.InsertPt;
DebugLoc OldDL = DL;