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