Ping: [patch] -m128bit-long-double asserts
Stuart Hastings
stuart@apple.com
Fri Apr 1 22:29:00 GMT 2005
On Apr 1, 2005, at 11:21 AM, Geoff Keating wrote:
>
> On 01/04/2005, at 12:58 AM, Stuart Hastings wrote:
[snip]
>> To insure we're talking about the same patch, here's what I thought
>> you suggested:
>>
>> <i386.h.diffs.txt>
>>
>> 2005-04-01 Stuart Hastings <stuart@apple.com>
>>
>> * config/i386/i386.h(HARD_REGNO_NREGS): Aware of
>> -m128bit-long-double.
>>
>> (This version hasn't been bootstrapped yet...)
>
> Yes, that looks right. I think it's the most reasonable thing. Of
> course, it may cause other problems
> (does x86 even have 8 32-bit registers?)
Define "register." :-) Literally yes, there are eight, but two are
effectively dedicated to be the frame-pointer and stack-pointer, so the
useful answer is "six."
I'm not sure this will be a problem; I tried a trivial XFmode testcase
with -msoft-float -m128bit-long-double, and it did not touch the
padding. Maybe six registers are sufficient.
> so it'll need a full dejagnu run.
Bootstrapped, DejaGnu (C/C++/ObjC) on Linux/x86. No regressions.
2005-04-01 Stuart Hastings <stuart@apple.com>
* config/i386/i386.h(HARD_REGNO_NREGS): Aware of -m128bit-long-double.
* gcc.c-torture/compile/20050401-1.c: New testcase for above.
The testcase is here:
http://gcc.gnu.org/ml/gcc-patches/2005-03/msg01503.html
O.K. to commit? (Mainline, I presume?)
stuart
More information about the Gcc-patches
mailing list