[PATCH] D105820: [DebugInfo][InstrRef] Fix a broken substitution method, add test coverage

[Orlando Cazalet-Hyams via Phabricator via llvm-commits]

Orlando accepted this revision.
Orlando added a comment.
This revision is now accepted and ready to land.

The tests look good. LGTM!



================
Comment at: llvm/test/DebugInfo/MIR/InstrRef/x86-lea-fixup-2.mir:3
+#
+# Test that various LEA <=> ADD transforms get substitutions applied to them.
+---
----------------
nit: It loos like this file is just testing LEA => ADD (rather than LEA <=> ADD), or am I missing something?


================
Comment at: llvm/test/DebugInfo/MIR/InstrRef/x86-lea-fixup-2.mir:40
+    $ebx = LEA32r killed $ebp, 1, $ebp, 0, $noreg, debug-instr-number 1
+    ; jmorse, i386 -mcpu=corei7-avx
+    RETQ $ebx
----------------
What is this comment for?


================
Comment at: llvm/test/DebugInfo/MIR/InstrRef/x86-lea-fixup.mir:5
+#
+# Test each call to substitute debug instr-ref labeles in the fixup-leas pass.
+# Some are only reachable under specific CPU modes it seems -- each function
----------------
I'm finding it difficult to understand this opening sentence and nit: labeles -> labels 




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105820