Skip to content

Commit

Permalink
[DebugInfo] Re-enable instruction referencing for x86_64
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
jmorse committed Feb 2, 2022
1 parent 30420bc commit 6e03a68
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
5 changes: 5 additions & 0 deletions llvm/lib/CodeGen/LiveDebugValues/LiveDebugValues.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -123,6 +123,11 @@ bool LiveDebugValues::runOnMachineFunction(MachineFunction &MF) {
}

bool llvm::debuginfoShouldUseDebugInstrRef(const Triple &T) {
// Enable by default on x86_64, disable if explicitly turned off on cmdline.
if (T.getArch() == llvm::Triple::x86_64 &&
ValueTrackingVariableLocations != cl::boolOrDefault::BOU_FALSE)
return true;

// Enable if explicitly requested on command line.
return ValueTrackingVariableLocations == cl::boolOrDefault::BOU_TRUE;
}
4 changes: 0 additions & 4 deletions llvm/test/DebugInfo/X86/instr-ref-flag.ll
Original file line number Diff line number Diff line change
Expand Up @@ -13,10 +13,6 @@
;; by llc by default, and that it can be turned explicitly on or off as
;; desired.

;; Xfail due to faults found in the discussion on
;; https://reviews.llvm.org/D116821
; XFAIL: *

; INSTRREFON: DBG_INSTR_REF
; INSTRREFOFF: DBG_VALUE

Expand Down

0 comments on commit 6e03a68

Please sign in to comment.