https://gcc.gnu.org/g:66c879571ab1fbdb4b119b8b0a1a30ebc7160057

commit r16-2450-g66c879571ab1fbdb4b119b8b0a1a30ebc7160057
Author: Andrew Pinski <quic_apin...@quicinc.com>
Date:   Tue Jul 22 10:26:54 2025 -0700

    aarch64: Fix fma steering when rename fails [PR120119]
    
    Regrename can fail in some case and `insn_rr[INSN_UID (insn)].op_info`
    will be null. The FMA steering code was not expecting the failure to happen.
    This started to happen after early RA was added but it has been a latent bug
    before that.
    
    Build and tested for aarch64-linux-gnu.
    
            PR target/120119
    
    gcc/ChangeLog:
    
            * config/aarch64/cortex-a57-fma-steering.cc 
(func_fma_steering::analyze):
            Skip if renaming fails.
    
    gcc/testsuite/ChangeLog:
    
            * g++.dg/torture/pr120119-1.C: New test.
    
    Signed-off-by: Andrew Pinski <quic_apin...@quicinc.com>

Diff:
---
 gcc/config/aarch64/cortex-a57-fma-steering.cc |  5 +++++
 gcc/testsuite/g++.dg/torture/pr120119-1.C     | 15 +++++++++++++++
 2 files changed, 20 insertions(+)

diff --git a/gcc/config/aarch64/cortex-a57-fma-steering.cc 
b/gcc/config/aarch64/cortex-a57-fma-steering.cc
index fd6da66d855e..f7675bed13d9 100644
--- a/gcc/config/aarch64/cortex-a57-fma-steering.cc
+++ b/gcc/config/aarch64/cortex-a57-fma-steering.cc
@@ -948,6 +948,11 @@ func_fma_steering::analyze ()
 
          /* Search the chain where this instruction is (one of) the root.  */
          dest_op_info = insn_rr[INSN_UID (insn)].op_info;
+
+         /* Register rename could fail. */
+         if (!dest_op_info)
+           continue;
+
          dest_regno = REGNO (SET_DEST (PATTERN (insn)));
          for (i = 0; i < dest_op_info->n_chains; i++)
            {
diff --git a/gcc/testsuite/g++.dg/torture/pr120119-1.C 
b/gcc/testsuite/g++.dg/torture/pr120119-1.C
new file mode 100644
index 000000000000..1206feb310be
--- /dev/null
+++ b/gcc/testsuite/g++.dg/torture/pr120119-1.C
@@ -0,0 +1,15 @@
+// { dg-do compile }
+// { dg-additional-options "-mcpu=cortex-a57" { target aarch64*-*-* } }
+
+// PR target/120119
+
+struct a {
+  float operator()(int b, int c) { return d[c * 4 + b]; }
+  float *d;
+};
+float e(float *);
+auto f(a b) {
+  float g[]{b(1, 1), b(2, 1), b(3, 1), b(1, 2), b(2, 2), b(3, 2), b(1, 3),
+            b(2, 3), b(3, 3), b(3, 2), b(1, 3), b(2, 3), b(3, 3)};
+  return b.d[0] * e(g);
+}

Reply via email to