Hi all, This patch adds a new target hook called "asm_post_cfi_startproc". This hook is intended to be used by the aarch64 backend to emit a directive that enables support for unwinding frames signed with the pointer authentication B-key. This hook is triggered after the ".cfi_startproc" directive is emitted in gcc/dwarf2out.c.
Bootstrapped on aarch64-none-linux-gnu and tested on aarch64-none-elf with no regressions. Ok for trunk? gcc/ 2018-11-02 Sam Tebbs<sam.te...@arm.com> * doc/tm.texi (TARGET_ASM_POST_CFI_STARTPROC): Define. * doc/tm.texi.in (TARGET_ASM_POST_CFI_STARTPROC): Define. * dwarf2out.c (dwarf2out_do_cfi_startproc): Trigger the hook. * hooks.c (hook_void_FILEptr_tree): Define. * hooks.h (hook_void_FILEptr_tree): Define. * target.def (post_cfi_startproc): Define.
diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi index f841527..e26c0a7 100644 --- a/gcc/doc/tm.texi +++ b/gcc/doc/tm.texi @@ -9413,6 +9413,14 @@ If this macro is not defined, nothing special is output at the end of the jump-table. @end defmac +@deftypefn {Target Hook} void TARGET_ASM_POST_CFI_STARTPROC (FILE *@var{}, @var{tree}) +This target hook is used to emit assembly strings required by the target +after the .cfi_startproc directive. The first argument is the file stream to +write the strings to and the second argument is the function's declaration. + +The default is to not output any assembly strings. +@end deftypefn + @deftypefn {Target Hook} void TARGET_ASM_EMIT_UNWIND_LABEL (FILE *@var{stream}, tree @var{decl}, int @var{for_eh}, int @var{empty}) This target hook emits a label at the beginning of each FDE@. It should be defined on targets where FDEs need special labels, and it diff --git a/gcc/doc/tm.texi.in b/gcc/doc/tm.texi.in index 967ef3a..7d933c0 100644 --- a/gcc/doc/tm.texi.in +++ b/gcc/doc/tm.texi.in @@ -6426,6 +6426,8 @@ If this macro is not defined, nothing special is output at the end of the jump-table. @end defmac +@hook TARGET_ASM_POST_CFI_STARTPROC + @hook TARGET_ASM_EMIT_UNWIND_LABEL @hook TARGET_ASM_EMIT_EXCEPT_TABLE_LABEL diff --git a/gcc/dwarf2out.c b/gcc/dwarf2out.c index 30bbfee..6c1531a 100644 --- a/gcc/dwarf2out.c +++ b/gcc/dwarf2out.c @@ -969,6 +969,8 @@ dwarf2out_do_cfi_startproc (bool second) fprintf (asm_out_file, "\t.cfi_startproc\n"); + targetm.asm_out.post_cfi_startproc (asm_out_file, current_function_decl); + /* .cfi_personality and .cfi_lsda are only relevant to DWARF2 eh unwinders. */ if (targetm_common.except_unwind_info (&global_options) != UI_DWARF2) diff --git a/gcc/hooks.h b/gcc/hooks.h index 0ed5b95..bcfc231 100644 --- a/gcc/hooks.h +++ b/gcc/hooks.h @@ -82,6 +82,7 @@ extern void hook_void_FILEptr_constcharptr_const_tree (FILE *, const char *, const_tree); extern bool hook_bool_FILEptr_rtx_false (FILE *, rtx); extern void hook_void_rtx_tree (rtx, tree); +extern void hook_void_FILEptr_tree (FILE *, tree); extern void hook_void_tree (tree); extern void hook_void_tree_treeptr (tree, tree *); extern void hook_void_int_int (int, int); diff --git a/gcc/hooks.c b/gcc/hooks.c index 780cc1e..46bf2a8 100644 --- a/gcc/hooks.c +++ b/gcc/hooks.c @@ -277,6 +277,11 @@ hook_void_tree (tree) } void +hook_void_FILEptr_tree (FILE *, tree) +{ +} + +void hook_void_rtx_tree (rtx, tree) { } diff --git a/gcc/target.def b/gcc/target.def index ad27d35..3b0022d 100644 --- a/gcc/target.def +++ b/gcc/target.def @@ -87,6 +87,17 @@ when the relevant string is @code{NULL}.", bool, (rtx x, unsigned int size, int aligned_p), default_assemble_integer) +/* Assembly strings required after the .cfi_startproc label. */ +DEFHOOK +(post_cfi_startproc, + "This target hook is used to emit assembly strings required by the target\n\ +after the .cfi_startproc directive. The first argument is the file stream to\n\ +write the strings to and the second argument is the function\'s declaration.\n\ +\n\ +The default is to not output any assembly strings.", + void, (FILE *, tree), + hook_void_FILEptr_tree) + /* Notify the backend that we have completed emitting the data for a decl. */ DEFHOOK