Skip to content

Commit 6e03a68

Browse files
committed
[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.
1 parent 30420bc commit 6e03a68

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)