RE: [PR79793] Fixes interrupt stack alignment

2017-03-06 Thread Koval, Julia 
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' <ubiz...@gmail.com>
Cc: GCC Patches <gcc-patches@gcc.gnu.org>; H.J. Lu <hjl.to...@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:ubiz...@gmail.com] 
Sent: Friday, March 03, 2017 4:29 PM
To: Koval, Julia <julia.ko...@intel.com>
Cc: GCC Patches <gcc-patches@gcc.gnu.org>; H.J. Lu <hjl.to...@gmail.com>
Subject: Re: [PR79793] Fixes interrupt stack alignment

On Fri, Mar 3, 2017 at 4:20 PM, Koval, Julia <julia.ko...@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.

RE: [PR79793] Fixes interrupt stack alignment

2017-03-06 Thread Koval, Julia 
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:ubiz...@gmail.com] 
Sent: Friday, March 03, 2017 4:29 PM
To: Koval, Julia <julia.ko...@intel.com>
Cc: GCC Patches <gcc-patches@gcc.gnu.org>; H.J. Lu <hjl.to...@gmail.com>
Subject: Re: [PR79793] Fixes interrupt stack alignment

On Fri, Mar 3, 2017 at 4:20 PM, Koval, Julia <julia.ko...@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.

Re: [PR79793] Fixes interrupt stack alignment

2017-03-03 Thread Uros Bizjak 
On Fri, Mar 3, 2017 at 4:20 PM, Koval, Julia  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.