Re: [PATCH] reassoc: Fix -fcompare-debug bug in reassociate_bb [PR94329]

2020-03-28 Thread Richard Biener
On March 28, 2020 1:50:59 AM GMT+01:00, Jakub Jelinek wrote: >Hi! > >The following testcase FAILs with -fcompare-debug, because >reassociate_bb >mishandles the case when the last stmt in a bb has zero uses. In that >case >reassoc_remove_stmt (like gsi_remove) moves the iterator to the next >stmt,

[PATCH] reassoc: Fix -fcompare-debug bug in reassociate_bb [PR94329]

2020-03-27 Thread Jakub Jelinek via Gcc-patches
Hi! The following testcase FAILs with -fcompare-debug, because reassociate_bb mishandles the case when the last stmt in a bb has zero uses. In that case reassoc_remove_stmt (like gsi_remove) moves the iterator to the next stmt, i.e. gsi_end_p is true, which means the code sets the iterator back t