public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug testsuite/46249] New: gcc.dg/tree-ssa/ssa-ccp-32.c: dubious test
@ 2010-10-31 14:45 nikai at nikai dot net
  2010-11-01 22:49 ` [Bug testsuite/46249] " rguenth at gcc dot gnu.org
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: nikai at nikai dot net @ 2010-10-31 14:45 UTC (permalink / raw)
  To: gcc-bugs

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

           Summary: gcc.dg/tree-ssa/ssa-ccp-32.c: dubious test
           Product: gcc
           Version: unknown
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: testsuite
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: nikai@nikai.net


Operator precedence must cause the below test to always pass,
my guess is this wasn't originally intended.

I hope I'm not missing the point of this test?

Best regards,
Nicolas Kaiser
---
--- a/gcc/testsuite/gcc.dg/tree-ssa/ssa-ccp-32.c    2010-10-29
11:08:42.713609198 +0200
+++ b/gcc/testsuite/gcc.dg/tree-ssa/ssa-ccp-32.c    2010-10-31
15:08:01.479911011 +0100
@@ -41,7 +41,7 @@ test2 (int a)
   if (a & 0xff)
     link_error ();
   a = a - 1;
-  if (a & 0xff != 0xff)
+  if ((a & 0xff) != 0xff)
     link_error ();
   return a;
 }


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

* [Bug testsuite/46249] gcc.dg/tree-ssa/ssa-ccp-32.c: dubious test
  2010-10-31 14:45 [Bug testsuite/46249] New: gcc.dg/tree-ssa/ssa-ccp-32.c: dubious test nikai at nikai dot net
@ 2010-11-01 22:49 ` rguenth at gcc dot gnu.org
  2010-11-02 14:35 ` rguenth at gcc dot gnu.org
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: rguenth at gcc dot gnu.org @ 2010-11-01 22:49 UTC (permalink / raw)
  To: gcc-bugs

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2010.11.01 22:48:50
         AssignedTo|unassigned at gcc dot       |rguenth at gcc dot gnu.org
                   |gnu.org                     |
     Ever Confirmed|0                           |1

--- Comment #1 from Richard Guenther <rguenth at gcc dot gnu.org> 2010-11-01 22:48:50 UTC ---
Oops.  Mine.


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

* [Bug testsuite/46249] gcc.dg/tree-ssa/ssa-ccp-32.c: dubious test
  2010-10-31 14:45 [Bug testsuite/46249] New: gcc.dg/tree-ssa/ssa-ccp-32.c: dubious test nikai at nikai dot net
  2010-11-01 22:49 ` [Bug testsuite/46249] " rguenth at gcc dot gnu.org
@ 2010-11-02 14:35 ` rguenth at gcc dot gnu.org
  2010-11-02 14:36 ` rguenth at gcc dot gnu.org
  2010-11-02 17:45 ` nikai at nikai dot net
  3 siblings, 0 replies; 5+ messages in thread
From: rguenth at gcc dot gnu.org @ 2010-11-02 14:35 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #2 from Richard Guenther <rguenth at gcc dot gnu.org> 2010-11-02 14:35:00 UTC ---
Author: rguenth
Date: Tue Nov  2 14:34:53 2010
New Revision: 166183

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=166183
Log:
2010-11-02  Richard Guenther  <rguenther@suse.de>

    PR testsuite/46249
    * gcc.dg/tree-ssa/ssa-ccp-32.c: Fix operator precedence.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.dg/tree-ssa/ssa-ccp-32.c


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

* [Bug testsuite/46249] gcc.dg/tree-ssa/ssa-ccp-32.c: dubious test
  2010-10-31 14:45 [Bug testsuite/46249] New: gcc.dg/tree-ssa/ssa-ccp-32.c: dubious test nikai at nikai dot net
  2010-11-01 22:49 ` [Bug testsuite/46249] " rguenth at gcc dot gnu.org
  2010-11-02 14:35 ` rguenth at gcc dot gnu.org
@ 2010-11-02 14:36 ` rguenth at gcc dot gnu.org
  2010-11-02 17:45 ` nikai at nikai dot net
  3 siblings, 0 replies; 5+ messages in thread
From: rguenth at gcc dot gnu.org @ 2010-11-02 14:36 UTC (permalink / raw)
  To: gcc-bugs

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.6.0

--- Comment #3 from Richard Guenther <rguenth at gcc dot gnu.org> 2010-11-02 14:36:45 UTC ---
Fixed.


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

* [Bug testsuite/46249] gcc.dg/tree-ssa/ssa-ccp-32.c: dubious test
  2010-10-31 14:45 [Bug testsuite/46249] New: gcc.dg/tree-ssa/ssa-ccp-32.c: dubious test nikai at nikai dot net
                   ` (2 preceding siblings ...)
  2010-11-02 14:36 ` rguenth at gcc dot gnu.org
@ 2010-11-02 17:45 ` nikai at nikai dot net
  3 siblings, 0 replies; 5+ messages in thread
From: nikai at nikai dot net @ 2010-11-02 17:45 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #4 from Nicolas Kaiser <nikai at nikai dot net> 2010-11-02 17:44:56 UTC ---
I noticed some similar occurrences and filed them as bug 46274.


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

end of thread, other threads:[~2010-11-02 17:45 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-10-31 14:45 [Bug testsuite/46249] New: gcc.dg/tree-ssa/ssa-ccp-32.c: dubious test nikai at nikai dot net
2010-11-01 22:49 ` [Bug testsuite/46249] " rguenth at gcc dot gnu.org
2010-11-02 14:35 ` rguenth at gcc dot gnu.org
2010-11-02 14:36 ` rguenth at gcc dot gnu.org
2010-11-02 17:45 ` nikai at nikai dot net

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).