* PING: PATCH: PR target/70738: Add -mgeneral-regs-only option
@ 2016-05-20 15:02 H.J. Lu
0 siblings, 0 replies; only message in thread
From: H.J. Lu @ 2016-05-20 15:02 UTC (permalink / raw)
To: Sandra Loosemore
Cc: Koval, Julia, gcc-patches, Lu, Hongjiu, vaalfreja, ubizjak, law,
Zamyatin, Igor
On Fri, May 13, 2016 at 8:00 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Thu, May 12, 2016 at 10:54 AM, H.J. Lu <hjl.tools@gmail.com> 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.
>>>>
>>>> OK for trunk if there is no regression?
>>>
>>>
>>> I can't comment on the code patch, but for the documentation part:
>>>
>>>> @@ -24242,6 +24242,12 @@ opcodes, to mitigate against certain forms of
>>>> attack. At the moment,
>>>> this option is limited in what it can do and should not be relied
>>>> on to provide serious protection.
>>>>
>>>> +@item -mgeneral-regs-only
>>>> +@opindex mgeneral-regs-only
>>>> +Generate code which uses only the general-purpose registers. This will
>>>
>>>
>>> s/which/that/
>>>
>>>> +prevent the compiler from using floating-point, vector, mask and bound
>>>
>>>
>>> s/will prevent/prevents/
>>>
>>>> +registers, but will not impose any restrictions on the assembler.
>>>
>>>
>>> Maybe you mean to say "does not restrict use of those registers in inline
>>> assembly code"? In any case, please get rid of the future tense here, too.
>>
>> 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. Tested on x86-64. OK for trunk?
>
PING.
--
H.J.
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2016-05-20 15:02 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-20 15:02 PING: PATCH: PR target/70738: Add -mgeneral-regs-only option H.J. Lu
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).