[PATCH] D83462: [DWARF] Avoid entry_values production for SCE

[Adrian Prantl via Phabricator via llvm-commits]

aprantl added inline comments.


================
Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp:429
   // the debug entry values feature. It can also be enabled explicitly.
-  EmitDebugEntryValues = (Asm->TM.Options.ShouldEmitDebugEntryValues() &&
-                          (tuneForGDB() || tuneForLLDB())) ||
-                         EmitDwarfDebugEntryValues;
+  EmitDebugEntryValues = Asm->TM.Options.ShouldEmitDebugEntryValues();
 
----------------
Wouldn't we want `(tuneForGDB() || tuneForLLDB())` to override the target default? If someone were to debug with a hypothetical GDB port for SCE targets, for example?


================
Comment at: llvm/test/DebugInfo/MIR/X86/DW_OP_entry_value.mir:18
 
+# SCE-NOT: DW_OP_GNU_entry_value
+
----------------
Wouldn't you want to check for `DW_OP_{{.*}}entry_value`?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83462/new/

https://reviews.llvm.org/D83462