PATC: Round up the SSE register save area only if needed

Uros Bizjak ubizjak@gmail.com
Thu Oct 8 18:44:00 GMT 2015


On Thu, Oct 8, 2015 at 8:29 PM, H.J. Lu <hongjiu.lu@intel.com> wrote:
> There is is no point to round up the SSE register save area to 16 bytes if
> the incoming stack boundary is less than 16 bytes.
>
> OK for trunk?

OK, with the improved comment:

    /* The only ABI that has saved SSE registers (Win64) also has a
       16-byte aligned default stack, and thus we don't need to be
           within the re-aligned local stack frame to save them.  In case
       incoming stack boundary is aligned to less than 16 bytes,
       unaligned move of SSE register will be emitted, so there is
       no point to round up the SSE register save area outside the
       re-aligned local stack frame to 16 bytes.  */

> H.J.
> ---
>         * config/i386/i386.c (ix86_compute_frame_layout): Round up the
>         SSE register save area to 16 bytes only if the incoming stack
>         boundary is no less than 16 bytes.

Thanks,
Uros.



More information about the Gcc-patches mailing list