Re: RFR: 8314117: RISC-V: Incorrect VMReg encoding in RISCV64Frame.java

2023-08-14 Thread Feilong Jiang
On Mon, 14 Aug 2023 09:53:04 GMT, Vladimir Kempik wrote: > Please consider backporting this to 21u/17u if they are affected Sure, I'll backport it later. - PR Comment: https://git.openjdk.org/jdk/pull/15226#issuecomment-1677343419

Re: RFR: 8314117: RISC-V: Incorrect VMReg encoding in RISCV64Frame.java

2023-08-14 Thread Vladimir Kempik
On Thu, 10 Aug 2023 14:01:25 GMT, Feilong Jiang wrote: > Hi, inspired by [JDK-8247351](https://bugs.openjdk.org/browse/JDK-8247351), > this patch fixes a VMReg encoding issue in `RISCV64Frame.java`. As the VMReg > has two slots in a 64-bit VM, the encoding of `fp` should be `8 << 1` instead >

Re: RFR: 8314117: RISC-V: Incorrect VMReg encoding in RISCV64Frame.java

2023-08-14 Thread Feilong Jiang
On Fri, 11 Aug 2023 06:50:59 GMT, Fei Yang wrote: >> Hi, inspired by [JDK-8247351](https://bugs.openjdk.org/browse/JDK-8247351), >> this patch fixes a VMReg encoding issue in `RISCV64Frame.java`. As the VMReg >> has two slots in a 64-bit VM, the encoding of `fp` should be `8 << 1` >> instead

Re: RFR: 8314117: RISC-V: Incorrect VMReg encoding in RISCV64Frame.java

2023-08-11 Thread Fei Yang
On Thu, 10 Aug 2023 14:01:25 GMT, Feilong Jiang wrote: > Hi, inspired by [JDK-8247351](https://bugs.openjdk.org/browse/JDK-8247351), > this patch fixes a VMReg encoding issue in `RISCV64Frame.java`. As the VMReg > has two slots in a 64-bit VM, the encoding of `fp` should be `8 << 1` instead >

RFR: 8314117: RISC-V: Incorrect VMReg encoding in RISCV64Frame.java

2023-08-10 Thread Feilong Jiang
Hi, this patch fixes a VMReg encoding issue in `RISCV64Frame.java`. As the VMReg has two slots in a 64-bit VM, the encoding of `fp` should be `8 << 1` instead of `8`. Testing: - [x] tier1 with release build - [ ] tier2 with release build - [ ] tier3 with release build - Commit mess