http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760
Jakub Jelinek changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760
--- Comment #10 from Jakub Jelinek 2012-03-29
14:56:02 UTC ---
Author: jakub
Date: Thu Mar 29 14:55:53 2012
New Revision: 185965
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=185965
Log:
PR tree-optimization/52760
* tree-vect-slp
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760
--- Comment #9 from rguenther at suse dot de
2012-03-29 13:19:33 UTC ---
On Thu, 29 Mar 2012, jakub at gcc dot gnu.org wrote:
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760
>
> --- Comment #8 from Jakub Jelinek 2012-03-29
> 12:54:31 UTC
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760
--- Comment #8 from Jakub Jelinek 2012-03-29
12:54:31 UTC ---
(In reply to comment #7)
> (In reply to comment #6)
> > Created attachment 27030 [details]
> > gcc48-pr52760.patch
> >
> > So do you prefer this instead?
>
> Yes. In a non-constant
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760
--- Comment #7 from Richard Guenther 2012-03-29
12:46:20 UTC ---
(In reply to comment #6)
> Created attachment 27030 [details]
> gcc48-pr52760.patch
>
> So do you prefer this instead?
Yes. In a non-constant case we'd miss a conversion, so I su
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760
Jakub Jelinek changed:
What|Removed |Added
Attachment #27028|0 |1
is obsolete|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760
--- Comment #5 from rguenther at suse dot de
2012-03-29 10:50:49 UTC ---
On Thu, 29 Mar 2012, jakub at gcc dot gnu.org wrote:
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760
>
> --- Comment #4 from Jakub Jelinek 2012-03-29
> 10:39:24 UTC
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760
--- Comment #4 from Jakub Jelinek 2012-03-29
10:39:24 UTC ---
But where else? The op is copied from the stmts (where it is certainly ok as
is) just a few lines above this.
Note, using VCE if the types have different sizes is invalid anyway.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760
--- Comment #3 from Richard Guenther 2012-03-29
10:34:54 UTC ---
Hmm, but that means the shift count should be promoted/demoted at another
place, not here. Because this place is not supposed to handle
promotion/demotion
but conversion of tcc_con
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760
--- Comment #2 from Jakub Jelinek 2012-03-29
09:33:31 UTC ---
Created attachment 27028
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27028
gcc48-pr52760.patch
Untested fix. Using VCE here is undesriable, on big-endian it turns the shift
c
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760
Jakub Jelinek changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed|
11 matches
Mail list logo