]> git.ipfire.org Git - thirdparty/gcc.git/commit - gcc/config/arc/arc.cc
ARC: Split SImode shifts pre-reload on !TARGET_BARREL_SHIFTER.
authorRoger Sayle <roger@nextmovesoftware.com>
Wed, 4 Oct 2023 16:17:03 +0000 (17:17 +0100)
committerRoger Sayle <roger@nextmovesoftware.com>
Wed, 4 Oct 2023 16:17:03 +0000 (17:17 +0100)
commit3ceb109fcb045de9d78b650112399bbb7df78cdc
treefc5f66f208c42f61ddee8a422f61852bda8d4b10
parentf4e7bba98c1ad143d253d57bcf39dade2a3a868e
ARC: Split SImode shifts pre-reload on !TARGET_BARREL_SHIFTER.

This patch splits SImode shifts, for !TARGET_BARREL_SHIFTER targets,
after combine and before reload, in the split1 pass, as suggested by
the FIXME comment above output_shift in arc.cc.  To do this I've
copied the implementation of the x86_pre_reload_split function from
the i386 backend, and renamed it arc_pre_reload_split.

Although the actual implementations of shifts remain the same
(as in output_shift), having them as explicit instructions in
the RTL stream allows better scheduling and use of compact forms
when available.  The benefits can be seen in two short examples
below.

For the function:
unsigned int foo(unsigned int x, unsigned int y) {
  return y << 2;
}

GCC with -O2 -mcpu=em would previously generate:
foo:    add r1,r1,r1
        add r1,r1,r1
        j_s.d   [blink]
        mov_s   r0,r1   ;4
and with this patch now generates:
foo:    asl_s r0,r1
        j_s.d   [blink]
        asl_s r0,r0

Notice the original (from shift_si3's output_shift) requires the
shift sequence to be monolithic with the same destination register
as the source (requiring an extra mov_s).  The new version can
eliminate this move, and schedule the second asl in the branch
delay slot of the return.

For the function:
int x,y,z;

void bar()
{
  x <<= 3;
  y <<= 3;
  z <<= 3;
}

GCC -O2 -mcpu=em currently generates:
bar: push_s  r13
        ld.as   r12,[gp,@x@sda] ;23
        ld.as   r3,[gp,@y@sda]  ;23
        mov r2,0
        add3 r12,r2,r12
        mov r2,0
        add3 r3,r2,r3
        ld.as   r2,[gp,@z@sda]  ;23
        st.as   r12,[gp,@x@sda] ;26
        mov r13,0
        add3 r2,r13,r2
        st.as   r3,[gp,@y@sda]  ;26
        st.as   r2,[gp,@z@sda]  ;26
        j_s.d   [blink]
        pop_s   r13

where each shift by 3, uses ARC's add3 instruction, which is similar
to x86's lea implementing x = (y<<3) + z, but requires the value zero
to be placed in a temporary register "z".  Splitting this before reload
allows these pseudos to be shared/reused.  With this patch, we get

bar: ld.as   r2,[gp,@x@sda]  ;23
        mov_s   r3,0    ;3
        add3    r2,r3,r2
        ld.as   r3,[gp,@y@sda]  ;23
        st.as   r2,[gp,@x@sda]  ;26
        ld.as   r2,[gp,@z@sda]  ;23
        mov_s   r12,0   ;3
        add3    r3,r12,r3
        add3    r2,r12,r2
        st.as   r3,[gp,@y@sda]  ;26
        st.as   r2,[gp,@z@sda]  ;26
        j_s     [blink]

Unfortunately, register allocation means that we only share two of the
three "mov_s z,0", but this is sufficient to reduce register pressure
enough to avoid spilling r13 in the prologue/epilogue.

2023-10-04  Roger Sayle  <roger@nextmovesoftware.com>

gcc/ChangeLog
* config/arc/arc-protos.h (emit_shift): Delete prototype.
(arc_pre_reload_split): New function prototype.
* config/arc/arc.cc (emit_shift): Delete function.
(arc_pre_reload_split): New predicate function, copied from i386,
to schedule define_insn_and_split splitters to the split1 pass.
* config/arc/arc.md (ashlsi3): Expand RTL template unconditionally.
(ashrsi3): Likewise.
(lshrsi3): Likewise.
(shift_si3): Move after other shift patterns, and disable when
operands[2] is one (which is handled by its own define_insn).
Use shiftr4_operator, instead of shift4_operator, as this is no
longer used for left shifts.
(shift_si3_loop): Likewise.  Additionally remove match_scratch.
(*ashlsi3_nobs): New pre-reload define_insn_and_split.
(*ashrsi3_nobs): Likewise.
(*lshrsi3_nobs): Likewise.
(rotrsi3_cnt1): Rename define_insn from *rotrsi3_cnt1.
(add_shift): Rename define_insn from *add_shift.
* config/arc/predicates.md (shiftl4_operator): Delete.
(shift4_operator): Delete.

gcc/testsuite/ChangeLog
* gcc.target/arc/ashrsi-1.c: New TARGET_BARREL_SHIFTER test case.
* gcc.target/arc/ashrsi-2.c: New !TARGET_BARREL_SHIFTER test case.
* gcc.target/arc/ashrsi-3.c: Likewise.
* gcc.target/arc/ashrsi-4.c: Likewise.
* gcc.target/arc/ashrsi-5.c: Likewise.
* gcc.target/arc/lshrsi-1.c: New TARGET_BARREL_SHIFTER test case.
* gcc.target/arc/lshrsi-2.c: New !TARGET_BARREL_SHIFTER test case.
* gcc.target/arc/lshrsi-3.c: Likewise.
* gcc.target/arc/lshrsi-4.c: Likewise.
* gcc.target/arc/lshrsi-5.c: Likewise.
* gcc.target/arc/shlsi-1.c: New TARGET_BARREL_SHIFTER test case.
* gcc.target/arc/shlsi-2.c: New !TARGET_BARREL_SHIFTER test case.
* gcc.target/arc/shlsi-3.c: Likewise.
* gcc.target/arc/shlsi-4.c: Likewise.
* gcc.target/arc/shlsi-5.c: Likewise.
19 files changed:
gcc/config/arc/arc-protos.h
gcc/config/arc/arc.cc
gcc/config/arc/arc.md
gcc/config/arc/predicates.md
gcc/testsuite/gcc.target/arc/ashrsi-1.c [new file with mode: 0644]
gcc/testsuite/gcc.target/arc/ashrsi-2.c [new file with mode: 0644]
gcc/testsuite/gcc.target/arc/ashrsi-3.c [new file with mode: 0644]
gcc/testsuite/gcc.target/arc/ashrsi-4.c [new file with mode: 0644]
gcc/testsuite/gcc.target/arc/ashrsi-5.c [new file with mode: 0644]
gcc/testsuite/gcc.target/arc/lshrsi-1.c [new file with mode: 0644]
gcc/testsuite/gcc.target/arc/lshrsi-2.c [new file with mode: 0644]
gcc/testsuite/gcc.target/arc/lshrsi-3.c [new file with mode: 0644]
gcc/testsuite/gcc.target/arc/lshrsi-4.c [new file with mode: 0644]
gcc/testsuite/gcc.target/arc/lshrsi-5.c [new file with mode: 0644]
gcc/testsuite/gcc.target/arc/shlsi-1.c [new file with mode: 0644]
gcc/testsuite/gcc.target/arc/shlsi-2.c [new file with mode: 0644]
gcc/testsuite/gcc.target/arc/shlsi-3.c [new file with mode: 0644]
gcc/testsuite/gcc.target/arc/shlsi-4.c [new file with mode: 0644]
gcc/testsuite/gcc.target/arc/shlsi-5.c [new file with mode: 0644]