https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88296

--- Comment #5 from Vladimir Makarov <vmakarov at gcc dot gnu.org> ---
(In reply to Jakub Jelinek from comment #3)
> for Vlad the question
> is just whether r266862 is a real fix or just made it latent.  Given that
> both are IRA costs changes, I assume it is a real fix.

I've checked the generated code difference.  The recent code uses more accurate
register classes and as the result avoid LRA cycling.  So I believe the patch
is a real fix for the PR.  But I think there is still possibility that the 1st
insn scheduler can create a situation when RA (as the old reload) can fail
because of existing constraints in hard register splitting in LRA.  This
problem is less severe every year but still exists and honestly I don't know
when it will be fully solved.

As for this PR, I think it should be closed.

Reply via email to