Hi Andrea,

> -----Original Message-----
> From: Gcc-patches <gcc-patches-
> bounces+kyrylo.tkachov=arm....@gcc.gnu.org> On Behalf Of Andrea
> Corallo via Gcc-patches
> Sent: Friday, August 12, 2022 4:40 PM
> To: Andrea Corallo via Gcc-patches <gcc-patches@gcc.gnu.org>
> Cc: Richard Earnshaw <richard.earns...@arm.com>; nd <n...@arm.com>
> Subject: [PATCH 11/15] aarch64: Make bti pass generic so it can be used by
> the arm backend
> 
> Hi all,
> 
> this patch splits and restructures the aarch64 bti pass code in order
> to have it usable by the arm backend as well.  These changes have no
> functional impact.
> 
> The original patch was approved here:
> <https://gcc.gnu.org/pipermail/gcc-patches/2022-May/594172.html>.
> 
> After that Richard E. noted that was better to move the new pass
> definition for arm in the following patch and so I did.
> 

Ok. The renaming and splits look fine and as long as it builds without problems 
on arm and aarch64 it's all good.
Thanks,
Kyirll


> Best Regards
> 
>   Andrea
> 
> gcc/Changelog
> 
>       * config.gcc (aarch64*-*-*): Rename 'aarch64-bti-insert.o' into
>       'aarch-bti-insert.o'.
>       * config/aarch64/aarch64-protos.h: Remove 'aarch64_bti_enabled'
>       proto.
>       * config/aarch64/aarch64.cc (aarch_bti_enabled): Rename.
>       (aarch_bti_j_insn_p, aarch_pac_insn_p): New functions.
>       (aarch64_output_mi_thunk)
>       (aarch64_print_patchable_function_entry)
>       (aarch64_file_end_indicate_exec_stack): Update renamed function
>       calls to renamed functions.
>       * config/aarch64/t-aarch64 (aarch-bti-insert.o): Update target.
>       * config/arm/aarch-bti-insert.cc: New file including and
>       generalizing code from aarch64-bti-insert.cc.
>       * config/arm/aarch-common-protos.h: Update.

Reply via email to