From: chenglulu <chenglulu@loongson.cn>
To: Jiajie Chen <c@jia.je>, gcc-patches@gcc.gnu.org
Cc: xuchenghua@loongson.cn
Subject: Re:[pushed] [PATCH v2] extend.texi: Fix typos in LSX intrinsics
Date: Thu, 21 Dec 2023 17:44:15 +0800 [thread overview]
Message-ID: <3cae32b5-4ab6-31c9-d82a-e2f0451f27db@loongson.cn> (raw)
In-Reply-To: <20231213152615.4132230-1-c@jia.je>
Pushed to r14-6775.
Thank you so much!
在 2023/12/13 下午11:26, Jiajie Chen 写道:
> Several typos have been found and fixed: missing semicolons, using
> variable name instead of type, duplicate functions and wrong types.
>
> gcc/ChangeLog:
>
> * doc/extend.texi(__lsx_vabsd_di): remove extra `i' in name.
> (__lsx_vfrintrm_d, __lsx_vfrintrm_s, __lsx_vfrintrne_d,
> __lsx_vfrintrne_s, __lsx_vfrintrp_d, __lsx_vfrintrp_s, __lsx_vfrintrz_d,
> __lsx_vfrintrz_s): fix return types.
> (__lsx_vld, __lsx_vldi, __lsx_vldrepl_b, __lsx_vldrepl_d,
> __lsx_vldrepl_h, __lsx_vldrepl_w, __lsx_vmaxi_b, __lsx_vmaxi_d,
> __lsx_vmaxi_h, __lsx_vmaxi_w, __lsx_vmini_b, __lsx_vmini_d,
> __lsx_vmini_h, __lsx_vmini_w, __lsx_vsrani_d_q, __lsx_vsrarni_d_q,
> __lsx_vsrlni_d_q, __lsx_vsrlrni_d_q, __lsx_vssrani_d_q,
> __lsx_vssrarni_d_q, __lsx_vssrarni_du_q, __lsx_vssrlni_d_q,
> __lsx_vssrlrni_du_q, __lsx_vst, __lsx_vstx, __lsx_vssrani_du_q,
> __lsx_vssrlni_du_q, __lsx_vssrlrni_d_q): add missing semicolon.
> (__lsx_vpickve2gr_bu, __lsx_vpickve2gr_hu): fix typo in return
> type.
> (__lsx_vstelm_b, __lsx_vstelm_d, __lsx_vstelm_h,
> __lsx_vstelm_w): use imm type for the last argument.
> (__lsx_vsigncov_b, __lsx_vsigncov_h, __lsx_vsigncov_w,
> __lsx_vsigncov_d): remove duplicate definitions.
> ---
> gcc/doc/extend.texi | 90 ++++++++++++++++++++++-----------------------
> 1 file changed, 43 insertions(+), 47 deletions(-)
>
> diff --git a/gcc/doc/extend.texi b/gcc/doc/extend.texi
> index f0c789f6cb4..ba1317c3510 100644
> --- a/gcc/doc/extend.texi
> +++ b/gcc/doc/extend.texi
> @@ -17563,7 +17563,7 @@ int __lsx_bz_v (__m128i);
> int __lsx_bz_w (__m128i);
> __m128i __lsx_vabsd_b (__m128i, __m128i);
> __m128i __lsx_vabsd_bu (__m128i, __m128i);
> -__m128i __lsx_vabsd_di (__m128i, __m128i);
> +__m128i __lsx_vabsd_d (__m128i, __m128i);
> __m128i __lsx_vabsd_du (__m128i, __m128i);
> __m128i __lsx_vabsd_h (__m128i, __m128i);
> __m128i __lsx_vabsd_hu (__m128i, __m128i);
> @@ -17769,14 +17769,14 @@ __m128 __lsx_vfnmsub_s (__m128, __m128, __m128);
> __m128d __lsx_vfrecip_d (__m128d);
> __m128 __lsx_vfrecip_s (__m128);
> __m128d __lsx_vfrint_d (__m128d);
> -__m128i __lsx_vfrintrm_d (__m128d);
> -__m128i __lsx_vfrintrm_s (__m128);
> -__m128i __lsx_vfrintrne_d (__m128d);
> -__m128i __lsx_vfrintrne_s (__m128);
> -__m128i __lsx_vfrintrp_d (__m128d);
> -__m128i __lsx_vfrintrp_s (__m128);
> -__m128i __lsx_vfrintrz_d (__m128d);
> -__m128i __lsx_vfrintrz_s (__m128);
> +__m128d __lsx_vfrintrm_d (__m128d);
> +__m128 __lsx_vfrintrm_s (__m128);
> +__m128d __lsx_vfrintrne_d (__m128d);
> +__m128 __lsx_vfrintrne_s (__m128);
> +__m128d __lsx_vfrintrp_d (__m128d);
> +__m128 __lsx_vfrintrp_s (__m128);
> +__m128d __lsx_vfrintrz_d (__m128d);
> +__m128 __lsx_vfrintrz_s (__m128);
> __m128 __lsx_vfrint_s (__m128);
> __m128d __lsx_vfrsqrt_d (__m128d);
> __m128 __lsx_vfrsqrt_s (__m128);
> @@ -17845,12 +17845,12 @@ __m128i __lsx_vinsgr2vr_b (__m128i, int, imm0_15);
> __m128i __lsx_vinsgr2vr_d (__m128i, long int, imm0_1);
> __m128i __lsx_vinsgr2vr_h (__m128i, int, imm0_7);
> __m128i __lsx_vinsgr2vr_w (__m128i, int, imm0_3);
> -__m128i __lsx_vld (void *, imm_n2048_2047)
> -__m128i __lsx_vldi (imm_n1024_1023)
> -__m128i __lsx_vldrepl_b (void *, imm_n2048_2047)
> -__m128i __lsx_vldrepl_d (void *, imm_n256_255)
> -__m128i __lsx_vldrepl_h (void *, imm_n1024_1023)
> -__m128i __lsx_vldrepl_w (void *, imm_n512_511)
> +__m128i __lsx_vld (void *, imm_n2048_2047);
> +__m128i __lsx_vldi (imm_n1024_1023);
> +__m128i __lsx_vldrepl_b (void *, imm_n2048_2047);
> +__m128i __lsx_vldrepl_d (void *, imm_n256_255);
> +__m128i __lsx_vldrepl_h (void *, imm_n1024_1023);
> +__m128i __lsx_vldrepl_w (void *, imm_n512_511);
> __m128i __lsx_vldx (void *, long int);
> __m128i __lsx_vmadd_b (__m128i, __m128i, __m128i);
> __m128i __lsx_vmadd_d (__m128i, __m128i, __m128i);
> @@ -17886,13 +17886,13 @@ __m128i __lsx_vmax_d (__m128i, __m128i);
> __m128i __lsx_vmax_du (__m128i, __m128i);
> __m128i __lsx_vmax_h (__m128i, __m128i);
> __m128i __lsx_vmax_hu (__m128i, __m128i);
> -__m128i __lsx_vmaxi_b (__m128i, imm_n16_15)
> +__m128i __lsx_vmaxi_b (__m128i, imm_n16_15);
> __m128i __lsx_vmaxi_bu (__m128i, imm0_31);
> -__m128i __lsx_vmaxi_d (__m128i, imm_n16_15)
> +__m128i __lsx_vmaxi_d (__m128i, imm_n16_15);
> __m128i __lsx_vmaxi_du (__m128i, imm0_31);
> -__m128i __lsx_vmaxi_h (__m128i, imm_n16_15)
> +__m128i __lsx_vmaxi_h (__m128i, imm_n16_15);
> __m128i __lsx_vmaxi_hu (__m128i, imm0_31);
> -__m128i __lsx_vmaxi_w (__m128i, imm_n16_15)
> +__m128i __lsx_vmaxi_w (__m128i, imm_n16_15);
> __m128i __lsx_vmaxi_wu (__m128i, imm0_31);
> __m128i __lsx_vmax_w (__m128i, __m128i);
> __m128i __lsx_vmax_wu (__m128i, __m128i);
> @@ -17902,13 +17902,13 @@ __m128i __lsx_vmin_d (__m128i, __m128i);
> __m128i __lsx_vmin_du (__m128i, __m128i);
> __m128i __lsx_vmin_h (__m128i, __m128i);
> __m128i __lsx_vmin_hu (__m128i, __m128i);
> -__m128i __lsx_vmini_b (__m128i, imm_n16_15)
> +__m128i __lsx_vmini_b (__m128i, imm_n16_15);
> __m128i __lsx_vmini_bu (__m128i, imm0_31);
> -__m128i __lsx_vmini_d (__m128i, imm_n16_15)
> +__m128i __lsx_vmini_d (__m128i, imm_n16_15);
> __m128i __lsx_vmini_du (__m128i, imm0_31);
> -__m128i __lsx_vmini_h (__m128i, imm_n16_15)
> +__m128i __lsx_vmini_h (__m128i, imm_n16_15);
> __m128i __lsx_vmini_hu (__m128i, imm0_31);
> -__m128i __lsx_vmini_w (__m128i, imm_n16_15)
> +__m128i __lsx_vmini_w (__m128i, imm_n16_15);
> __m128i __lsx_vmini_wu (__m128i, imm0_31);
> __m128i __lsx_vmin_w (__m128i, __m128i);
> __m128i __lsx_vmin_wu (__m128i, __m128i);
> @@ -17997,11 +17997,11 @@ __m128i __lsx_vpickod_d (__m128i, __m128i);
> __m128i __lsx_vpickod_h (__m128i, __m128i);
> __m128i __lsx_vpickod_w (__m128i, __m128i);
> int __lsx_vpickve2gr_b (__m128i, imm0_15);
> -unsinged int __lsx_vpickve2gr_bu (__m128i, imm0_15);
> +unsigned int __lsx_vpickve2gr_bu (__m128i, imm0_15);
> long int __lsx_vpickve2gr_d (__m128i, imm0_1);
> unsigned long int __lsx_vpickve2gr_du (__m128i, imm0_1);
> int __lsx_vpickve2gr_h (__m128i, imm0_7);
> -unsinged int __lsx_vpickve2gr_hu (__m128i, imm0_7);
> +unsigned int __lsx_vpickve2gr_hu (__m128i, imm0_7);
> int __lsx_vpickve2gr_w (__m128i, imm0_3);
> unsigned int __lsx_vpickve2gr_wu (__m128i, imm0_3);
> __m128i __lsx_vreplgr2vr_b (int);
> @@ -18064,10 +18064,6 @@ __m128i __lsx_vsigncov_b (__m128i, __m128i);
> __m128i __lsx_vsigncov_d (__m128i, __m128i);
> __m128i __lsx_vsigncov_h (__m128i, __m128i);
> __m128i __lsx_vsigncov_w (__m128i, __m128i);
> -__m128i __lsx_vsigncov_b (__m128i, __m128i);
> -__m128i __lsx_vsigncov_d (__m128i, __m128i);
> -__m128i __lsx_vsigncov_h (__m128i, __m128i);
> -__m128i __lsx_vsigncov_w (__m128i, __m128i);
> __m128i __lsx_vsle_b (__m128i, __m128i);
> __m128i __lsx_vsle_bu (__m128i, __m128i);
> __m128i __lsx_vsle_d (__m128i, __m128i);
> @@ -18124,7 +18120,7 @@ __m128i __lsx_vsrai_w (__m128i, imm0_31);
> __m128i __lsx_vsran_b_h (__m128i, __m128i);
> __m128i __lsx_vsran_h_w (__m128i, __m128i);
> __m128i __lsx_vsrani_b_h (__m128i, __m128i, imm0_15);
> -__m128i __lsx_vsrani_d_q (__m128i, __m128i, imm0_127)
> +__m128i __lsx_vsrani_d_q (__m128i, __m128i, imm0_127);
> __m128i __lsx_vsrani_h_w (__m128i, __m128i, imm0_31);
> __m128i __lsx_vsrani_w_d (__m128i, __m128i, imm0_63);
> __m128i __lsx_vsran_w_d (__m128i, __m128i);
> @@ -18138,7 +18134,7 @@ __m128i __lsx_vsrari_w (__m128i, imm0_31);
> __m128i __lsx_vsrarn_b_h (__m128i, __m128i);
> __m128i __lsx_vsrarn_h_w (__m128i, __m128i);
> __m128i __lsx_vsrarni_b_h (__m128i, __m128i, imm0_15);
> -__m128i __lsx_vsrarni_d_q (__m128i, __m128i, imm0_127)
> +__m128i __lsx_vsrarni_d_q (__m128i, __m128i, imm0_127);
> __m128i __lsx_vsrarni_h_w (__m128i, __m128i, imm0_31);
> __m128i __lsx_vsrarni_w_d (__m128i, __m128i, imm0_63);
> __m128i __lsx_vsrarn_w_d (__m128i, __m128i);
> @@ -18154,7 +18150,7 @@ __m128i __lsx_vsrli_w (__m128i, imm0_31);
> __m128i __lsx_vsrln_b_h (__m128i, __m128i);
> __m128i __lsx_vsrln_h_w (__m128i, __m128i);
> __m128i __lsx_vsrlni_b_h (__m128i, __m128i, imm0_15);
> -__m128i __lsx_vsrlni_d_q (__m128i, __m128i, imm0_127)
> +__m128i __lsx_vsrlni_d_q (__m128i, __m128i, imm0_127);
> __m128i __lsx_vsrlni_h_w (__m128i, __m128i, imm0_31);
> __m128i __lsx_vsrlni_w_d (__m128i, __m128i, imm0_63);
> __m128i __lsx_vsrln_w_d (__m128i, __m128i);
> @@ -18168,7 +18164,7 @@ __m128i __lsx_vsrlri_w (__m128i, imm0_31);
> __m128i __lsx_vsrlrn_b_h (__m128i, __m128i);
> __m128i __lsx_vsrlrn_h_w (__m128i, __m128i);
> __m128i __lsx_vsrlrni_b_h (__m128i, __m128i, imm0_15);
> -__m128i __lsx_vsrlrni_d_q (__m128i, __m128i, imm0_127)
> +__m128i __lsx_vsrlrni_d_q (__m128i, __m128i, imm0_127);
> __m128i __lsx_vsrlrni_h_w (__m128i, __m128i, imm0_31);
> __m128i __lsx_vsrlrni_w_d (__m128i, __m128i, imm0_63);
> __m128i __lsx_vsrlrn_w_d (__m128i, __m128i);
> @@ -18180,8 +18176,8 @@ __m128i __lsx_vssran_hu_w (__m128i, __m128i);
> __m128i __lsx_vssran_h_w (__m128i, __m128i);
> __m128i __lsx_vssrani_b_h (__m128i, __m128i, imm0_15);
> __m128i __lsx_vssrani_bu_h (__m128i, __m128i, imm0_15);
> -__m128i __lsx_vssrani_d_q (__m128i, __m128i, imm0_127)
> -__m128i __lsx_vssrani_du_q (__m128i, __m128i, imm0_127)
> +__m128i __lsx_vssrani_d_q (__m128i, __m128i, imm0_127);
> +__m128i __lsx_vssrani_du_q (__m128i, __m128i, imm0_127);
> __m128i __lsx_vssrani_hu_w (__m128i, __m128i, imm0_31);
> __m128i __lsx_vssrani_h_w (__m128i, __m128i, imm0_31);
> __m128i __lsx_vssrani_w_d (__m128i, __m128i, imm0_63);
> @@ -18194,8 +18190,8 @@ __m128i __lsx_vssrarn_hu_w (__m128i, __m128i);
> __m128i __lsx_vssrarn_h_w (__m128i, __m128i);
> __m128i __lsx_vssrarni_b_h (__m128i, __m128i, imm0_15);
> __m128i __lsx_vssrarni_bu_h (__m128i, __m128i, imm0_15);
> -__m128i __lsx_vssrarni_d_q (__m128i, __m128i, imm0_127)
> -__m128i __lsx_vssrarni_du_q (__m128i, __m128i, imm0_127)
> +__m128i __lsx_vssrarni_d_q (__m128i, __m128i, imm0_127);
> +__m128i __lsx_vssrarni_du_q (__m128i, __m128i, imm0_127);
> __m128i __lsx_vssrarni_hu_w (__m128i, __m128i, imm0_31);
> __m128i __lsx_vssrarni_h_w (__m128i, __m128i, imm0_31);
> __m128i __lsx_vssrarni_w_d (__m128i, __m128i, imm0_63);
> @@ -18208,8 +18204,8 @@ __m128i __lsx_vssrln_hu_w (__m128i, __m128i);
> __m128i __lsx_vssrln_h_w (__m128i, __m128i);
> __m128i __lsx_vssrlni_b_h (__m128i, __m128i, imm0_15);
> __m128i __lsx_vssrlni_bu_h (__m128i, __m128i, imm0_15);
> -__m128i __lsx_vssrlni_d_q (__m128i, __m128i, imm0_127)
> -__m128i __lsx_vssrlni_du_q (__m128i, __m128i, imm0_127)
> +__m128i __lsx_vssrlni_d_q (__m128i, __m128i, imm0_127);
> +__m128i __lsx_vssrlni_du_q (__m128i, __m128i, imm0_127);
> __m128i __lsx_vssrlni_hu_w (__m128i, __m128i, imm0_31);
> __m128i __lsx_vssrlni_h_w (__m128i, __m128i, imm0_31);
> __m128i __lsx_vssrlni_w_d (__m128i, __m128i, imm0_63);
> @@ -18222,8 +18218,8 @@ __m128i __lsx_vssrlrn_hu_w (__m128i, __m128i);
> __m128i __lsx_vssrlrn_h_w (__m128i, __m128i);
> __m128i __lsx_vssrlrni_b_h (__m128i, __m128i, imm0_15);
> __m128i __lsx_vssrlrni_bu_h (__m128i, __m128i, imm0_15);
> -__m128i __lsx_vssrlrni_d_q (__m128i, __m128i, imm0_127)
> -__m128i __lsx_vssrlrni_du_q (__m128i, __m128i, imm0_127)
> +__m128i __lsx_vssrlrni_d_q (__m128i, __m128i, imm0_127);
> +__m128i __lsx_vssrlrni_du_q (__m128i, __m128i, imm0_127);
> __m128i __lsx_vssrlrni_hu_w (__m128i, __m128i, imm0_31);
> __m128i __lsx_vssrlrni_h_w (__m128i, __m128i, imm0_31);
> __m128i __lsx_vssrlrni_w_d (__m128i, __m128i, imm0_63);
> @@ -18238,12 +18234,12 @@ __m128i __lsx_vssub_h (__m128i, __m128i);
> __m128i __lsx_vssub_hu (__m128i, __m128i);
> __m128i __lsx_vssub_w (__m128i, __m128i);
> __m128i __lsx_vssub_wu (__m128i, __m128i);
> -void __lsx_vst (__m128i, void *, imm_n2048_2047)
> -void __lsx_vstelm_b (__m128i, void *, imm_n128_127, idx);
> -void __lsx_vstelm_d (__m128i, void *, imm_n128_127, idx);
> -void __lsx_vstelm_h (__m128i, void *, imm_n128_127, idx);
> -void __lsx_vstelm_w (__m128i, void *, imm_n128_127, idx);
> -void __lsx_vstx (__m128i, void *, long int)
> +void __lsx_vst (__m128i, void *, imm_n2048_2047);
> +void __lsx_vstelm_b (__m128i, void *, imm_n128_127, imm0_15);
> +void __lsx_vstelm_d (__m128i, void *, imm_n128_127, imm0_1);
> +void __lsx_vstelm_h (__m128i, void *, imm_n128_127, imm0_7);
> +void __lsx_vstelm_w (__m128i, void *, imm_n128_127, imm0_3);
> +void __lsx_vstx (__m128i, void *, long int);
> __m128i __lsx_vsub_b (__m128i, __m128i);
> __m128i __lsx_vsub_d (__m128i, __m128i);
> __m128i __lsx_vsub_h (__m128i, __m128i);
prev parent reply other threads:[~2023-12-21 9:44 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-12-13 15:26 Jiajie Chen
[not found] ` <718171e7-3787-b28b-a816-2d644976e09e@loongson.cn>
2023-12-15 2:21 ` Fwd: " chenxiaolong
2023-12-19 7:34 ` chenglulu
2023-12-21 9:44 ` chenglulu [this message]
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=3cae32b5-4ab6-31c9-d82a-e2f0451f27db@loongson.cn \
--to=chenglulu@loongson.cn \
--cc=c@jia.je \
--cc=gcc-patches@gcc.gnu.org \
--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).