forked from OSchip/llvm-project
DBG_VALUE machine instruction pointing to undefined register for a variable justify a separate scope if the variable is inlined function's argument.
Radar 8122864. llvm-svn: 106792
This commit is contained in:
parent
24e98486a3
commit
c657c621b7
|
@ -2390,8 +2390,6 @@ DbgScope *DwarfDebug::getOrCreateDbgScope(const MDNode *Scope, const MDNode *Inl
|
|||
static bool hasValidLocation(LLVMContext &Ctx,
|
||||
const MachineInstr *MInsn,
|
||||
const MDNode *&Scope, const MDNode *&InlinedAt) {
|
||||
if (MInsn->isDebugValue())
|
||||
return false;
|
||||
DebugLoc DL = MInsn->getDebugLoc();
|
||||
if (DL.isUnknown()) return false;
|
||||
|
||||
|
@ -2655,7 +2653,6 @@ void DwarfDebug::beginFunction(const MachineFunction *MF) {
|
|||
assert (MI->getNumOperands() > 1 && "Invalid machine instruction!");
|
||||
DIVariable DV(MI->getOperand(MI->getNumOperands() - 1).getMetadata());
|
||||
if (!DV.Verify()) continue;
|
||||
if (isDbgValueInUndefinedReg(MI)) continue;
|
||||
// If DBG_VALUE is for a local variable then it needs a label.
|
||||
if (DV.getTag() != dwarf::DW_TAG_arg_variable)
|
||||
InsnNeedsLabel.insert(MI);
|
||||
|
@ -2663,7 +2660,7 @@ void DwarfDebug::beginFunction(const MachineFunction *MF) {
|
|||
else if (!DISubprogram(DV.getContext()).describes(MF->getFunction()))
|
||||
InsnNeedsLabel.insert(MI);
|
||||
// DBG_VALUE indicating argument location change needs a label.
|
||||
else if (!ProcessedArgs.insert(DV))
|
||||
else if (isDbgValueInUndefinedReg(MI) == false && !ProcessedArgs.insert(DV))
|
||||
InsnNeedsLabel.insert(MI);
|
||||
} else {
|
||||
// If location is unknown then instruction needs a location only if
|
||||
|
|
|
@ -0,0 +1,15 @@
|
|||
// RUN: %llvmgcc -S -O2 -g %s -o - | llc -O2 -o %t.s
|
||||
// RUN: grep "# DW_TAG_formal_parameter" %t.s | count 4
|
||||
// Radar 8122864
|
||||
static int foo(int a, int j) {
|
||||
int k = 0;
|
||||
if (a)
|
||||
k = a + j;
|
||||
else
|
||||
k = j;
|
||||
return k;
|
||||
}
|
||||
int bar(int o, int p) {
|
||||
|
||||
return foo(o, p);
|
||||
}
|
Loading…
Reference in New Issue