[PATCH] D116628: [X86] add dwarf information for loop stack probe

Thu Jan 6 03:37:03 PST 2022

nagisa added a comment.

LGTM after the code change is applied.

================
Comment at: llvm/lib/Target/X86/X86FrameLowering.cpp:715-720
+      BuildCFI(MBB, MBBI, DL,
+               MCCFIInstruction::createDefCfaRegister(
+                   nullptr, TRI->getDwarfRegNum(DwarfFinalStackProbed, true)));
+      BuildCFI(MBB, MBBI, DL,
+               MCCFIInstruction::createAdjustCfaOffset(nullptr, BoundOffset));
+    }
----------------

================
Comment at: llvm/test/CodeGen/X86/stack-clash-large.ll:19-20
 ; CHECK-X64-NEXT:    subq $2248, %rsp # imm = 0x8C8
+; CHECK-X64-NEXT:    .cfi_def_cfa_register %rsp
 ; CHECK-X64-NEXT:    .cfi_def_cfa_offset 71888
 ; CHECK-X64-NEXT:    movl $1, 264(%rsp)
----------------
In theory this could also be `.cfi_def_cfa %rsp, 71888` but seems like it may be non-trivial to make it happen.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116628