recommended alignment for local i386 FP variables patch (version 3)
Jeffrey A Law
law@hurl.cygnus.com
Sun Feb 28 18:15:00 GMT 1999
In message < 199902050714.CAA08334@jwlab.FEITH.COM >you write:
> [ This version allows assign_stack_temp_for_type to reuse slots
> which have larger alignments in addition to updating the comments. ]
>
> This patch causes gcc to use the recommended Intel alignment for
> i386 FP variables when allocating them within the frame.
>
> Notes:
>
> 1) This patch * does not * address the issue of aligning the frame,
> just the issue of aligning the variables within the frame. There
> are two ways of aligning the frame which I address in two other
> separate patches.
>
> ChangeLog:
>
> Fri Feb 5 00:04:46 EST 1999 John Wehle (john@feith.com)
>
> * i386.h (LOCAL_ALIGNMENT): Define.
> * function.c (assign_stack_local, assign_outer_stack_local): Use it.
> (assign_stack_temp_for_type): New function based on assign_stack_temp.
> (assign_stack_temp): Call it.
> (assign_temp): Use assign_stack_temp_for_type, not assign_stack_temp.
> * stmt.c: Use assign_temp, not assign_stack_temp.
> * tm.texi: Document LOCAL_ALIGNMENT.
I installed this (and the incremental patch you sent on Sat).
I made a minor change -- function.c will provide a definition for
LOCAL_ALIGNMENT if the backend doesn't. And the #ifdef LOCAL_ALIGNMENT #endif
pairs went away.
Thanks,
jeff
ps. Now to wrap our heads around the best way to align the stack itself :-)
:-)
More information about the Gcc-patches
mailing list