Re: PATCH: PR target/70738: Add -mgeneral-regs-only option

2016-05-20 Thread Rainer Orth
"H.J. Lu" writes: > On Thu, May 12, 2016 at 10:54 AM, H.J. Lu wrote: Here is a patch to add -mgeneral-regs-only option to x86 backend. We can update spec for interrupt handle to recommend compiling interrupt handler with -mgeneral-regs-only option and add a note for compile

Re: PATCH: PR target/70738: Add -mgeneral-regs-only option

2016-05-20 Thread Sandra Loosemore
On 05/13/2016 09:00 AM, H.J. Lu wrote: I changed it to --- @item -mgeneral-regs-only @opindex mgeneral-regs-only Generate code that uses only the general-purpose registers. This prevents the compiler from using floating-point, vector, mask and bound registers. --- Here is the updated patch.

Re: PATCH: PR target/70738: Add -mgeneral-regs-only option

2016-05-20 Thread Uros Bizjak
On Fri, May 13, 2016 at 5:00 PM, H.J. Lu wrote: > On Thu, May 12, 2016 at 10:54 AM, H.J. Lu wrote: Here is a patch to add -mgeneral-regs-only option to x86 backend. We can update spec for interrupt handle to recommend compiling interrupt handler with -mgeneral-regs-only opti

PING: PATCH: PR target/70738: Add -mgeneral-regs-only option

2016-05-20 Thread H.J. Lu
On Fri, May 13, 2016 at 8:00 AM, H.J. Lu wrote: > On Thu, May 12, 2016 at 10:54 AM, H.J. Lu wrote: Here is a patch to add -mgeneral-regs-only option to x86 backend. We can update spec for interrupt handle to recommend compiling interrupt handler with -mgeneral-regs-only opti

Re: PATCH: PR target/70738: Add -mgeneral-regs-only option

2016-05-13 Thread H.J. Lu
On Thu, May 12, 2016 at 10:54 AM, H.J. Lu wrote: >>> Here is a patch to add >>> -mgeneral-regs-only option to x86 backend. We can update >>> spec for interrupt handle to recommend compiling interrupt handler >>> with -mgeneral-regs-only option and add a note for compiler >>> implementers. >>> >>

Re: PATCH: PR target/70738: Add -mgeneral-regs-only option

2016-05-12 Thread H.J. Lu
On Thu, May 12, 2016 at 8:46 AM, Sandra Loosemore wrote: > On 05/11/2016 11:02 AM, H.J. Lu wrote: >> >> On Tue, May 10, 2016 at 1:02 PM, Sandra Loosemore >> wrote: >>> >>> >>> Again, this sounds like implementor-speak, and there are grammatical >>> errors >>> (noun/verb disagreement, missing arti

Re: PATCH: PR target/70738: Add -mgeneral-regs-only option

2016-05-12 Thread Sandra Loosemore
On 05/11/2016 11:02 AM, H.J. Lu wrote: On Tue, May 10, 2016 at 1:02 PM, Sandra Loosemore wrote: Again, this sounds like implementor-speak, and there are grammatical errors (noun/verb disagreement, missing articles). Do users of this attribute need to know what instructions the compiler is emi

PATCH: PR target/70738: Add -mgeneral-regs-only option

2016-05-11 Thread H.J. Lu
On Tue, May 10, 2016 at 1:02 PM, Sandra Loosemore wrote: > On 04/20/2016 07:42 AM, Koval, Julia wrote: >> >> diff --git a/gcc/doc/extend.texi b/gcc/doc/extend.texi >> index a5a8b23..82de5bf 100644 >> --- a/gcc/doc/extend.texi >> +++ b/gcc/doc/extend.texi >> @@ -5263,6 +5263,83 @@ On x86-32 targets