Re: [PATCH] lra: Avoid cycling on certain subreg reloads [PR96796]

2021-04-23 Thread Vladimir Makarov via Gcc-patches
On 2021-04-23 12:13 p.m., Richard Sandiford wrote: This is a backport of the PR96796 fix to GCC 10 and GCC 9. The original trunk patch was: https://gcc.gnu.org/pipermail/gcc-patches/2020-August/552878.html reviewed here:

[PATCH] lra: Avoid cycling on certain subreg reloads [PR96796]

2021-04-23 Thread Richard Sandiford via Gcc-patches
This is a backport of the PR96796 fix to GCC 10 and GCC 9. The original trunk patch was: https://gcc.gnu.org/pipermail/gcc-patches/2020-August/552878.html reviewed here: https://gcc.gnu.org/pipermail/gcc-patches/2020-September/553308.html I'm not aware of any fallout since then.

Re: [PATCH] lra: Avoid cycling on certain subreg reloads [PR96796]

2020-09-04 Thread Vladimir Makarov via Gcc-patches
Richard, thank you for working on this issue and for as usually detailed explanation of the problem. On 2020-08-28 9:52 a.m., Richard Sandiford wrote: ... The patch is quite aggressive in that it does this for all reload pseudos in all reload instructions. I wondered about reusing the

[PATCH] lra: Avoid cycling on certain subreg reloads [PR96796]

2020-08-28 Thread Richard Sandiford
This PR is about LRA cycling for a reload of the form: Changing pseudo 196 in operand 1 of insn 103 on equiv [r105:DI*0x8+r140:DI] Creating newreg=287, assigning class ALL_REGS to slow/invalid mem r287