]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
LoongArch: Remove unneeded sign extension after crc/crcc instructions
authorXi Ruoyao <xry111@xry111.site>
Sun, 25 Feb 2024 12:44:34 +0000 (20:44 +0800)
committerXi Ruoyao <xry111@xry111.site>
Thu, 29 Feb 2024 06:48:18 +0000 (14:48 +0800)
The specification of crc/crcc instructions is clear that the output is
sign-extended to GRLEN.  Add a define_insn to tell the compiler this
fact and allow it to remove the unneeded sign extension on crc/crcc
output.  As crc/crcc instructions are usually used in a tight loop,
this should produce a significant performance gain.

gcc/ChangeLog:

* config/loongarch/loongarch.md
(loongarch_<crc>_w_<size>_w_extended): New define_insn.

gcc/testsuite/ChangeLog:

* gcc.target/loongarch/crc-sext.c: New test;

gcc/config/loongarch/loongarch.md
gcc/testsuite/gcc.target/loongarch/crc-sext.c [new file with mode: 0644]

index 9646fa90eab784d881f31187833a977b45d7a2d0..f3b5c641fce96162472179361da888e0018f88da 100644 (file)
   [(set_attr "type" "unknown")
    (set_attr "mode" "<MODE>")])
 
+(define_insn "loongarch_<crc>_w_<size>_w_extended"
+  [(set (match_operand:DI 0 "register_operand" "=r")
+       (sign_extend:DI
+         (unspec:SI [(match_operand:QHSD 1 "register_operand" "r")
+                     (match_operand:SI 2 "register_operand" "r")]
+                    CRC)))]
+  "TARGET_64BIT"
+  "<crc>.w.<size>.w\t%0,%1,%2"
+  [(set_attr "type" "unknown")
+   (set_attr "mode" "<MODE>")])
+
 ;; With normal or medium code models, if the only use of a pc-relative
 ;; address is for loading or storing a value, then relying on linker
 ;; relaxation is not better than emitting the machine instruction directly.
diff --git a/gcc/testsuite/gcc.target/loongarch/crc-sext.c b/gcc/testsuite/gcc.target/loongarch/crc-sext.c
new file mode 100644 (file)
index 0000000..9ade5a8
--- /dev/null
@@ -0,0 +1,13 @@
+/* { dg-do compile } */
+/* { dg-options "-O2 -march=loongarch64" } */
+/* { dg-final { check-function-bodies "**" "" } } */
+
+/*
+**my_crc:
+**     crc.w.d.w       \$r4,\$r4,\$r5
+**     jr      \$r1
+*/
+int my_crc(long long dword, int crc)
+{
+       return __builtin_loongarch_crc_w_d_w(dword, crc);
+}