Re: [PATCH] store-merging: Fix up a -fcompare-debug bug in get_status_for_store_merging [PR104263]

2022-01-28 Thread Richard Biener via Gcc-patches
> Am 28.01.2022 um 18:36 schrieb Jeff Law : > >  > >> On 1/28/2022 9:31 AM, Jakub Jelinek via Gcc-patches wrote: >> Hi! >> >> As mentioned in the PRthe following testcase fails, because the last >> stmt of a bb with -g is a debug stmt and get_status_for_store_merging >> uses

Re: [PATCH] store-merging: Fix up a -fcompare-debug bug in get_status_for_store_merging [PR104263]

2022-01-28 Thread Jeff Law via Gcc-patches
On 1/28/2022 9:31 AM, Jakub Jelinek via Gcc-patches wrote: Hi! As mentioned in the PRthe following testcase fails, because the last stmt of a bb with -g is a debug stmt and get_status_for_store_merging uses gimple_seq_last_stmt (bb_seq (bb)) when testing if it is valid for store merging.

[PATCH] store-merging: Fix up a -fcompare-debug bug in get_status_for_store_merging [PR104263]

2022-01-28 Thread Jakub Jelinek via Gcc-patches
Hi! As mentioned in the PRthe following testcase fails, because the last stmt of a bb with -g is a debug stmt and get_status_for_store_merging uses gimple_seq_last_stmt (bb_seq (bb)) when testing if it is valid for store merging. The debug stmt isn't valid, while a stmt at that position with -g0