Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics [v6]

2023-10-20 Thread Ilya Gavrilin
> Hi all, please review this changes into risc-v floating point copysign and > signum intrinsics. > CopySign - returns first argument with the sign of second. On risc-v we have > `fsgnj.x` instruction, which can implement this intrinsic. > Signum - returns input value if it is +/- 0.0 or NaN, oth

Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics [v3]

2023-10-19 Thread Hamlin Li
On Wed, 18 Oct 2023 17:35:58 GMT, Ilya Gavrilin wrote: >> Hi all, please review this changes into risc-v floating point copysign and >> signum intrinsics. >> CopySign - returns first argument with the sign of second. On risc-v we have >> `fsgnj.x` instruction, which can implement this intrinsic

Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics [v5]

2023-10-19 Thread Ilya Gavrilin
> Hi all, please review this changes into risc-v floating point copysign and > signum intrinsics. > CopySign - returns first argument with the sign of second. On risc-v we have > `fsgnj.x` instruction, which can implement this intrinsic. > Signum - returns input value if it is +/- 0.0 or NaN, oth

Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics [v4]

2023-10-19 Thread Ilya Gavrilin
On Thu, 19 Oct 2023 12:22:29 GMT, Fei Yang wrote: > Still good. You might want to correct the remaining typo. Thanks for review! - PR Comment: https://git.openjdk.org/jdk/pull/16186#issuecomment-1770876188

Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics [v4]

2023-10-19 Thread Ilya Gavrilin
On Thu, 19 Oct 2023 12:14:52 GMT, Ilya Gavrilin wrote: >> Hi all, please review this changes into risc-v floating point copysign and >> signum intrinsics. >> CopySign - returns first argument with the sign of second. On risc-v we have >> `fsgnj.x` instruction, which can implement this intrinsic

Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics [v4]

2023-10-19 Thread Fei Yang
On Thu, 19 Oct 2023 12:14:52 GMT, Ilya Gavrilin wrote: >> Hi all, please review this changes into risc-v floating point copysign and >> signum intrinsics. >> CopySign - returns first argument with the sign of second. On risc-v we have >> `fsgnj.x` instruction, which can implement this intrinsic

Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics [v4]

2023-10-19 Thread Vladimir Kempik
On Thu, 19 Oct 2023 12:14:52 GMT, Ilya Gavrilin wrote: >> Hi all, please review this changes into risc-v floating point copysign and >> signum intrinsics. >> CopySign - returns first argument with the sign of second. On risc-v we have >> `fsgnj.x` instruction, which can implement this intrinsic

Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics [v4]

2023-10-19 Thread Ilya Gavrilin
> Hi all, please review this changes into risc-v floating point copysign and > signum intrinsics. > CopySign - returns first argument with the sign of second. On risc-v we have > `fsgnj.x` instruction, which can implement this intrinsic. > Signum - returns input value if it is +/- 0.0 or NaN, oth

Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics [v3]

2023-10-18 Thread Fei Yang
On Wed, 18 Oct 2023 17:35:58 GMT, Ilya Gavrilin wrote: >> Hi all, please review this changes into risc-v floating point copysign and >> signum intrinsics. >> CopySign - returns first argument with the sign of second. On risc-v we have >> `fsgnj.x` instruction, which can implement this intrinsic

Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics [v3]

2023-10-18 Thread Ilya Gavrilin
> Hi all, please review this changes into risc-v floating point copysign and > signum intrinsics. > CopySign - returns first argument with the sign of second. On risc-v we have > `fsgnj.x` instruction, which can implement this intrinsic. > Signum - returns input value if it is +/- 0.0 or NaN, oth

Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics [v2]

2023-10-18 Thread Fei Yang
On Tue, 17 Oct 2023 09:52:15 GMT, Ilya Gavrilin wrote: >> src/hotspot/cpu/riscv/c2_MacroAssembler_riscv.cpp line 1682: >> >>> 1680: // use floating-point 1.0 with a sign of input >>> 1681: is_double ? fsgnj_d(dst, one, src) >>> 1682: : fsgnj_s(dst, one, src); >> >> What if the `

Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics [v2]

2023-10-17 Thread Ilya Gavrilin
On Tue, 17 Oct 2023 09:09:52 GMT, Fei Yang wrote: >> Ilya Gavrilin has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Fix some registers usages and typos > > src/hotspot/cpu/riscv/c2_MacroAssembler_riscv.cpp line 1682: > >> 1680: // use

Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics [v2]

2023-10-17 Thread Fei Yang
On Mon, 16 Oct 2023 21:14:39 GMT, Ilya Gavrilin wrote: >> Hi all, please review this changes into risc-v floating point copysign and >> signum intrinsics. >> CopySign - returns first argument with the sign of second. On risc-v we have >> `fsgnj.x` instruction, which can implement this intrinsic

Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics [v2]

2023-10-16 Thread Robbin Ehn
On Mon, 16 Oct 2023 21:14:39 GMT, Ilya Gavrilin wrote: >> Hi all, please review this changes into risc-v floating point copysign and >> signum intrinsics. >> CopySign - returns first argument with the sign of second. On risc-v we have >> `fsgnj.x` instruction, which can implement this intrinsic

Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics [v2]

2023-10-16 Thread Ilya Gavrilin
> Hi all, please review this changes into risc-v floating point copysign and > signum intrinsics. > CopySign - returns first argument with the sign of second. On risc-v we have > `fsgnj.x` instruction, which can implement this intrinsic. > Signum - returns input value if it is +/- 0.0 or NaN, oth

Re: RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics

2023-10-16 Thread Fei Yang
On Fri, 13 Oct 2023 15:36:56 GMT, Ilya Gavrilin wrote: > Hi all, please review this changes into risc-v floating point copysign and > signum intrinsics. > CopySign - returns first argument with the sign of second. On risc-v we have > `fsgnj.x` instruction, which can implement this intrinsic. >

RFR: 8317971: RISC-V: implement copySignF/D and signumF/D intrinsics

2023-10-13 Thread Ilya Gavrilin
Hi all, please review this changes into risc-v floating point copysign and signum intrinsics. CopySign - returns first argument with the sign of second. On risc-v we have `fsgnj.x` instruction, which can implement this intrinsic. Signum - returns input value if it is +/- 0.0 or NaN, otherwise 1.0