[Bug tree-optimization/52760] Revision 185599 causes miscompare on sphinx3

2012-03-29 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760 Jakub Jelinek changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|

[Bug tree-optimization/52760] Revision 185599 causes miscompare on sphinx3

2012-03-29 Thread jakub at gcc dot gnu.org
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

[Bug tree-optimization/52760] Revision 185599 causes miscompare on sphinx3

2012-03-29 Thread rguenther at suse dot de
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

[Bug tree-optimization/52760] Revision 185599 causes miscompare on sphinx3

2012-03-29 Thread jakub at gcc dot gnu.org
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

[Bug tree-optimization/52760] Revision 185599 causes miscompare on sphinx3

2012-03-29 Thread rguenth at gcc dot gnu.org
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

[Bug tree-optimization/52760] Revision 185599 causes miscompare on sphinx3

2012-03-29 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760 Jakub Jelinek changed: What|Removed |Added Attachment #27028|0 |1 is obsolete|

[Bug tree-optimization/52760] Revision 185599 causes miscompare on sphinx3

2012-03-29 Thread rguenther at suse dot de
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

[Bug tree-optimization/52760] Revision 185599 causes miscompare on sphinx3

2012-03-29 Thread jakub at gcc dot gnu.org
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.

[Bug tree-optimization/52760] Revision 185599 causes miscompare on sphinx3

2012-03-29 Thread rguenth at gcc dot gnu.org
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

[Bug tree-optimization/52760] Revision 185599 causes miscompare on sphinx3

2012-03-29 Thread jakub at gcc dot gnu.org
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

[Bug tree-optimization/52760] Revision 185599 causes miscompare on sphinx3

2012-03-29 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52760 Jakub Jelinek changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|