[DebugInfo] Turn instruction referencing on by default for x86
authorJeremy Morse <jeremy.morse@sony.com>
Tue, 30 Nov 2021 12:41:59 +0000 (12:41 +0000)
committerJeremy Morse <jeremy.morse@sony.com>
Tue, 30 Nov 2021 13:44:07 +0000 (13:44 +0000)
This patch is designed to be reverted -- it activates a reasonably large
block of new-ish code, so some turbulence is likely.

Instruction referencing is best summarised, and it being on-by-default,
is discussed here:

    https://lists.llvm.org/pipermail/llvm-dev/2021-November/153653.html

Differential Revision: https://reviews.llvm.org/D114631

llvm/lib/CodeGen/CommandFlags.cpp
llvm/test/DebugInfo/X86/instr-ref-flag.ll

index ecf3d931fc33b1f8422fa62657970befe868af5f..3bed81d5841d13db6fc2587c4e5a767e07fd06f9 100644 (file)
@@ -699,5 +699,7 @@ void codegen::setFunctionAttributes(StringRef CPU, StringRef Features,
 }
 
 bool codegen::getDefaultValueTrackingVariableLocations(const llvm::Triple &T) {
+  if (T.getArch() == llvm::Triple::x86_64)
+    return true;
   return false;
 }
index b496bc418dec9cb4c0bd7020142ca4b79d662af7..f9d5f99edf77ff4e67c953d76d61567cf0b14f3b 100644 (file)
 ;; by llc by default, and that it can be turned explicitly on or off as
 ;; desired.
 
-;; XFail it for pre-landing the patch, and to allow a minimal delta if it
-;; has to be reverted.
-; XFAIL: *
-
 ; INSTRREFON: DBG_INSTR_REF
 ; INSTRREFOFF: DBG_VALUE