Re: [PATCH 2/2] rtl-ssa: Fix prev_any_insn [PR115785]

2024-07-12 Thread Jeff Law
On 7/9/24 9:28 AM, Richard Sandiford wrote: Bit of a brown paper bag issue, but: due to the representation of the insn chain, insn_info::prev_any_insn would sometimes skip over instructions. This led to an invalid update in the PR when adding and removing instructions. I think one of the rea

[PATCH 2/2] rtl-ssa: Fix prev_any_insn [PR115785]

2024-07-09 Thread Richard Sandiford
Bit of a brown paper bag issue, but: due to the representation of the insn chain, insn_info::prev_any_insn would sometimes skip over instructions. This led to an invalid update in the PR when adding and removing instructions. I think one of the reasons I failed to spot this when checking the code