Re: [PATCH] gimple: sccopy: Don't increment i after vec::unordered_remove()

2025-04-05 Thread Richard Biener
On Thu, 20 Mar 2025, Filip Kastl wrote: > Hi, > > Ok to push if bootstrap and regtest (on x86_64 linux) succeeds? OK. > Thanks, > Filip Kastl > > > -- 8< -- > > > I increment the index variable in a loop even when I do > vec::unordered_remove() which causes the vector traversal to miss some

Re: [PATCH] gimple: sccopy: Don't increment i after vec::unordered_remove()

2025-04-04 Thread Filip Kastl
On Thu 2025-03-20 13:31:38, Richard Biener wrote: > On Thu, 20 Mar 2025, Filip Kastl wrote: > > > Hi, > > > > Ok to push if bootstrap and regtest (on x86_64 linux) succeeds? > > OK. > And I again almost forgot: Can I backport this to releases/gcc-14 branch? Filip > > Thanks, > > Filip Kastl

Re: [PATCH] gimple: sccopy: Don't increment i after vec::unordered_remove()

2025-03-21 Thread Richard Biener
On Fri, 21 Mar 2025, Filip Kastl wrote: > On Thu 2025-03-20 13:31:38, Richard Biener wrote: > > On Thu, 20 Mar 2025, Filip Kastl wrote: > > > > > Hi, > > > > > > Ok to push if bootstrap and regtest (on x86_64 linux) succeeds? > > > > OK. > > > > And I again almost forgot: Can I backport this

[PATCH] gimple: sccopy: Don't increment i after vec::unordered_remove()

2025-03-20 Thread Filip Kastl
Hi, Ok to push if bootstrap and regtest (on x86_64 linux) succeeds? Thanks, Filip Kastl -- 8< -- I increment the index variable in a loop even when I do vec::unordered_remove() which causes the vector traversal to miss some elements. Mikael notified me of this mistake I made in my last patch