[PATCH] D127171: [DWARF] Support 'G' in dwarf parser

[Fangrui Song via Phabricator via llvm-commits]

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

LGTM.



================
Comment at: llvm/test/DebugInfo/AArch64/stack-tagging-cfi.s:1
+# RUN: llvm-mc -filetype=obj -triple aarch64-generic-linux -mattr=+mte %s -o %t.o
+# RUN: llvm-objdump -s --dwarf=frames %t.o | FileCheck %s
----------------
Use an generic ELF target triple to indicate that this does not need Linux features.


================
Comment at: llvm/test/DebugInfo/AArch64/stack-tagging-cfi.s:2
+# RUN: llvm-mc -filetype=obj -triple aarch64-generic-linux -mattr=+mte %s -o %t.o
+# RUN: llvm-objdump -s --dwarf=frames %t.o | FileCheck %s
+
----------------
`llvm-dwarfdump --eh-frame %t.o`


================
Comment at: llvm/test/DebugInfo/AArch64/stack-tagging-cfi.s:19
+                                        // -- End function
+	.globl	NoUnwind                        // -- Begin function NoUnwind
+	.p2align	2
----------------
Remove


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127171