[PATCH] D101339: [DebugInfo] Add tests that we emit .eh_frame instead of .debug_frame

Venkata Ramanaiah Nalamothu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 26 21:27:02 PDT 2021


RamNalamothu added a subscriber: scott.linder.
RamNalamothu added inline comments.


================
Comment at: llvm/test/DebugInfo/X86/cfi_sections.ll:3
+; RUN: llc < %t/1.ll -mtriple=x86_64 | FileCheck %s --check-prefix=DEBUG_FRAME
+; RUN: llc < %t/2.ll -mtriple=x86_64 | FileCheck %s --check-prefix=EH_FRAME
+; RUN: llc < %t/3.ll -mtriple=x86_64 | FileCheck %s --check-prefix=EH_FRAME
----------------
May be add `--force-dwarf-frame-section` scenario as well, to make sure we get `.debug_frame` along with `.eh_frame` when asked for?

While the rest looks good to me, from logical point of view, I can't read LLVM IR well. So, not able to comment on that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101339



More information about the llvm-commits mailing list