LoongArch: Fix gnome-desktop boot error
Signed-off-by: Peng Fan <fanpeng@loongson.cn>
This commit is contained in:
parent
e5c622d736
commit
4ee61e1191
37
LoongArch-Fixed-bug-in-bstrins_-mode-_for_ior_mask-t.patch
Normal file
37
LoongArch-Fixed-bug-in-bstrins_-mode-_for_ior_mask-t.patch
Normal file
@ -0,0 +1,37 @@
|
|||||||
|
From 245c9ef2b8574fcaf277416d5986f46c0babbbec Mon Sep 17 00:00:00 2001
|
||||||
|
From: Li Wei <liwei@loongson.cn>
|
||||||
|
Date: Mon, 25 Dec 2023 11:20:23 +0800
|
||||||
|
Subject: [PATCH] LoongArch: Fixed bug in *bstrins_<mode>_for_ior_mask
|
||||||
|
template.
|
||||||
|
|
||||||
|
We found that using the latest compiled gcc will cause a miscompare error
|
||||||
|
when running spec2006 400.perlbench test with -flto turned on. After testing,
|
||||||
|
it was found that only the LoongArch architecture will report errors.
|
||||||
|
The first error commit was located through the git bisect command as
|
||||||
|
r14-3773-g5b857e87201335. Through debugging, it was found that the problem
|
||||||
|
was that the split condition of the *bstrins_<mode>_for_ior_mask template was
|
||||||
|
empty, which should actually be consistent with the insn condition.
|
||||||
|
|
||||||
|
gcc/ChangeLog:
|
||||||
|
|
||||||
|
* config/loongarch/loongarch.md: Adjust.
|
||||||
|
---
|
||||||
|
gcc/config/loongarch/loongarch.md | 2 +-
|
||||||
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||||
|
|
||||||
|
diff --git a/gcc/config/loongarch/loongarch.md b/gcc/config/loongarch/loongarch.md
|
||||||
|
index 7021105b241..2b0609f2f31 100644
|
||||||
|
--- a/gcc/config/loongarch/loongarch.md
|
||||||
|
+++ b/gcc/config/loongarch/loongarch.md
|
||||||
|
@@ -1489,7 +1489,7 @@
|
||||||
|
"loongarch_pre_reload_split () && \
|
||||||
|
loongarch_use_bstrins_for_ior_with_mask (<MODE>mode, operands)"
|
||||||
|
"#"
|
||||||
|
- ""
|
||||||
|
+ "&& true"
|
||||||
|
[(set (match_dup 0) (match_dup 1))
|
||||||
|
(set (zero_extract:GPR (match_dup 0) (match_dup 2) (match_dup 4))
|
||||||
|
(match_dup 3))]
|
||||||
|
--
|
||||||
|
2.33.0
|
||||||
|
|
||||||
112
LoongArch-Optimize-the-loading-of-immediate-numbers-.patch
Normal file
112
LoongArch-Optimize-the-loading-of-immediate-numbers-.patch
Normal file
@ -0,0 +1,112 @@
|
|||||||
|
From 96577dfae97e49a052c0b4d9843fb10f89632fd9 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Guo Jie <guojie@loongson.cn>
|
||||||
|
Date: Thu, 23 Nov 2023 11:04:17 +0800
|
||||||
|
Subject: [PATCH] LoongArch: Optimize the loading of immediate numbers with the
|
||||||
|
same high and low 32-bit values
|
||||||
|
|
||||||
|
For the following immediate load operation in gcc/testsuite/gcc.target/loongarch/imm-load1.c:
|
||||||
|
|
||||||
|
long long r = 0x0101010101010101;
|
||||||
|
|
||||||
|
Before this patch:
|
||||||
|
|
||||||
|
lu12i.w $r15,16842752>>12
|
||||||
|
ori $r15,$r15,257
|
||||||
|
lu32i.d $r15,0x1010100000000>>32
|
||||||
|
lu52i.d $r15,$r15,0x100000000000000>>52
|
||||||
|
|
||||||
|
After this patch:
|
||||||
|
|
||||||
|
lu12i.w $r15,16842752>>12
|
||||||
|
ori $r15,$r15,257
|
||||||
|
bstrins.d $r15,$r15,63,32
|
||||||
|
|
||||||
|
gcc/ChangeLog:
|
||||||
|
|
||||||
|
* config/loongarch/loongarch.cc
|
||||||
|
(enum loongarch_load_imm_method): Add new method.
|
||||||
|
(loongarch_build_integer): Add relevant implementations for
|
||||||
|
new method.
|
||||||
|
(loongarch_move_integer): Ditto.
|
||||||
|
|
||||||
|
gcc/testsuite/ChangeLog:
|
||||||
|
|
||||||
|
* gcc.target/loongarch/imm-load1.c: Change old check.
|
||||||
|
---
|
||||||
|
gcc/config/loongarch/loongarch.cc | 22 ++++++++++++++++++-
|
||||||
|
.../gcc.target/loongarch/imm-load1.c | 3 ++-
|
||||||
|
2 files changed, 23 insertions(+), 2 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/gcc/config/loongarch/loongarch.cc b/gcc/config/loongarch/loongarch.cc
|
||||||
|
index 33357c670e1..c95fc83362e 100644
|
||||||
|
--- a/gcc/config/loongarch/loongarch.cc
|
||||||
|
+++ b/gcc/config/loongarch/loongarch.cc
|
||||||
|
@@ -142,12 +142,16 @@ struct loongarch_address_info
|
||||||
|
|
||||||
|
METHOD_LU52I:
|
||||||
|
Load 52-63 bit of the immediate number.
|
||||||
|
+
|
||||||
|
+ METHOD_MIRROR:
|
||||||
|
+ Copy 0-31 bit of the immediate number to 32-63bit.
|
||||||
|
*/
|
||||||
|
enum loongarch_load_imm_method
|
||||||
|
{
|
||||||
|
METHOD_NORMAL,
|
||||||
|
METHOD_LU32I,
|
||||||
|
- METHOD_LU52I
|
||||||
|
+ METHOD_LU52I,
|
||||||
|
+ METHOD_MIRROR
|
||||||
|
};
|
||||||
|
|
||||||
|
struct loongarch_integer_op
|
||||||
|
@@ -1556,11 +1560,23 @@ loongarch_build_integer (struct loongarch_integer_op *codes,
|
||||||
|
|
||||||
|
int sign31 = (value & (HOST_WIDE_INT_1U << 31)) >> 31;
|
||||||
|
int sign51 = (value & (HOST_WIDE_INT_1U << 51)) >> 51;
|
||||||
|
+
|
||||||
|
+ uint32_t hival = (uint32_t) (value >> 32);
|
||||||
|
+ uint32_t loval = (uint32_t) value;
|
||||||
|
+
|
||||||
|
/* Determine whether the upper 32 bits are sign-extended from the lower
|
||||||
|
32 bits. If it is, the instructions to load the high order can be
|
||||||
|
ommitted. */
|
||||||
|
if (lu32i[sign31] && lu52i[sign31])
|
||||||
|
return cost;
|
||||||
|
+ /* If the lower 32 bits are the same as the upper 32 bits, just copy
|
||||||
|
+ the lower 32 bits to the upper 32 bits. */
|
||||||
|
+ else if (loval == hival)
|
||||||
|
+ {
|
||||||
|
+ codes[cost].method = METHOD_MIRROR;
|
||||||
|
+ codes[cost].curr_value = value;
|
||||||
|
+ return cost + 1;
|
||||||
|
+ }
|
||||||
|
/* Determine whether bits 32-51 are sign-extended from the lower 32
|
||||||
|
bits. If so, directly load 52-63 bits. */
|
||||||
|
else if (lu32i[sign31])
|
||||||
|
@@ -3234,6 +3250,10 @@ loongarch_move_integer (rtx temp, rtx dest, unsigned HOST_WIDE_INT value)
|
||||||
|
gen_rtx_AND (DImode, x, GEN_INT (0xfffffffffffff)),
|
||||||
|
GEN_INT (codes[i].value));
|
||||||
|
break;
|
||||||
|
+ case METHOD_MIRROR:
|
||||||
|
+ gcc_assert (mode == DImode);
|
||||||
|
+ emit_insn (gen_insvdi (x, GEN_INT (32), GEN_INT (32), x));
|
||||||
|
+ break;
|
||||||
|
default:
|
||||||
|
gcc_unreachable ();
|
||||||
|
}
|
||||||
|
diff --git a/gcc/testsuite/gcc.target/loongarch/imm-load1.c b/gcc/testsuite/gcc.target/loongarch/imm-load1.c
|
||||||
|
index 2ff02971239..f64cc2956a3 100644
|
||||||
|
--- a/gcc/testsuite/gcc.target/loongarch/imm-load1.c
|
||||||
|
+++ b/gcc/testsuite/gcc.target/loongarch/imm-load1.c
|
||||||
|
@@ -1,6 +1,7 @@
|
||||||
|
/* { dg-do compile } */
|
||||||
|
/* { dg-options "-mabi=lp64d -O2" } */
|
||||||
|
-/* { dg-final { scan-assembler "test:.*lu52i\.d.*\n\taddi\.w.*\n\.L2:" } } */
|
||||||
|
+/* { dg-final { scan-assembler-not "test:.*lu52i\.d.*\n\taddi\.w.*\n\.L2:" } } */
|
||||||
|
+/* { dg-final { scan-assembler "test:.*lu12i\.w.*\n\tbstrins\.d.*\n\.L2:" } } */
|
||||||
|
|
||||||
|
|
||||||
|
extern long long b[10];
|
||||||
|
--
|
||||||
|
2.33.0
|
||||||
|
|
||||||
11
gcc.spec
11
gcc.spec
@ -2,7 +2,7 @@
|
|||||||
%global gcc_major 12
|
%global gcc_major 12
|
||||||
# Note, gcc_release must be integer, if you want to add suffixes to
|
# Note, gcc_release must be integer, if you want to add suffixes to
|
||||||
# %%{release}, append them after %%{gcc_release} on Release: line.
|
# %%{release}, append them after %%{gcc_release} on Release: line.
|
||||||
%global gcc_release 32
|
%global gcc_release 33
|
||||||
|
|
||||||
%global _unpackaged_files_terminate_build 0
|
%global _unpackaged_files_terminate_build 0
|
||||||
%global _performance_build 1
|
%global _performance_build 1
|
||||||
@ -336,6 +336,8 @@ Patch3128: LoongArch-Add-LA664-support.patch
|
|||||||
Patch3129: LoongArch-Fix-internal-error-running-gcc-march-nativ.patch
|
Patch3129: LoongArch-Fix-internal-error-running-gcc-march-nativ.patch
|
||||||
Patch3130: LoongArch-Fix-lsx-vshuf.c-and-lasx-xvshuf_b.c-tests-.patch
|
Patch3130: LoongArch-Fix-lsx-vshuf.c-and-lasx-xvshuf_b.c-tests-.patch
|
||||||
Patch3131: LoongArch-Fix-osdirname.patch
|
Patch3131: LoongArch-Fix-osdirname.patch
|
||||||
|
Patch3132: LoongArch-Optimize-the-loading-of-immediate-numbers-.patch
|
||||||
|
Patch3133: LoongArch-Fixed-bug-in-bstrins_-mode-_for_ior_mask-t.patch
|
||||||
%endif
|
%endif
|
||||||
|
|
||||||
# On ARM EABI systems, we do want -gnueabi to be part of the
|
# On ARM EABI systems, we do want -gnueabi to be part of the
|
||||||
@ -1003,6 +1005,8 @@ not stable, so plugins must be rebuilt any time GCC is updated.
|
|||||||
%patch3129 -p1
|
%patch3129 -p1
|
||||||
%patch3130 -p1
|
%patch3130 -p1
|
||||||
%patch3131 -p1
|
%patch3131 -p1
|
||||||
|
%patch3132 -p1
|
||||||
|
%patch3133 -p1
|
||||||
%endif
|
%endif
|
||||||
|
|
||||||
echo '%{_vendor} %{version}-%{release}' > gcc/DEV-PHASE
|
echo '%{_vendor} %{version}-%{release}' > gcc/DEV-PHASE
|
||||||
@ -3271,6 +3275,11 @@ end
|
|||||||
%doc rpm.doc/changelogs/libcc1/ChangeLog*
|
%doc rpm.doc/changelogs/libcc1/ChangeLog*
|
||||||
|
|
||||||
%changelog
|
%changelog
|
||||||
|
* Wed Jul 24 2024 Peng Fan <fanpeng@loongson.cn> - 12.3.1-33
|
||||||
|
- Type: Sync
|
||||||
|
- DESC: Optimize the loading of immediate numbers.
|
||||||
|
- Fixed bug in bstrins_<mode>for_ior_mask.
|
||||||
|
|
||||||
* Fri Jun 14 2024 peng.zou <peng.zou@shingroup.cn> - 12.3.1-32
|
* Fri Jun 14 2024 peng.zou <peng.zou@shingroup.cn> - 12.3.1-32
|
||||||
- Type: SPEC
|
- Type: SPEC
|
||||||
- DESC: set default configuration for the ppc64le
|
- DESC: set default configuration for the ppc64le
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user