[Bug tree-optimization/43280] [4.5 Regression] gcc4.5 -m32 -O2: misoptimizes sha256!

2010-03-11 Thread krebbel at gcc dot gnu dot org 


--- Comment #7 from krebbel at gcc dot gnu dot org  2010-03-11 13:51 ---
Subject: Bug 43280

Author: krebbel
Date: Thu Mar 11 13:51:00 2010
New Revision: 157386

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=157386
Log:
2010-03-11  Andreas Krebbel  andreas.kreb...@de.ibm.com

PR tree-optimization/43280
* tree-ssa-math-opts.c (find_bswap_1): Modify symbolic number
generation.
Move calculation of size out of the if branch.
(find_bswap): Modify compare number generation.

2010-03-11  Andreas Krebbel  andreas.kreb...@de.ibm.com

* gcc.dg/optimize-bswapdi-1.c: Add OpenSSL bswap variant.
* gcc.dg/pr43280.c: New testcase.



Added:
trunk/gcc/testsuite/gcc.dg/pr43280.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/gcc.dg/optimize-bswapdi-1.c
trunk/gcc/tree-ssa-math-opts.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43280

[Bug tree-optimization/43280] [4.5 Regression] gcc4.5 -m32 -O2: misoptimizes sha256!

2010-03-11 Thread krebbel at gcc dot gnu dot org 


--- Comment #8 from krebbel at gcc dot gnu dot org  2010-03-11 14:06 ---
(In reply to comment #6)
 Why does the new testcase have
 
 +/* { dg-require-effective-target lp64 } */
 
 when the failure is shown with -m32 on x86-64?
 

I've removed that in the new version. Thanks!


-- 

krebbel at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43280

[Bug tree-optimization/43280] [4.5 Regression] gcc4.5 -m32 -O2: misoptimizes sha256!

2010-03-10 Thread rguenth at gcc dot gnu dot org 


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

   Priority|P3  |P1


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43280

[Bug tree-optimization/43280] [4.5 Regression] gcc4.5 -m32 -O2: misoptimizes sha256!

2010-03-10 Thread krebbel at gcc dot gnu dot org 


--- Comment #5 from krebbel at gcc dot gnu dot org  2010-03-10 16:31 ---
Created an attachment (id=20074)
 -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20074action=view)
Experimental patch

This patch fixes the problem for me. Testsuites are still running.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43280

[Bug tree-optimization/43280] [4.5 Regression] gcc4.5 -m32 -O2: misoptimizes sha256!

2010-03-10 Thread hjl dot tools at gmail dot com 


--- Comment #6 from hjl dot tools at gmail dot com  2010-03-10 17:50 ---
(In reply to comment #5)
 Created an attachment (id=20074)
 -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20074action=view) [edit]
 Experimental patch
 
 This patch fixes the problem for me. Testsuites are still running.
 

Why does the new testcase have

+/* { dg-require-effective-target lp64 } */

when the failure is shown with -m32 on x86-64?


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43280

[Bug tree-optimization/43280] [4.5 Regression] gcc4.5 -m32 -O2: misoptimizes sha256!

2010-03-08 Thread krebbel at gcc dot gnu dot org 


-- 

krebbel at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |krebbel at gcc dot gnu dot
   |dot org |org
 Status|NEW |ASSIGNED
   Last reconfirmed|2010-03-07 15:47:23 |2010-03-08 08:54:15
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43280

[Bug tree-optimization/43280] [4.5 Regression] gcc4.5 -m32 -O2: misoptimizes sha256!

2010-03-07 Thread rguenth at gcc dot gnu dot org 


--- Comment #3 from rguenth at gcc dot gnu dot org  2010-03-07 15:47 ---
Confirmed.  This is exposed by the new bswap pass.


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||krebbel1 at de dot ibm dot
   ||com
 Status|UNCONFIRMED |NEW
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2010-03-07 15:47:23
   date||
Summary|gcc4.5 -m32 -O2:|[4.5 Regression] gcc4.5 -m32
   |misoptimizes sha256!|-O2: misoptimizes sha256!
   Target Milestone|--- |4.5.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43280

[Bug tree-optimization/43280] [4.5 Regression] gcc4.5 -m32 -O2: misoptimizes sha256!

2010-03-07 Thread hjl dot tools at gmail dot com 


--- Comment #4 from hjl dot tools at gmail dot com  2010-03-07 16:56 ---
It is caused by revision 148848:

http://gcc.gnu.org/ml/gcc-cvs/2009-06/msg00831.html


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43280