This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [PR79793] Fixes interrupt stack alignment
Uhhh, copypaste error, sorry.
PR target/79793
gcc/
* config/i386/i386.c (ix86_minimum_incoming_stack_boundary): Set incoming
stack boundary to 128 for 64-bit targets.
gcc/testsuite/
* gcc.target/i386/interrupt-12.c: Update scan-assembler-times directives.
* gcc.target/i386/interrupt-13.c: Ditto.
* gcc.target/i386/interrupt-14.c: Ditto.
* gcc.target/i386/interrupt-15.c: Ditto.
-----Original Message-----
From: Koval, Julia
Sent: Monday, March 06, 2017 1:03 PM
To: 'Uros Bizjak' <ubizjak@gmail.com>
Cc: GCC Patches <gcc-patches@gcc.gnu.org>; H.J. Lu <hjl.tools@gmail.com>
Subject: RE: [PR79793] Fixes interrupt stack alignment
Ok, fixed it. Can you please commit it for me?
PR target/79731
gcc/
* config/i386/i386.c (ix86_minimum_incoming_stack_boundary): Set incoming
stack boundary to 128 for 64-bit targets.
gcc/testsuite/
* gcc.target/i386/interrupt-12.c: Update scan-assembler-times directives.
* gcc.target/i386/interrupt-13.c: Ditto.
* gcc.target/i386/interrupt-14.c: Ditto.
* gcc.target/i386/interrupt-15.c: Ditto.
Thanks,
Julia
-----Original Message-----
From: Uros Bizjak [mailto:ubizjak@gmail.com]
Sent: Friday, March 03, 2017 4:29 PM
To: Koval, Julia <julia.koval@intel.com>
Cc: GCC Patches <gcc-patches@gcc.gnu.org>; H.J. Lu <hjl.tools@gmail.com>
Subject: Re: [PR79793] Fixes interrupt stack alignment
On Fri, Mar 3, 2017 at 4:20 PM, Koval, Julia <julia.koval@intel.com> wrote:
> Hi,
> This patch fixes PR79731. Ok for trunk?
>
> gcc/
> * config/i386/i386.c (ix86_minimum_incoming_stack_boundary): Fix boundary for interrupt
...: Set incoming stack boundary to 128 for 64-bit targets.
> gcc/testsuite/
> * gcc.target/i386/interrupt-12.c: Fix test.
> * gcc.target/i386/interrupt-13.c: Ditto.
> * gcc.target/i386/interrupt-14.c: Ditto.
> * gcc.target/i386/interrupt-15.c: Ditto.
...: Update scan-assembler-times directives.
Please also add "PR target/79731" to the top of ChangeLog entry.
OK with the above ChangeLog changes.
Thanks,
Uros.