Re: [PR106746] drop cselib addr lookup in debug insn mem

2023-01-15 Thread Richard Biener via Gcc-patches
On Sat, Jan 14, 2023 at 12:26 PM Alexandre Oliva via Gcc-patches wrote: > > > The testcase used to get scheduled differently depending on the > presence of debug insns with MEMs. It's not clear to me why those > MEMs affected scheduling, but the cselib pre-canonicalization of the > MEM address is

[PR106746] drop cselib addr lookup in debug insn mem

2023-01-14 Thread Alexandre Oliva via Gcc-patches
The testcase used to get scheduled differently depending on the presence of debug insns with MEMs. It's not clear to me why those MEMs affected scheduling, but the cselib pre-canonicalization of the MEM address is not used at all when analyzing debug insns, so the memory allocation and lookup ar