From: Lulu Cheng <chenglulu@loongson.cn>
To: Xi Ruoyao <xry111@xry111.site>, gcc-patches@gcc.gnu.org
Cc: Wang Xuerui <i@xen0n.name>, Chenghua Xu <xuchenghua@loongson.cn>,
Xiaolin Tang <tangxiaolin@loongson.cn>
Subject: Re: [PATCH] LoongArch: Add fcopysign instructions
Date: Sun, 6 Nov 2022 09:57:58 +0800 [thread overview]
Message-ID: <490fbb16-36e1-aa12-6a9d-ead6fbd56c6e@loongson.cn> (raw)
In-Reply-To: <20221104143719.1709284-1-xry111@xry111.site>
在 2022/11/4 下午10:37, Xi Ruoyao 写道:
> Add fcopysign.{s,d} with the names copysign{sf,df}3 so GCC will expand
> __builtin_copysign{f,} to a single instruction.
>
> Link: https://sourceware.org/pipermail/libc-alpha/2022-November/143177.html
>
> gcc/ChangeLog:
>
> * config/loongarch/loongarch.md (UNSPEC_FCOPYSIGN): New unspec.
> (type): Add fcopysign.
> (copysign<mode>3): New instruction template.
>
> gcc/testsuite/ChangeLog:
>
> * gcc.target/loongarch/fcopysign.c: New test.
> ---
>
> Bootstrapped and regtested on loongarch64-linux-gnu. Ok for trunk?
>
> gcc/config/loongarch/loongarch.md | 22 ++++++++++++++++++-
> .../gcc.target/loongarch/fcopysign.c | 16 ++++++++++++++
> 2 files changed, 37 insertions(+), 1 deletion(-)
> create mode 100644 gcc/testsuite/gcc.target/loongarch/fcopysign.c
>
> diff --git a/gcc/config/loongarch/loongarch.md b/gcc/config/loongarch/loongarch.md
> index 214b14bddd3..042e6e74491 100644
> --- a/gcc/config/loongarch/loongarch.md
> +++ b/gcc/config/loongarch/loongarch.md
> @@ -37,6 +37,7 @@ (define_c_enum "unspec" [
> UNSPEC_FCLASS
> UNSPEC_FMAX
> UNSPEC_FMIN
> + UNSPEC_FCOPYSIGN
>
> ;; Override return address for exception handling.
> UNSPEC_EH_RETURN
> @@ -214,6 +215,7 @@ (define_attr "qword_mode" "no,yes"
> ;; fabs floating point absolute value
> ;; fneg floating point negation
> ;; fcmp floating point compare
> +;; fcopysign floating point copysign
> ;; fcvt floating point convert
> ;; fsqrt floating point square root
> ;; frsqrt floating point reciprocal square root
> @@ -226,7 +228,7 @@ (define_attr "type"
> "unknown,branch,jump,call,load,fpload,fpidxload,store,fpstore,fpidxstore,
> prefetch,prefetchx,condmove,mgtf,mftg,const,arith,logical,
> shift,slt,signext,clz,trap,imul,idiv,move,
> - fmove,fadd,fmul,fmadd,fdiv,frdiv,fabs,fneg,fcmp,fcvt,fsqrt,
> + fmove,fadd,fmul,fmadd,fdiv,frdiv,fabs,fneg,fcmp,fcopysign,fcvt,fsqrt,
> frsqrt,accext,accmod,multi,atomic,syncloop,nop,ghost"
> (cond [(eq_attr "jirl" "!unset") (const_string "call")
> (eq_attr "got" "load") (const_string "load")
> @@ -976,6 +978,24 @@ (define_insn "abs<mode>2"
> (set_attr "mode" "<UNITMODE>")])
> \f
> ;;
> +;; ....................
> +;;
> +;; FLOATING POINT COPYSIGN
> +;;
> +;; ....................
> +
> +(define_insn "copysign<mode>3"
> + [(set (match_operand:ANYF 0 "register_operand" "=f")
> + (unspec:ANYF [(match_operand:ANYF 1 "register_operand" "f")
> + (match_operand:ANYF 2 "register_operand" "f")]
> + UNSPEC_FCOPYSIGN))]
> + ""
We need to add "TARGET_HARD_FLOAT" here.
Otherwise LGTM.
Thanks!
> + "fcopysign.<fmt>\t%0,%1,%2"
> + [(set_attr "type" "fcopysign")
> + (set_attr "mode" "<UNITMODE>")])
> +
> +\f
> +;;
> ;; ...................
> ;;
> ;; Count leading zeroes.
> diff --git a/gcc/testsuite/gcc.target/loongarch/fcopysign.c b/gcc/testsuite/gcc.target/loongarch/fcopysign.c
> new file mode 100644
> index 00000000000..058ba2cf573
> --- /dev/null
> +++ b/gcc/testsuite/gcc.target/loongarch/fcopysign.c
> @@ -0,0 +1,16 @@
> +/* { dg-do compile } */
> +/* { dg-options "-mdouble-float" } */
> +/* { dg-final { scan-assembler "fcopysign\\.s" } } */
> +/* { dg-final { scan-assembler "fcopysign\\.d" } } */
> +
> +double
> +my_copysign (double a, double b)
> +{
> + return __builtin_copysign (a, b);
> +}
> +
> +float
> +my_copysignf (float a, float b)
> +{
> + return __builtin_copysignf (a, b);
> +}
next prev parent reply other threads:[~2022-11-06 1:58 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-11-04 14:37 Xi Ruoyao
2022-11-06 1:57 ` Lulu Cheng [this message]
2022-11-06 2:23 ` Xi Ruoyao
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=490fbb16-36e1-aa12-6a9d-ead6fbd56c6e@loongson.cn \
--to=chenglulu@loongson.cn \
--cc=gcc-patches@gcc.gnu.org \
--cc=i@xen0n.name \
--cc=tangxiaolin@loongson.cn \
--cc=xry111@xry111.site \
--cc=xuchenghua@loongson.cn \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).