public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug tree-optimization/50161] New: [4.7 Regression] wrong code with -fno-tree-ter and __builtin_popcountl
@ 2011-08-23  7:54 zsojka at seznam dot cz
  2011-08-23  8:01 ` [Bug middle-end/50161] " jakub at gcc dot gnu.org
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: zsojka at seznam dot cz @ 2011-08-23  7:54 UTC (permalink / raw)
  To: gcc-bugs

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

             Bug #: 50161
           Summary: [4.7 Regression] wrong code with -fno-tree-ter and
                    __builtin_popcountl
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: tree-optimization
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: zsojka@seznam.cz
              Host: x86_64-pc-linux-gnu
            Target: x86_64-pc-linux-gnu


Created attachment 25081
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25081
reduced testcase

Output:
$ gcc -O2 -fno-tree-ter -funroll-loops testcase.c
$ ./a.out 
Aborted

Tested revisions:
r177969 - fail
r173150 - OK
4.6.1 - OK


^ permalink raw reply	[flat|nested] 5+ messages in thread
* [Bug middle-end/50161] [4.7 Regression] wrong code with -fno-tree-ter and __builtin_popcountl
  2011-08-23  7:54 [Bug tree-optimization/50161] New: [4.7 Regression] wrong code with -fno-tree-ter and __builtin_popcountl zsojka at seznam dot cz
@ 2011-08-23  8:01 ` jakub at gcc dot gnu.org
  2011-08-23  8:41 ` jakub at gcc dot gnu.org
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: jakub at gcc dot gnu.org @ 2011-08-23  8:01 UTC (permalink / raw)
  To: gcc-bugs

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2011-08-23
                 CC|                            |jakub at gcc dot gnu.org
   Target Milestone|---                         |4.7.0
     Ever Confirmed|0                           |1

--- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-08-23 07:53:48 UTC ---
Caused by http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=175261 , looking
into it.


^ permalink raw reply	[flat|nested] 5+ messages in thread
* [Bug middle-end/50161] [4.7 Regression] wrong code with -fno-tree-ter and __builtin_popcountl
  2011-08-23  7:54 [Bug tree-optimization/50161] New: [4.7 Regression] wrong code with -fno-tree-ter and __builtin_popcountl zsojka at seznam dot cz
  2011-08-23  8:01 ` [Bug middle-end/50161] " jakub at gcc dot gnu.org
@ 2011-08-23  8:41 ` jakub at gcc dot gnu.org
  2011-08-23 15:53 ` jakub at gcc dot gnu.org
  2011-08-23 15:57 ` jakub at gcc dot gnu.org
  3 siblings, 0 replies; 5+ messages in thread
From: jakub at gcc dot gnu.org @ 2011-08-23  8:41 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-08-23 08:21:25 UTC ---
The simplify_const_unary_operation changes from that patch look totally wrong.


^ permalink raw reply	[flat|nested] 5+ messages in thread
* [Bug middle-end/50161] [4.7 Regression] wrong code with -fno-tree-ter and __builtin_popcountl
  2011-08-23  7:54 [Bug tree-optimization/50161] New: [4.7 Regression] wrong code with -fno-tree-ter and __builtin_popcountl zsojka at seznam dot cz
  2011-08-23  8:01 ` [Bug middle-end/50161] " jakub at gcc dot gnu.org
  2011-08-23  8:41 ` jakub at gcc dot gnu.org
@ 2011-08-23 15:53 ` jakub at gcc dot gnu.org
  2011-08-23 15:57 ` jakub at gcc dot gnu.org
  3 siblings, 0 replies; 5+ messages in thread
From: jakub at gcc dot gnu.org @ 2011-08-23 15:53 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-08-23 15:51:55 UTC ---
Author: jakub
Date: Tue Aug 23 15:51:45 2011
New Revision: 177991

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=177991
Log:
    PR middle-end/50161
    * simplify-rtx.c (simplify_const_unary_operation): If
    op is CONST_INT, don't look at op_mode, but use instead
    mode.
    * optabs.c (add_equal_note): For FFS, CLZ, CTZ,
    CLRSB, POPCOUNT, PARITY and BSWAP use operand mode for
    operation and TRUNCATE/ZERO_EXTEND if needed.
    * doc/rtl.texi (ffs, clrsb, clz, ctz, popcount, parity, bswap):
    Document that operand mode must be same as operation mode,
    or VOIDmode.
    * config/avr/avr.md (paritysi2, *parityqihi2.libgcc,
    *paritysihi2.libgcc, popcountsi2, *popcountsi2.libgcc,
    *popcountqihi2.libgcc, clzsi2, *clzsihi2.libgcc, ctzsi2,
    *ctzsihi2.libgcc, ffssi2, *ffssihi2.libgcc): For unary ops
    use the mode of operand for the operation and add truncate
    or zero_extend around if needed.
    * config/c6x/c6x.md (ctzdi2): Likewise.
    * config/bfin/bfin.md (clrsbsi2, signbitssi2): Likewise.

    * gcc.dg/pr50161.c: New test.

Added:
    trunk/gcc/testsuite/gcc.dg/pr50161.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/avr/avr.md
    trunk/gcc/config/bfin/bfin.md
    trunk/gcc/config/c6x/c6x.md
    trunk/gcc/doc/rtl.texi
    trunk/gcc/optabs.c
    trunk/gcc/simplify-rtx.c
    trunk/gcc/testsuite/ChangeLog


^ permalink raw reply	[flat|nested] 5+ messages in thread
* [Bug middle-end/50161] [4.7 Regression] wrong code with -fno-tree-ter and __builtin_popcountl
  2011-08-23  7:54 [Bug tree-optimization/50161] New: [4.7 Regression] wrong code with -fno-tree-ter and __builtin_popcountl zsojka at seznam dot cz
                   ` (2 preceding siblings ...)
  2011-08-23 15:53 ` jakub at gcc dot gnu.org
@ 2011-08-23 15:57 ` jakub at gcc dot gnu.org
  3 siblings, 0 replies; 5+ messages in thread
From: jakub at gcc dot gnu.org @ 2011-08-23 15:57 UTC (permalink / raw)
  To: gcc-bugs

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-08-23 15:55:02 UTC ---
Fixed.


^ permalink raw reply	[flat|nested] 5+ messages in thread
end of thread, other threads:[~2011-08-23 15:55 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-08-23  7:54 [Bug tree-optimization/50161] New: [4.7 Regression] wrong code with -fno-tree-ter and __builtin_popcountl zsojka at seznam dot cz
2011-08-23  8:01 ` [Bug middle-end/50161] " jakub at gcc dot gnu.org
2011-08-23  8:41 ` jakub at gcc dot gnu.org
2011-08-23 15:53 ` jakub at gcc dot gnu.org
2011-08-23 15:57 ` jakub at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).