]> gcc.gnu.org Git - gcc.git/commitdiff
rs6000: Disparage lfiwzx and similar
authorSegher Boessenkool <segher@kernel.crashing.org>
Sun, 2 Jan 2022 14:08:35 +0000 (14:08 +0000)
committerSegher Boessenkool <segher@kernel.crashing.org>
Tue, 16 Aug 2022 19:08:06 +0000 (19:08 +0000)
RA now chooses GEN_OR_VSX_REGS in most cases.  This is great in most
cases, but we often (or always?) use {l,st}{f,xs}iwzx now, which is
problematic because the integer load and store insns can use cheaper
addressing modes.  We can fix that by putting a small penalty on the
instruction alternatives for those.

2022-04-21  Segher Boessenkool  <segher@kernel.crashing.org>

PR target/103197
PR target/102146
* config/rs6000/rs6000.md (zero_extendqi<mode>2 for EXTQI): Disparage
the "Z" alternatives in {l,st}{f,xs}iwzx.
(zero_extendhi<mode>2 for EXTHI): Ditto.
(zero_extendsi<mode>2 for EXTSI): Ditto.
(*movsi_internal1): Ditto.
(*mov<mode>_internal1 for QHI): Ditto.
(movsd_hardfloat): Ditto.

(cherry picked from commit 26fa464f42622c60d6929720dd37143a21054ede)

gcc/config/rs6000/rs6000.md

index a4a7d105f5e2d3e7d86916f736c0a8c51e371ed9..4ef392ebe7b06b06bd012c618f494fd861eaa160 100644 (file)
 ;; complex forms.  Basic data transfer is done later.
 
 (define_insn "zero_extendqi<mode>2"
-  [(set (match_operand:EXTQI 0 "gpc_reg_operand" "=r,r,^wa,^v")
-       (zero_extend:EXTQI (match_operand:QI 1 "reg_or_mem_operand" "m,r,Z,v")))]
+  [(set (match_operand:EXTQI 0 "gpc_reg_operand" "=r,r,wa,^v")
+       (zero_extend:EXTQI (match_operand:QI 1 "reg_or_mem_operand" "m,r,?Z,v")))]
   ""
   "@
    lbz%U1%X1 %0,%1
 
 
 (define_insn "zero_extendhi<mode>2"
-  [(set (match_operand:EXTHI 0 "gpc_reg_operand" "=r,r,^wa,^v")
-       (zero_extend:EXTHI (match_operand:HI 1 "reg_or_mem_operand" "m,r,Z,v")))]
+  [(set (match_operand:EXTHI 0 "gpc_reg_operand" "=r,r,wa,^v")
+       (zero_extend:EXTHI (match_operand:HI 1 "reg_or_mem_operand" "m,r,?Z,v")))]
   ""
   "@
    lhz%U1%X1 %0,%1
 
 (define_insn "zero_extendsi<mode>2"
   [(set (match_operand:EXTSI 0 "gpc_reg_operand" "=r,r,d,wa,wa,r,wa")
-       (zero_extend:EXTSI (match_operand:SI 1 "reg_or_mem_operand" "m,r,Z,Z,r,wa,wa")))]
+       (zero_extend:EXTSI (match_operand:SI 1 "reg_or_mem_operand" "m,r,?Z,?Z,r,wa,wa")))]
   ""
   "@
    lwz%U1%X1 %0,%1
   [(set (match_operand:SI 0 "nonimmediate_operand"
          "=r,         r,
           r,          d,          v,
-          m,          Z,          Z,
+          m,          ?Z,         ?Z,
           r,          r,          r,          r,
           wa,         wa,         wa,         v,
           wa,         v,          v,
           r,          *h,         *h")
        (match_operand:SI 1 "input_operand"
          "r,          U,
-          m,          Z,          Z,
+          m,          ?Z,         ?Z,
           r,          d,          v,
           I,          L,          eI,         n,
           wa,         O,          wM,         wB,
 ;;             MTVSRWZ     MF%1       MT%1       NOP
 (define_insn "*mov<mode>_internal"
   [(set (match_operand:QHI 0 "nonimmediate_operand"
-               "=r,        r,         wa,        m,         Z,         r,
+               "=r,        r,         wa,        m,         ?Z,        r,
                 wa,        wa,        wa,        v,         ?v,        r,
                 wa,        r,         *c*l,      *h")
        (match_operand:QHI 1 "input_operand"
-               "r,         m,         Z,         r,         wa,        i,
+               "r,         m,         ?Z,        r,         wa,        i,
                 wa,        O,         wM,        wB,        wS,        wa,
                 r,         *h,        r,         0"))]
   "gpc_reg_operand (operands[0], <MODE>mode)
 ;;     FMR          MR         MT%0       MF%1       NOP
 (define_insn "movsd_hardfloat"
   [(set (match_operand:SD 0 "nonimmediate_operand"
-        "=!r,       d,         m,         Z,         ?d,        ?r,
+        "=!r,       d,         m,         ?Z,        ?d,        ?r,
          f,         !r,        *c*l,      !r,        *h")
        (match_operand:SD 1 "input_operand"
-        "m,         Z,         r,         wx,        r,         d,
+        "m,         ?Z,        r,         wx,        r,         d,
          f,         r,         r,         *h,        0"))]
   "(register_operand (operands[0], SDmode)
    || register_operand (operands[1], SDmode))
This page took 0.08098 seconds and 5 git commands to generate.