--- Comment #1 from pinskia at gcc dot gnu dot org 2006-04-03 19:10 ---
This worked in 4.0.2, by just loading the constant via memory so this is a
regression.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--
--- Comment #2 from pinskia at gcc dot gnu dot org 2006-04-03 19:17 ---
Confirmed, this is definitely wrong but we can do better than disabling this
for odd vectors I think. Adding 1 to them should work.
--
pinskia at gcc dot gnu dot org changed:
What|Removed
--- Comment #3 from bonzini at gnu dot org 2006-04-06 10:56 ---
For 4.1 I'd surely go with Ulrich's patch. And for 4.2 I think that it starts
being too expensive to have two vsplat and two adds.
vspltisw 0,15
vspltisw 1,1
vadduwm 0,0,0
vadduwm 0,0,1
Ulr
--- Comment #4 from uweigand at gcc dot gnu dot org 2006-04-06 14:03
---
(In reply to comment #3)
> Ulrich, can you prepare a patch or should I do so?
It would be great if you could do that, I don't yet
have a proper setup for ppc testing ...
--
http://gcc.gnu.org/bugzilla/show_bu
--- Comment #5 from bonzini at gnu dot org 2006-04-06 14:47 ---
Created an attachment (id=11217)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=11217&action=view)
patch to fix the bug
Patch untested but quite trivial.
--
bonzini at gnu dot org changed:
What|Remo
--- Comment #6 from uweigand at gcc dot gnu dot org 2006-04-13 11:47
---
I've now tested and submitted the patch, thanks.
--
uweigand at gcc dot gnu dot org changed:
What|Removed |Added
-
--- Comment #7 from dje at gcc dot gnu dot org 2006-04-13 15:37 ---
Paolo, I'm not sure why you think that two splats and two adds is too
expensive. Memory is moving farther and farther away from the processor.
--
dje at gcc dot gnu dot org changed:
What|Removed
--- Comment #8 from uweigand at gcc dot gnu dot org 2006-04-13 20:27
---
Subject: Bug 27006
Author: uweigand
Date: Thu Apr 13 20:26:59 2006
New Revision: 112923
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=112923
Log:
2006-04-13 Paolo Bonzini <[EMAIL PROTECTED]>
--- Comment #9 from uweigand at gcc dot gnu dot org 2006-04-13 20:33
---
Subject: Bug 27006
Author: uweigand
Date: Thu Apr 13 20:33:51 2006
New Revision: 112924
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=112924
Log:
2006-04-13 Paolo Bonzini <[EMAIL PROTECTED]>
--- Comment #10 from uweigand at gcc dot gnu dot org 2006-04-13 20:35
---
Fixed for 4.1 and mainline.
--
uweigand at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #11 from paolo dot bonzini at lu dot unisi dot ch 2006-04-14
07:27 ---
Subject: Re: [4.1/4.2 Regression] Invalid altivec constant
loading code
> I'm not sure why you think that two splats and two adds is too
> expensive.
>
I'd hope that these constants stay in the cac
--- Comment #12 from dje at watson dot ibm dot com 2006-04-14 14:32 ---
Subject: Re: [4.1/4.2 Regression] Invalid altivec constant loading code
One can produce a few more values fairly easily, but having GCC
know the optimal sequence for all constants could be rather bulky.
h
12 matches
Mail list logo