[PATCH] D76879: [AMDGPU] Begin emitting CFI for AMDGCN

David Greene via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 30 13:32:00 PDT 2020


greened accepted this revision.
greened added inline comments.


================
Comment at: llvm/test/MC/ELF/AMDGPU/cfi.s:13
+# ASM-NEXT: .cfi_sections .debug_frame
+# FIXME Why emit an extra empty line?
+# ASM-EMPTY:
----------------
Are these intended to be fixed before merge or is the test just nothing something should be fixed?  If the latter, perhaps we need an XFAIL test for it?  But maybe it's not worth it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76879



More information about the llvm-commits mailing list