[Bug target/118358] [15/16 Regression] LRA hangs on xstormy16

2025-11-21 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=118358

--- Comment #10 from Andrew Pinski  ---
(In reply to Jeffrey A. Law from comment #9)
> Given we've only seen this on xstormy, I don't think this is worth
> backporting.  So closing as resolved.  This appears to fix pr122717 as well.
> 
> THanks Vlad!

yes thanks vlad for fixing this. one less regression due to my patches to worry
about.

[Bug target/118358] [15/16 Regression] LRA hangs on xstormy16

2025-11-21 Thread law at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=118358

Jeffrey A. Law  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED

--- Comment #9 from Jeffrey A. Law  ---
Given we've only seen this on xstormy, I don't think this is worth backporting.
 So closing as resolved.  This appears to fix pr122717 as well.

THanks Vlad!

[Bug target/118358] [15/16 Regression] LRA hangs on xstormy16

2025-11-21 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=118358

--- Comment #8 from GCC Commits  ---
The master branch has been updated by Vladimir Makarov :

https://gcc.gnu.org/g:b7f137e56867dd3aa1f05f3409f146391dec131c

commit r16-5499-gb7f137e56867dd3aa1f05f3409f146391dec131c
Author: Vladimir N. Makarov 
Date:   Fri Nov 21 11:49:31 2025 -0500

[PR118358, LRA]: Decrease pressure after issuing input reload insns

LRA can generate sequence of reload insns for one input operand using
intermediate pseudos.  Register pressure when reload insn for another
input operand is placed before the sequence is more than when the
reload insn is placed after the sequence.  The problem report reveals
a case when several such sequences increase the pressure for input
reload insns beyond available registers and as a consequence this
results in LRA cycling.

gcc/ChangeLog:

PR target/118358
* lra-constraints.cc (curr_insn_transform): Move insn reloading
constant into a register right before insn using it.

gcc/testsuite/ChangeLog:

PR target/118358
* gcc.target/xstormy16/pr118358.c: New.

[Bug target/118358] [15/16 Regression] LRA hangs on xstormy16

2025-11-20 Thread vmakarov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=118358

Vladimir Makarov  changed:

   What|Removed |Added

 CC||vmakarov at gcc dot gnu.org

--- Comment #7 from Vladimir Makarov  ---
I've reproduced the bug.

It seems a complicated combination of register elimination and reloads which
increases reg pressure resulting in spilling and repeated changes FP offset.

I hope to find a simple solution (currently I see only non-trivial ones) and
fix it on this week or the next week.

[Bug target/118358] [15/16 Regression] LRA hangs on xstormy16

2025-08-17 Thread mikpelinux at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=118358

Mikael Pettersson  changed:

   What|Removed |Added

 CC||mikpelinux at gmail dot com

--- Comment #6 from Mikael Pettersson  ---
(In reply to Sam James from comment #5)
> If you change your email on BZ to [email protected], you can fix the title
> yourself then ;)

I think the title is correct; I was referring to this snippet in
gcc/config.gcc:

> xstormy16-*-elf)
> # For historical reasons, the target files omit the 'x'.

[Bug target/118358] [15/16 Regression] LRA hangs on xstormy16

2025-08-16 Thread sjames at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=118358

--- Comment #5 from Sam James  ---
(In reply to Mikael Pettersson from comment #4)

If you change your email on BZ to [email protected], you can fix the title
yourself then ;)

[Bug target/118358] [15/16 Regression] LRA hangs on xstormy16

2025-08-16 Thread mikpelinux at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=118358

--- Comment #4 from Mikael Pettersson  ---
(In reply to Tamar Christina from comment #1)
> Is the target backend gcc/config/xtensa/xtensa.md? there are patterns LRA is
> using that I can't find.

In case you haven't already spotted it, the backend is
gcc/config/stormy16/stormy16.md. (No, I don't know where this stormy16 vs
xstormy16 confusion comes from.)

[Bug target/118358] [15/16 Regression] LRA hangs on xstormy16

2025-08-08 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=118358

Richard Biener  changed:

   What|Removed |Added

   Target Milestone|15.2|15.3

--- Comment #3 from Richard Biener  ---
GCC 15.2 is being released, retargeting bugs to 15.3.

[Bug target/118358] [15/16 Regression] LRA hangs on xstormy16

2025-04-25 Thread jakub at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=118358

Jakub Jelinek  changed:

   What|Removed |Added

   Target Milestone|15.0|15.2

--- Comment #2 from Jakub Jelinek  ---
GCC 15.1 is being released, retargeting bugs to GCC 15.2.