Skip to content

Commit a03ffad

Browse files
jmorsetstellar
authored andcommitted
[DebugInfo] Re-enable instruction referencing for x86_64
After discussion in D116821 this was turned off in 74db5c8, 14aaaa1 applied to limit the maximum memory consumption in rare conditions, plus some performance patches. (cherry picked from commit 6e03a68)
1 parent 64adff8 commit a03ffad

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

llvm/lib/CodeGen/LiveDebugValues/LiveDebugValues.cpp

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,11 @@ bool LiveDebugValues::runOnMachineFunction(MachineFunction &MF) {
123123
}
124124

125125
bool llvm::debuginfoShouldUseDebugInstrRef(const Triple &T) {
126+
// Enable by default on x86_64, disable if explicitly turned off on cmdline.
127+
if (T.getArch() == llvm::Triple::x86_64 &&
128+
ValueTrackingVariableLocations != cl::boolOrDefault::BOU_FALSE)
129+
return true;
130+
126131
// Enable if explicitly requested on command line.
127132
return ValueTrackingVariableLocations == cl::boolOrDefault::BOU_TRUE;
128133
}

llvm/test/DebugInfo/X86/instr-ref-flag.ll

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,6 @@
1313
;; by llc by default, and that it can be turned explicitly on or off as
1414
;; desired.
1515

16-
;; Xfail due to faults found in the discussion on
17-
;; https://reviews.llvm.org/D116821
18-
; XFAIL: *
19-
2016
; INSTRREFON: DBG_INSTR_REF
2117
; INSTRREFOFF: DBG_VALUE
2218

0 commit comments

Comments
 (0)