[Bug tree-optimization/60454] [4.7/4.8/4.9 Regression] Code mistakenly detected as doing bswap

2014-03-12 Thread jakub at gcc dot gnu.org 
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60454

--- Comment #6 from Jakub Jelinek jakub at gcc dot gnu.org ---
Author: jakub
Date: Wed Mar 12 10:15:29 2014
New Revision: 208509

URL: http://gcc.gnu.org/viewcvs?rev=208509root=gccview=rev
Log:
PR tree-optimization/60454
* tree-ssa-math-opts.c (find_bswap_1): Fix bswap detection.

* gcc.c-torture/execute/pr60454.c: New test.

Added:
trunk/gcc/testsuite/gcc.c-torture/execute/pr60454.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/tree-ssa-math-opts.c

[Bug tree-optimization/60454] [4.7/4.8/4.9 Regression] Code mistakenly detected as doing bswap

2014-03-07 Thread rguenth at gcc dot gnu.org 
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60454

Richard Biener rguenth at gcc dot gnu.org changed:

   What|Removed |Added

   Keywords||wrong-code
 Status|UNCONFIRMED |NEW
   Last reconfirmed||2014-03-07
 CC||wschmidt at gcc dot gnu.org
   Target Milestone|--- |4.7.4
Summary|Code mistakenly detected as |[4.7/4.8/4.9 Regression]
   |doing bswap |Code mistakenly detected as
   ||doing bswap
 Ever confirmed|0   |1

--- Comment #2 from Richard Biener rguenth at gcc dot gnu.org ---
Confirmed.  Regression from bswap pass introduction.

Bill, can you have a look?

[Bug tree-optimization/60454] [4.7/4.8/4.9 Regression] Code mistakenly detected as doing bswap

2014-03-07 Thread thomas.preudhomme at arm dot com 
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60454

--- Comment #3 from Thomas Preud'homme thomas.preudhomme at arm dot com ---
Created attachment 32299
  -- http://gcc.gnu.org/bugzilla/attachment.cgi?id=32299action=edit
Fix_bswap_detection

See in attachment for the patch I wrote to fix the issue. I'm welcoming any
comment on it.

[Bug tree-optimization/60454] [4.7/4.8/4.9 Regression] Code mistakenly detected as doing bswap

2014-03-07 Thread thomas.preudhomme at arm dot com 
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60454

Thomas Preud'homme thomas.preudhomme at arm dot com changed:

   What|Removed |Added

  Attachment #32299|0   |1
is obsolete||

--- Comment #4 from Thomas Preud'homme thomas.preudhomme at arm dot com ---
Created attachment 32300
  -- http://gcc.gnu.org/bugzilla/attachment.cgi?id=32300action=edit
Fix_bswap_detection_with_ChangeLog

Added ChangeLog entries to previous patch.

[Bug tree-optimization/60454] [4.7/4.8/4.9 Regression] Code mistakenly detected as doing bswap

2014-03-07 Thread thomas.preudhomme at arm dot com 
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60454

--- Comment #5 from Thomas Preud'homme thomas.preudhomme at arm dot com ---
I have posted the patch on gcc-patches mailing list. The discussion can be
followed from http://gcc.gnu.org/ml/gcc-patches/2014-03/msg00313.html.