This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: *_ALIGN_MAX_SKIP macros
- From: Ian Lance Taylor <iant at google dot com>
- To: DJ Delorie <dj at redhat dot com>
- Cc: gcc at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
- Date: Tue, 12 Oct 2010 15:37:49 -0700
- Subject: Re: *_ALIGN_MAX_SKIP macros
- References: <201007272000.o6RK0IrS023354@greed.delorie.com> <mcraapcqhvs.fsf@google.com> <201010060034.o960Y5Qk022058@greed.delorie.com>
DJ Delorie <dj@redhat.com> writes:
> * doc/tm.texi.in (TARGET_ASM_JUMP_ALIGN_MAX_SKIP): New.
> (TARGET_ASM_LABEL_ALIGN_AFTER_BARRIER_MAX_SKIP): Change to hook.
> (TARGET_ASM_LOOP_ALIGN_MAX_SKIP): Likewise.
> (TARGET_ASM_LABEL_ALIGN_MAX_SKIP): Likewise.
>
> * targhooks.h (default_label_align_after_barrier_max_skip,
> default_loop_align_max_skip, default_label_align_max_skip,
> default_jump_align_max_skip): Declare.
>
> * target.def (label_align_after_barrier_max_skip): New.
> (loop_align_max_skip): New.
> (label_align_max_skip): New.
> (jump_align_max_skip): New.
>
> * final.c (LABEL_ALIGN_MAX_SKIP): Remove.
> (LOOP_ALIGN_MAX_SKIP): Remove.
> (LABEL_ALIGN_AFTER_BARRIER_MAX_SKIP): Remove.
> (JUMP_ALIGN_MAX_SKIP): Remove.
> (default_label_align_after_barrier_max_skip): New.
> (default_loop_align_max_skip): New.
> (default_label_align_max_skip): New.
> (default_jump_align_max_skip): New.
> (compute_alignments): Use the new hooks.
> (shorten_branches): Likewise.
This is OK if you add LABEL_ALIGN_MAX_SKIP, LOOP_ALIGN_MAX_SKIP,
LABEL_ALIGN_AFTER_BARRIER_MAX_SKIP, and JUMP_ALIGN_MAX_SKIP to the
/* Old target macros that have moved to the target hooks structure. */
#pragma GCC poison list in system.h.
Thanks.
Ian