recommended alignment for local i386 FP variables patch (version 3)

Jeffrey A Law
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  (
  > 	* 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.



ps.  Now to wrap our heads around the best way to align the stack itself :-)

More information about the Gcc-patches mailing list