[Bug tree-optimization/49498] [5/6/7 Regression]: gcc.dg/uninit-pred-8_b.c bogus warning (predicate analysis bugs)

2017-03-31 Thread law at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49498

--- Comment #30 from Jeffrey A. Law  ---
Author: law
Date: Fri Mar 31 15:26:18 2017
New Revision: 246618

URL: https://gcc.gnu.org/viewcvs?rev=246618=gcc=rev
Log:
PR tree-optimization/49498
* gcc.dg/uninit-pred-8_b.c: Reenable DOM.

Modified:
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/gcc.dg/uninit-pred-8_b.c

[Bug tree-optimization/49498] [5/6/7 Regression]: gcc.dg/uninit-pred-8_b.c bogus warning (predicate analysis bugs)

2017-03-31 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49498

--- Comment #29 from Jeffrey A. Law  ---
Yes, I checked cris-elf.  I've got the patch to reenable DOM for that test in
my local tree.

[Bug tree-optimization/49498] [5/6/7 Regression]: gcc.dg/uninit-pred-8_b.c bogus warning (predicate analysis bugs)

2017-03-31 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49498

--- Comment #28 from Richard Biener  ---
(In reply to Jeffrey A. Law from comment #27)
> Must. Read. dg-comments in testcase. More. Carefully.
> 
> 
> This was actually fixed a couple years ago with some pass ordering changes. 
> The testsuite hack mentioned in c#12 is no longer needed.

Hmm, note the report states it worked on x86_64/i586/arm anyway and only
some other archs were broken.

Did you verify any of those?  Iff it's really fixed we should remove the
testcase workaround (enable DOM again).

A quick check on avr doesn't reproduce the issue when re-enabling DOM.

[Bug tree-optimization/49498] [5/6/7 Regression]: gcc.dg/uninit-pred-8_b.c bogus warning (predicate analysis bugs)

2017-03-31 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49498

Jeffrey A. Law  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #27 from Jeffrey A. Law  ---
Must. Read. dg-comments in testcase. More. Carefully.


This was actually fixed a couple years ago with some pass ordering changes. 
The testsuite hack mentioned in c#12 is no longer needed.

[Bug tree-optimization/49498] [5/6/7 Regression]: gcc.dg/uninit-pred-8_b.c bogus warning (predicate analysis bugs)

2017-03-30 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49498

--- Comment #26 from Jeffrey A. Law  ---
So I had hoped this old BZ would be fixed by the changes for 71437.  Sadly,
that is not the case.

*But* the WIP for for 78496 does happen to fix this BZ.  In fact, we only need
the hunks from 78496 which are inexpensive.  So I may carve those out and push
them forward as a fix for this BZ as well as an improvement for 78496.