[PR99766] Consider relaxed memory associated more with memory instead of special...
authorVladimir Makarov <vmakarov@redhat.com>
Fri, 26 Mar 2021 17:09:24 +0000 (17:09 +0000)
committerVladimir Makarov <vmakarov@redhat.com>
Fri, 26 Mar 2021 17:11:30 +0000 (17:11 +0000)
Relaxed memory should be considered more like memory then special memory.

gcc/ChangeLog:

PR target/99766
* ira-costs.c (record_reg_classes): Put case with
CT_RELAXED_MEMORY adjacent to one with CT_MEMORY.
* ira.c (ira_setup_alts): Ditto.
* lra-constraints.c (process_alt_operands): Ditto.
* recog.c (asm_operand_ok): Ditto.
* reload.c (find_reloads): Ditto.

gcc/testsuite/ChangeLog:

PR target/99766
* g++.target/aarch64/sve/pr99766.C: New.

gcc/ira-costs.c
gcc/ira.c
gcc/lra-constraints.c
gcc/recog.c
gcc/reload.c
gcc/testsuite/g++.target/aarch64/sve/pr99766.C [new file with mode: 0644]

index 7547f3e..10727b5 100644 (file)
@@ -773,6 +773,7 @@ record_reg_classes (int n_alts, int n_ops, rtx *ops,
                      break;
 
                    case CT_MEMORY:
+                   case CT_RELAXED_MEMORY:
                      /* Every MEM can be reloaded to fit.  */
                      insn_allows_mem[i] = allows_mem[i] = 1;
                      if (MEM_P (op))
@@ -780,7 +781,6 @@ record_reg_classes (int n_alts, int n_ops, rtx *ops,
                      break;
 
                    case CT_SPECIAL_MEMORY:
-                   case CT_RELAXED_MEMORY:
                      insn_allows_mem[i] = allows_mem[i] = 1;
                      if (MEM_P (extract_mem_from_operand (op))
                          && constraint_satisfied_p (op, cn))
index 7e90328..b93588d 100644 (file)
--- a/gcc/ira.c
+++ b/gcc/ira.c
@@ -1871,10 +1871,10 @@ ira_setup_alts (rtx_insn *insn)
                          goto op_success;
 
                        case CT_MEMORY:
+                       case CT_RELAXED_MEMORY:
                          mem = op;
                          /* Fall through.  */
                        case CT_SPECIAL_MEMORY:
-                       case CT_RELAXED_MEMORY:
                          if (!mem)
                            mem = extract_mem_from_operand (op);
                          if (MEM_P (mem))
index 861b5aa..9993065 100644 (file)
@@ -2417,6 +2417,7 @@ process_alt_operands (int only_alternative)
                      break;
 
                    case CT_MEMORY:
+                   case CT_RELAXED_MEMORY:
                      if (MEM_P (op)
                          && satisfies_memory_constraint_p (op, cn))
                        win = true;
@@ -2459,7 +2460,6 @@ process_alt_operands (int only_alternative)
                      break;
 
                    case CT_SPECIAL_MEMORY:
-                   case CT_RELAXED_MEMORY:
                      if (satisfies_memory_constraint_p (op, cn))
                        win = true;
                      else if (spilled_pseudo_p (op))
index ee143bc..eb617f1 100644 (file)
@@ -2267,10 +2267,10 @@ asm_operand_ok (rtx op, const char *constraint, const char **constraints)
              break;
 
            case CT_MEMORY:
+           case CT_RELAXED_MEMORY:
              mem = op;
              /* Fall through.  */
            case CT_SPECIAL_MEMORY:
-           case CT_RELAXED_MEMORY:
              /* Every memory operand can be reloaded to fit.  */
              if (!mem)
                mem = extract_mem_from_operand (op);
index 7340125..461fd02 100644 (file)
@@ -3471,6 +3471,7 @@ find_reloads (rtx_insn *insn, int replace, int ind_levels, int live_known,
                        break;
 
                      case CT_MEMORY:
+                     case CT_RELAXED_MEMORY:
                        if (force_reload)
                          break;
                        if (constraint_satisfied_p (operand, cn))
@@ -3504,7 +3505,6 @@ find_reloads (rtx_insn *insn, int replace, int ind_levels, int live_known,
                        break;
 
                      case CT_SPECIAL_MEMORY:
-                     case CT_RELAXED_MEMORY:
                        if (force_reload)
                          break;
                        if (constraint_satisfied_p (operand, cn))
diff --git a/gcc/testsuite/g++.target/aarch64/sve/pr99766.C b/gcc/testsuite/g++.target/aarch64/sve/pr99766.C
new file mode 100644 (file)
index 0000000..0ca8aee
--- /dev/null
@@ -0,0 +1,24 @@
+/* { dg-do compile } */
+/* { dg-additional-options "-O3 -march=armv8.2-a+sve" } */
+typedef float a __attribute__((__mode__(HF)));
+typedef struct {
+  a b;
+  a c;
+} d;
+int e;
+d *f, *g;
+__fp16 h;
+void j() {
+  for (int i;; ++i) {
+    auto l = &f[i];
+    for (int k; k < e;) {
+      k = 0;
+      for (; k < e; ++k)
+        g[k].b = l[k].b * l[k].c;
+    }
+    for (int k; k < e; ++k) {
+      g[k].b *= h;
+      g[k].c *= h;
+    }
+  }
+}