public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc r12-8226] rs6000: Fix pack for soft-float (PR105334)
@ 2022-04-22 19:29 Segher Boessenkool
0 siblings, 0 replies; only message in thread
From: Segher Boessenkool @ 2022-04-22 19:29 UTC (permalink / raw)
To: gcc-cvs
https://gcc.gnu.org/g:4938888ae1a1680e2aebf394d8fe80faad745bc7
commit r12-8226-g4938888ae1a1680e2aebf394d8fe80faad745bc7
Author: Segher Boessenkool <segher@kernel.crashing.org>
Date: Fri Apr 22 15:45:00 2022 +0000
rs6000: Fix pack for soft-float (PR105334)
For PR103623 I fixed unpack, but pack is broken as well, as reported in
PR105334. Fixing that is a bit more code, but it is pretty simple code
nonetheless.
2022-04-22 Segher Boessenkool <segher@kernel.crashing.org>
PR target/105334
* config/rs6000/rs6000.md (pack<mode> for FMOVE128): New expander.
(pack<mode> for FMOVE128): Rename and split the insn_and_split to...
(pack<mode>_hard for FMOVE128): ... this...
(pack<mode>_soft for FMOVE128): ... and this.
Diff:
---
gcc/config/rs6000/rs6000.md | 45 +++++++++++++++++++++++++++++++++++++++++++--
1 file changed, 43 insertions(+), 2 deletions(-)
diff --git a/gcc/config/rs6000/rs6000.md b/gcc/config/rs6000/rs6000.md
index a39b95f7dff..64049a6e521 100644
--- a/gcc/config/rs6000/rs6000.md
+++ b/gcc/config/rs6000/rs6000.md
@@ -14602,13 +14602,26 @@
}
[(set_attr "type" "fp,fpstore,store")])
-(define_insn_and_split "pack<mode>"
+(define_expand "pack<mode>"
+ [(use (match_operand:FMOVE128 0 "register_operand"))
+ (use (match_operand:<FP128_64> 1 "register_operand"))
+ (use (match_operand:<FP128_64> 2 "register_operand"))]
+ "FLOAT128_2REG_P (<MODE>mode)"
+{
+ if (TARGET_HARD_FLOAT)
+ emit_insn (gen_pack<mode>_hard (operands[0], operands[1], operands[2]));
+ else
+ emit_insn (gen_pack<mode>_soft (operands[0], operands[1], operands[2]));
+ DONE;
+})
+
+(define_insn_and_split "pack<mode>_hard"
[(set (match_operand:FMOVE128 0 "register_operand" "=&d")
(unspec:FMOVE128
[(match_operand:<FP128_64> 1 "register_operand" "d")
(match_operand:<FP128_64> 2 "register_operand" "d")]
UNSPEC_PACK_128BIT))]
- "FLOAT128_2REG_P (<MODE>mode)"
+ "FLOAT128_2REG_P (<MODE>mode) && TARGET_HARD_FLOAT"
"#"
"&& reload_completed"
[(set (match_dup 3) (match_dup 1))
@@ -14626,6 +14639,34 @@
[(set_attr "type" "fp")
(set_attr "length" "8")])
+(define_insn_and_split "pack<mode>_soft"
+ [(set (match_operand:FMOVE128 0 "register_operand" "=&r")
+ (unspec:FMOVE128
+ [(match_operand:<FP128_64> 1 "register_operand" "r")
+ (match_operand:<FP128_64> 2 "register_operand" "r")]
+ UNSPEC_PACK_128BIT))]
+ "FLOAT128_2REG_P (<MODE>mode) && TARGET_SOFT_FLOAT"
+ "#"
+ "&& reload_completed"
+ [(set (match_dup 3) (match_dup 1))
+ (set (match_dup 4) (match_dup 2))]
+{
+ unsigned dest_hi = REGNO (operands[0]);
+ unsigned dest_lo = dest_hi + (TARGET_POWERPC64 ? 1 : 2);
+
+ gcc_assert (!IN_RANGE (REGNO (operands[1]), dest_hi, dest_lo));
+ gcc_assert (!IN_RANGE (REGNO (operands[2]), dest_hi, dest_lo));
+
+ operands[3] = gen_rtx_REG (<FP128_64>mode, dest_hi);
+ operands[4] = gen_rtx_REG (<FP128_64>mode, dest_lo);
+}
+ [(set_attr "type" "integer")
+ (set (attr "length")
+ (if_then_else
+ (match_test "TARGET_POWERPC64")
+ (const_string "8")
+ (const_string "16")))])
+
(define_insn "unpack<mode>"
[(set (match_operand:DI 0 "register_operand" "=wa,wa")
(unspec:DI [(match_operand:FMOVE128_VSX 1 "register_operand" "0,wa")
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2022-04-22 19:29 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-22 19:29 [gcc r12-8226] rs6000: Fix pack for soft-float (PR105334) Segher Boessenkool
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).