Still can't make stfiwx actually store directly, but...

Geoffrey Keating geoffk@apple.com
Wed May 18 00:23:00 GMT 2005


On 17/05/2005, at 4:59 PM, Zack Weinberg wrote:

> gkeating@apple.com (Geoffrey Keating) writes:
>
>
>> This patch has no effect.
>>
>> It ought to have an effect.  What it should do is make a short chunk
>> of code like this:
>>
>> void foo(int *x, double d)
>> { *x = d; }
>>
>> generate code like:
>>
>>         fctiwz f0,f1
>>         stfiwx f0,0,r3
>>         blr
>>
>> instead of pointlessly generating a temporary, like:
>>
>>         fctiwz f0,f1
>>         addi r2,r1,-24
>>         stfiwx f0,0,r2
>>         lwz r0,-24(r1)
>>         stw r0,0(r3)
>>         blr
>>
>
> Hm, you'd think combine could put it back together again... too late
> to not allocate the stack slot, but at least you'd save the store and
> load.

Combine only knows about pseudos, not memory locations, unfortunately.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 2408 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20050518/40842ef5/attachment.p7s>


More information about the Gcc-patches mailing list