* [Bug c/95108] [9/10/11 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292
2020-05-13 16:24 [Bug c/95108] New: [10/11 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292 gscfq@t-online.de
@ 2020-05-13 17:46 ` jakub at gcc dot gnu.org
2020-05-13 18:54 ` [Bug middle-end/95108] " jakub at gcc dot gnu.org
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: jakub at gcc dot gnu.org @ 2020-05-13 17:46 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95108
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Last reconfirmed| |2020-05-13
CC| |jakub at gcc dot gnu.org
Ever confirmed|0 |1
Target Milestone|--- |9.4
Status|UNCONFIRMED |NEW
Summary|[10/11 Regression] ICE in |[9/10/11 Regression] ICE in
|tree_fits_uhwi_p, at |tree_fits_uhwi_p, at
|tree.c:7292 |tree.c:7292
--- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Started with r9-6508-g33813f1d703c95d4fc87d16a17f6c834135ab209
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug middle-end/95108] [9/10/11 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292
2020-05-13 16:24 [Bug c/95108] New: [10/11 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292 gscfq@t-online.de
2020-05-13 17:46 ` [Bug c/95108] [9/10/11 " jakub at gcc dot gnu.org
@ 2020-05-13 18:54 ` jakub at gcc dot gnu.org
2020-05-14 6:16 ` rguenth at gcc dot gnu.org
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: jakub at gcc dot gnu.org @ 2020-05-13 18:54 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95108
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Assignee|unassigned at gcc dot gnu.org |jakub at gcc dot gnu.org
Status|NEW |ASSIGNED
--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Created attachment 48523
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=48523&action=edit
gcc11-pr95108.patch
Untested fix.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug middle-end/95108] [9/10/11 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292
2020-05-13 16:24 [Bug c/95108] New: [10/11 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292 gscfq@t-online.de
2020-05-13 17:46 ` [Bug c/95108] [9/10/11 " jakub at gcc dot gnu.org
2020-05-13 18:54 ` [Bug middle-end/95108] " jakub at gcc dot gnu.org
@ 2020-05-14 6:16 ` rguenth at gcc dot gnu.org
2020-05-14 7:53 ` cvs-commit at gcc dot gnu.org
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: rguenth at gcc dot gnu.org @ 2020-05-14 6:16 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95108
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Priority|P3 |P2
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug middle-end/95108] [9/10/11 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292
2020-05-13 16:24 [Bug c/95108] New: [10/11 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292 gscfq@t-online.de
` (2 preceding siblings ...)
2020-05-14 6:16 ` rguenth at gcc dot gnu.org
@ 2020-05-14 7:53 ` cvs-commit at gcc dot gnu.org
2020-05-14 8:07 ` [Bug middle-end/95108] [9/10 " jakub at gcc dot gnu.org
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2020-05-14 7:53 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95108
--- Comment #3 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Jakub Jelinek <jakub@gcc.gnu.org>:
https://gcc.gnu.org/g:d0fb9ffc1b8f3b86bbdf0e915cec2136141b329b
commit r11-381-gd0fb9ffc1b8f3b86bbdf0e915cec2136141b329b
Author: Jakub Jelinek <jakub@redhat.com>
Date: Thu May 14 09:51:05 2020 +0200
openmp: Fix placement of 2nd+ preparation statement for PHIs in simd clone
lowering [PR95108]
For normal stmts, preparation statements are inserted before the stmt, so
if we need multiple,
they are in the correct order, but for PHIs we emit them after labels in
the entry successor
bb, and we used to emit them in the reverse order that way.
2020-05-14 Jakub Jelinek <jakub@redhat.com>
PR middle-end/95108
* omp-simd-clone.c (struct modify_stmt_info): Add after_stmt
member.
(ipa_simd_modify_stmt_ops): For PHIs, only add before first stmt in
entry block if info->after_stmt is NULL, otherwise add after that
stmt
and update it after adding each stmt.
(ipa_simd_modify_function_body): Initialize info.after_stmt.
* gcc.dg/gomp/pr95108.c: New test.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug middle-end/95108] [9/10 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292
2020-05-13 16:24 [Bug c/95108] New: [10/11 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292 gscfq@t-online.de
` (3 preceding siblings ...)
2020-05-14 7:53 ` cvs-commit at gcc dot gnu.org
@ 2020-05-14 8:07 ` jakub at gcc dot gnu.org
2020-06-14 10:07 ` cvs-commit at gcc dot gnu.org
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: jakub at gcc dot gnu.org @ 2020-05-14 8:07 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95108
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|[9/10/11 Regression] ICE in |[9/10 Regression] ICE in
|tree_fits_uhwi_p, at |tree_fits_uhwi_p, at
|tree.c:7292 |tree.c:7292
--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Fixed on the trunk so far.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug middle-end/95108] [9/10 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292
2020-05-13 16:24 [Bug c/95108] New: [10/11 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292 gscfq@t-online.de
` (4 preceding siblings ...)
2020-05-14 8:07 ` [Bug middle-end/95108] [9/10 " jakub at gcc dot gnu.org
@ 2020-06-14 10:07 ` cvs-commit at gcc dot gnu.org
2020-06-14 11:47 ` [Bug middle-end/95108] [9 " jakub at gcc dot gnu.org
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2020-06-14 10:07 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95108
--- Comment #5 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-10 branch has been updated by Jakub Jelinek
<jakub@gcc.gnu.org>:
https://gcc.gnu.org/g:e09d626551fef9b5719073427170d11e97038c44
commit r10-8293-ge09d626551fef9b5719073427170d11e97038c44
Author: Jakub Jelinek <jakub@redhat.com>
Date: Thu May 14 09:51:05 2020 +0200
openmp: Fix placement of 2nd+ preparation statement for PHIs in simd clone
lowering [PR95108]
For normal stmts, preparation statements are inserted before the stmt, so
if we need multiple,
they are in the correct order, but for PHIs we emit them after labels in
the entry successor
bb, and we used to emit them in the reverse order that way.
2020-05-14 Jakub Jelinek <jakub@redhat.com>
PR middle-end/95108
* omp-simd-clone.c (struct modify_stmt_info): Add after_stmt
member.
(ipa_simd_modify_stmt_ops): For PHIs, only add before first stmt in
entry block if info->after_stmt is NULL, otherwise add after that
stmt
and update it after adding each stmt.
(ipa_simd_modify_function_body): Initialize info.after_stmt.
* gcc.dg/gomp/pr95108.c: New test.
(cherry picked from commit d0fb9ffc1b8f3b86bbdf0e915cec2136141b329b)
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug middle-end/95108] [9 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292
2020-05-13 16:24 [Bug c/95108] New: [10/11 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292 gscfq@t-online.de
` (5 preceding siblings ...)
2020-06-14 10:07 ` cvs-commit at gcc dot gnu.org
@ 2020-06-14 11:47 ` jakub at gcc dot gnu.org
2020-09-16 19:22 ` cvs-commit at gcc dot gnu.org
2020-09-17 17:51 ` jakub at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: jakub at gcc dot gnu.org @ 2020-06-14 11:47 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95108
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|[9/10 Regression] ICE in |[9 Regression] ICE in
|tree_fits_uhwi_p, at |tree_fits_uhwi_p, at
|tree.c:7292 |tree.c:7292
--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Fixed for 10.2+ too.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug middle-end/95108] [9 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292
2020-05-13 16:24 [Bug c/95108] New: [10/11 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292 gscfq@t-online.de
` (6 preceding siblings ...)
2020-06-14 11:47 ` [Bug middle-end/95108] [9 " jakub at gcc dot gnu.org
@ 2020-09-16 19:22 ` cvs-commit at gcc dot gnu.org
2020-09-17 17:51 ` jakub at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2020-09-16 19:22 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95108
--- Comment #7 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-9 branch has been updated by Jakub Jelinek
<jakub@gcc.gnu.org>:
https://gcc.gnu.org/g:b78ea59edbd354fed914d0f8eb78109db7316ce8
commit r9-8899-gb78ea59edbd354fed914d0f8eb78109db7316ce8
Author: Jakub Jelinek <jakub@redhat.com>
Date: Thu May 14 09:51:05 2020 +0200
openmp: Fix placement of 2nd+ preparation statement for PHIs in simd clone
lowering [PR95108]
For normal stmts, preparation statements are inserted before the stmt, so
if we need multiple,
they are in the correct order, but for PHIs we emit them after labels in
the entry successor
bb, and we used to emit them in the reverse order that way.
2020-05-14 Jakub Jelinek <jakub@redhat.com>
PR middle-end/95108
* omp-simd-clone.c (struct modify_stmt_info): Add after_stmt
member.
(ipa_simd_modify_stmt_ops): For PHIs, only add before first stmt in
entry block if info->after_stmt is NULL, otherwise add after that
stmt
and update it after adding each stmt.
(ipa_simd_modify_function_body): Initialize info.after_stmt.
* gcc.dg/gomp/pr95108.c: New test.
(cherry picked from commit d0fb9ffc1b8f3b86bbdf0e915cec2136141b329b)
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug middle-end/95108] [9 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292
2020-05-13 16:24 [Bug c/95108] New: [10/11 Regression] ICE in tree_fits_uhwi_p, at tree.c:7292 gscfq@t-online.de
` (7 preceding siblings ...)
2020-09-16 19:22 ` cvs-commit at gcc dot gnu.org
@ 2020-09-17 17:51 ` jakub at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: jakub at gcc dot gnu.org @ 2020-09-17 17:51 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95108
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution|--- |FIXED
--- Comment #8 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Fixed for 8.5 in r8-10505-g38484dac88b157151db984790be0582bf66adcfb and by the
above commit for 9.4+ too.
^ permalink raw reply [flat|nested] 10+ messages in thread