[Bug middle-end/81832] [8 Regression] ICE in expand_LOOP_DIST_ALIAS, at internal-fn.c:2273

2017-10-09 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81832

amker at gcc dot gnu.org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||amker at gcc dot gnu.org
 Resolution|--- |FIXED

--- Comment #4 from amker at gcc dot gnu.org ---
fixed.

[Bug middle-end/81832] [8 Regression] ICE in expand_LOOP_DIST_ALIAS, at internal-fn.c:2273

2017-09-13 Thread aldyh at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81832

--- Comment #3 from Aldy Hernandez  ---
Author: aldyh
Date: Wed Sep 13 17:01:56 2017
New Revision: 252444

URL: https://gcc.gnu.org/viewcvs?rev=252444=gcc=rev
Log:
PR tree-optimization/81832
* tree-ssa-loop-ch.c (should_duplicate_loop_header_p): Don't
copy loop header which has IFN_LOOP_DIST_ALIAS call.

gcc/testsuite
* gcc.dg/tree-ssa/pr81832.c: New test.

Added:
branches/range-gen2/gcc/testsuite/gcc.dg/tree-ssa/pr81832.c
Modified:
branches/range-gen2/gcc/ChangeLog
branches/range-gen2/gcc/testsuite/ChangeLog
branches/range-gen2/gcc/tree-ssa-loop-ch.c

[Bug middle-end/81832] [8 Regression] ICE in expand_LOOP_DIST_ALIAS, at internal-fn.c:2273

2017-08-16 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81832

--- Comment #2 from amker at gcc dot gnu.org ---
Author: amker
Date: Wed Aug 16 15:02:03 2017
New Revision: 251123

URL: https://gcc.gnu.org/viewcvs?rev=251123=gcc=rev
Log:
PR tree-optimization/81832
* tree-ssa-loop-ch.c (should_duplicate_loop_header_p): Don't
copy loop header which has IFN_LOOP_DIST_ALIAS call.

gcc/testsuite
* gcc.dg/tree-ssa/pr81832.c: New test.

Added:
trunk/gcc/testsuite/gcc.dg/tree-ssa/pr81832.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/tree-ssa-loop-ch.c

[Bug middle-end/81832] [8 Regression] ICE in expand_LOOP_DIST_ALIAS, at internal-fn.c:2273

2017-08-14 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81832

--- Comment #4 from amker at gcc dot gnu.org ---
Testing a patch.

[Bug middle-end/81832] [8 Regression] ICE in expand_LOOP_DIST_ALIAS, at internal-fn.c:2273

2017-08-14 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81832

--- Comment #3 from Martin Liška  ---
> I think this is a general and latent problem with the interaction between
> the copy-header pass, and the loop distribution pass. Tracing back further I
> see this start with r249994 .

Yes, sorry for not tracking precisely, r249994 is the first one ICEing.

[Bug middle-end/81832] [8 Regression] ICE in expand_LOOP_DIST_ALIAS, at internal-fn.c:2273

2017-08-14 Thread jgreenhalgh at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81832

James Greenhalgh  changed:

   What|Removed |Added

 CC||amker at gcc dot gnu.org

--- Comment #2 from James Greenhalgh  ---
(In reply to Martin Liška from comment #1)
> Confirmed, started with r250619.

Interesting. That commit seems unlikely to have broken anything (if it does,
the bug would be latent and would have been possible to trigger using the
revision prior). My bisect points to r250959 , which seems much more likely,
given the backtrace.

What I imagine you've done with your bisect is continued back through the
revisions with -ftree-loop-distribute set, that does get you to r250619, but as
this is also really just a change to default "options", you should continue
going back with -ftree-vectorize to find the real culprit. For example, r250617
will also ICE with -O3 -ftree-loop-distribute -ftree-vectorize .

I think this is a general and latent problem with the interaction between the
copy-header pass, and the loop distribution pass. Tracing back further I see
this start with r249994 .