public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/52086] New: [4.7 Regression] ICE caused by wrong peephole2 for QImode mem += reg followed by test
@ 2012-02-01 21:56 jakub at gcc dot gnu.org
  2012-02-01 21:56 ` [Bug target/52086] " jakub at gcc dot gnu.org
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: jakub at gcc dot gnu.org @ 2012-02-01 21:56 UTC (permalink / raw)
  To: gcc-bugs

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

             Bug #: 52086
           Summary: [4.7 Regression] ICE caused by wrong peephole2 for
                    QImode mem += reg followed by test
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Keywords: ice-on-valid-code
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: jakub@gcc.gnu.org
        ReportedBy: jakub@gcc.gnu.org
            Target: i?86-linux


/* { dg-do compile } */
/* { dg-options "-O2" } */
/* { dg-additional-options "-fpic" { target fpic } } */

struct S { char a; char b[100]; };
int bar (void);
int baz (int);

void
foo (struct S *x)
{
  if (bar () & 1)
    {
      char c = bar ();
      baz (4);
      x->a += c;
      while (x->a)
        x->b[c] = bar ();
    }
}

ICEs on i?86-linux (and x86_64-linux -m32) starting with:
http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=179646


^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug target/52086] [4.7 Regression] ICE caused by wrong peephole2 for QImode mem += reg followed by test
  2012-02-01 21:56 [Bug target/52086] New: [4.7 Regression] ICE caused by wrong peephole2 for QImode mem += reg followed by test jakub at gcc dot gnu.org
@ 2012-02-01 21:56 ` jakub at gcc dot gnu.org
  2012-02-01 22:01 ` jakub at gcc dot gnu.org
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: jakub at gcc dot gnu.org @ 2012-02-01 21:56 UTC (permalink / raw)
  To: gcc-bugs

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2012-02-01
      Known to work|                            |4.6.3
   Target Milestone|---                         |4.7.0
     Ever Confirmed|0                           |1
      Known to fail|                            |4.7.0


^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug target/52086] [4.7 Regression] ICE caused by wrong peephole2 for QImode mem += reg followed by test
  2012-02-01 21:56 [Bug target/52086] New: [4.7 Regression] ICE caused by wrong peephole2 for QImode mem += reg followed by test jakub at gcc dot gnu.org
  2012-02-01 21:56 ` [Bug target/52086] " jakub at gcc dot gnu.org
@ 2012-02-01 22:01 ` jakub at gcc dot gnu.org
  2012-02-02  9:05 ` jakub at gcc dot gnu.org
  2012-02-02  9:16 ` jakub at gcc dot gnu.org
  3 siblings, 0 replies; 5+ messages in thread
From: jakub at gcc dot gnu.org @ 2012-02-01 22:01 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-02-01 22:01:04 UTC ---
Created attachment 26548
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26548
gcc47-pr52086.patch

Untested fix.


^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug target/52086] [4.7 Regression] ICE caused by wrong peephole2 for QImode mem += reg followed by test
  2012-02-01 21:56 [Bug target/52086] New: [4.7 Regression] ICE caused by wrong peephole2 for QImode mem += reg followed by test jakub at gcc dot gnu.org
  2012-02-01 21:56 ` [Bug target/52086] " jakub at gcc dot gnu.org
  2012-02-01 22:01 ` jakub at gcc dot gnu.org
@ 2012-02-02  9:05 ` jakub at gcc dot gnu.org
  2012-02-02  9:16 ` jakub at gcc dot gnu.org
  3 siblings, 0 replies; 5+ messages in thread
From: jakub at gcc dot gnu.org @ 2012-02-02  9:05 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-02-02 09:05:03 UTC ---
Author: jakub
Date: Thu Feb  2 09:04:57 2012
New Revision: 183830

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=183830
Log:
    PR target/52086
    * config/i386/i386.md (*addqi_2 peephole with SImode addition): Check
    that operands[2] is either immediate, or q_regs_operand.

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

Added:
    trunk/gcc/testsuite/gcc.dg/pr52086.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/i386/i386.md
    trunk/gcc/testsuite/ChangeLog


^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug target/52086] [4.7 Regression] ICE caused by wrong peephole2 for QImode mem += reg followed by test
  2012-02-01 21:56 [Bug target/52086] New: [4.7 Regression] ICE caused by wrong peephole2 for QImode mem += reg followed by test jakub at gcc dot gnu.org
                   ` (2 preceding siblings ...)
  2012-02-02  9:05 ` jakub at gcc dot gnu.org
@ 2012-02-02  9:16 ` jakub at gcc dot gnu.org
  3 siblings, 0 replies; 5+ messages in thread
From: jakub at gcc dot gnu.org @ 2012-02-02  9:16 UTC (permalink / raw)
  To: gcc-bugs

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

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

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

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-02-02 09:15:55 UTC ---
Fixed.


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2012-02-02  9:16 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-02-01 21:56 [Bug target/52086] New: [4.7 Regression] ICE caused by wrong peephole2 for QImode mem += reg followed by test jakub at gcc dot gnu.org
2012-02-01 21:56 ` [Bug target/52086] " jakub at gcc dot gnu.org
2012-02-01 22:01 ` jakub at gcc dot gnu.org
2012-02-02  9:05 ` jakub at gcc dot gnu.org
2012-02-02  9:16 ` 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).