[Bug middle-end/81768] [8 Regression] error: control flow in the middle of basic block

2017-10-11 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81768

Jakub Jelinek  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #12 from Jakub Jelinek  ---
Fixed.

[Bug middle-end/81768] [8 Regression] error: control flow in the middle of basic block

2017-09-15 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81768

--- Comment #11 from Jakub Jelinek  ---
Author: jakub
Date: Fri Sep 15 21:39:36 2017
New Revision: 252861

URL: https://gcc.gnu.org/viewcvs?rev=252861=gcc=rev
Log:
Backported from mainline
2017-09-05  Jakub Jelinek  

PR middle-end/81768
* omp-low.c (lower_omp_for): Recompute tree invariant if
gimple_omp_for_initial/final is ADDR_EXPR.

* gcc.dg/gomp/pr81768-2.c: New test.

Added:
branches/gcc-6-branch/gcc/testsuite/gcc.dg/gomp/pr81768-2.c
Modified:
branches/gcc-6-branch/gcc/ChangeLog
branches/gcc-6-branch/gcc/omp-low.c
branches/gcc-6-branch/gcc/testsuite/ChangeLog

[Bug middle-end/81768] [8 Regression] error: control flow in the middle of basic block

2017-09-15 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81768

--- Comment #10 from Jakub Jelinek  ---
Author: jakub
Date: Fri Sep 15 21:38:43 2017
New Revision: 252860

URL: https://gcc.gnu.org/viewcvs?rev=252860=gcc=rev
Log:
Backported from mainline
2017-09-05  Jakub Jelinek  

PR middle-end/81768
* omp-low.c (expand_omp_simd): Force second operands of COND_EXPR
into gimple val before gimplification fo the COND_EXPR.

* gcc.dg/gomp/pr81768-1.c: New test.

Added:
branches/gcc-6-branch/gcc/testsuite/gcc.dg/gomp/pr81768-1.c
Modified:
branches/gcc-6-branch/gcc/ChangeLog
branches/gcc-6-branch/gcc/omp-low.c
branches/gcc-6-branch/gcc/testsuite/ChangeLog

[Bug middle-end/81768] [8 Regression] error: control flow in the middle of basic block

2017-09-07 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81768

--- Comment #9 from Jakub Jelinek  ---
Author: jakub
Date: Thu Sep  7 20:41:42 2017
New Revision: 251857

URL: https://gcc.gnu.org/viewcvs?rev=251857=gcc=rev
Log:
Backported from mainline
2017-09-05  Jakub Jelinek  

PR middle-end/81768
* omp-low.c (lower_omp_for): Recompute tree invariant if
gimple_omp_for_initial/final is ADDR_EXPR.

* gcc.dg/gomp/pr81768-2.c: New test.

Added:
branches/gcc-7-branch/gcc/testsuite/gcc.dg/gomp/pr81768-2.c
Modified:
branches/gcc-7-branch/gcc/ChangeLog
branches/gcc-7-branch/gcc/omp-low.c
branches/gcc-7-branch/gcc/testsuite/ChangeLog

[Bug middle-end/81768] [8 Regression] error: control flow in the middle of basic block

2017-09-07 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81768

--- Comment #8 from Jakub Jelinek  ---
Author: jakub
Date: Thu Sep  7 20:40:06 2017
New Revision: 251856

URL: https://gcc.gnu.org/viewcvs?rev=251856=gcc=rev
Log:
Backported from mainline
2017-09-05  Jakub Jelinek  

PR middle-end/81768
* omp-expand.c (expand_omp_simd): Force second operands of COND_EXPR
into gimple val before gimplification fo the COND_EXPR.

* gcc.dg/gomp/pr81768-1.c: New test.

Added:
branches/gcc-7-branch/gcc/testsuite/gcc.dg/gomp/pr81768-1.c
Modified:
branches/gcc-7-branch/gcc/ChangeLog
branches/gcc-7-branch/gcc/omp-expand.c
branches/gcc-7-branch/gcc/testsuite/ChangeLog

[Bug middle-end/81768] [8 Regression] error: control flow in the middle of basic block

2017-09-05 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81768

--- Comment #7 from Jakub Jelinek  ---
Author: jakub
Date: Tue Sep  5 21:32:35 2017
New Revision: 251742

URL: https://gcc.gnu.org/viewcvs?rev=251742=gcc=rev
Log:
PR middle-end/81768
* omp-low.c (lower_omp_for): Recompute tree invariant if
gimple_omp_for_initial/final is ADDR_EXPR.

* gcc.dg/gomp/pr81768-2.c: New test.

Added:
trunk/gcc/testsuite/gcc.dg/gomp/pr81768-2.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/omp-low.c
trunk/gcc/testsuite/ChangeLog

[Bug middle-end/81768] [8 Regression] error: control flow in the middle of basic block

2017-09-05 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81768

--- Comment #6 from Jakub Jelinek  ---
Author: jakub
Date: Tue Sep  5 21:31:39 2017
New Revision: 251741

URL: https://gcc.gnu.org/viewcvs?rev=251741=gcc=rev
Log:
PR middle-end/81768
* omp-expand.c (expand_omp_simd): Force second operands of COND_EXPR
into gimple val before gimplification fo the COND_EXPR.

* gcc.dg/gomp/pr81768-1.c: New test.

Added:
trunk/gcc/testsuite/gcc.dg/gomp/pr81768-1.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/omp-expand.c
trunk/gcc/testsuite/ChangeLog

[Bug middle-end/81768] [8 Regression] error: control flow in the middle of basic block

2017-09-05 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81768

--- Comment #5 from Jakub Jelinek  ---
Created attachment 42126
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=42126=edit
gcc8-pr81768-2.patch

And an untested fix for the other reported bug.

[Bug middle-end/81768] [8 Regression] error: control flow in the middle of basic block

2017-09-05 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81768

--- Comment #4 from Jakub Jelinek  ---
Created attachment 42123
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=42123=edit
gcc8-pr81768.patch

Untested fix for the simd expansion bug.

[Bug middle-end/81768] [8 Regression] error: control flow in the middle of basic block

2017-09-05 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81768

Jakub Jelinek  changed:

   What|Removed |Added

   Keywords|ice-on-invalid-code |ice-on-valid-code
 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2017-09-05
  Component|libgomp |middle-end
 Ever confirmed|0   |1

--- Comment #3 from Jakub Jelinek  ---
Why would it be ice-on-invalid-code?  If there is no #pragma omp declare
target, then when the global vars are referenced in the target region, they are
implicitly mapped - i.e. map(tofrom:b).