Re: RFR: 8337713: RISC-V: fix typos in macroAssembler_riscv.cpp

2024-10-06 Thread SendaoYan
On Sat, 5 Oct 2024 07:49:37 GMT, SendaoYan wrote: > Hi all, > This PR fix some typos bugs for RISC-V, and a typo bug in test directory. > Trivial fix, no risk. Thanks all for the review. - PR Comment: https://git.openjdk.org/jdk/pull/21368#issuecomment-2395419393

Re: RFR: 8337713: RISC-V: fix typos in macroAssembler_riscv.cpp

2024-10-06 Thread Fei Yang
On Sat, 5 Oct 2024 07:49:37 GMT, SendaoYan wrote: > Hi all, > This PR fix some typos bugs for RISC-V, and a typo bug in test directory. > Trivial fix, no risk. Marked as reviewed by fyang (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/21368#pullrequestreview-235050737

Re: RFR: 8337713: RISC-V: fix typos in macroAssembler_riscv.cpp

2024-10-06 Thread SendaoYan
On Sun, 6 Oct 2024 05:06:31 GMT, Julian Waters wrote: >> Hi all, >> This PR fix some typos bugs for RISC-V, and a typo bug in test directory. >> Trivial fix, no risk. > > src/hotspot/cpu/riscv/methodHandles_riscv.cpp line 447: > >> 445: } >> 446: } >> 447: > > This whitespace deletion

Re: RFR: 8337713: RISC-V: fix typos in macroAssembler_riscv.cpp

2024-10-05 Thread Julian Waters
On Sat, 5 Oct 2024 07:49:37 GMT, SendaoYan wrote: > Hi all, > This PR fix some typos bugs for RISC-V, and a typo bug in test directory. > Trivial fix, no risk. Looks good regardless src/hotspot/cpu/riscv/methodHandles_riscv.cpp line 447: > 445: } > 446: } > 447: This whitespace dele

RFR: 8337713: RISC-V: fix typos in macroAssembler_riscv.cpp

2024-10-05 Thread SendaoYan
Hi all, This PR fix some typos bugs for RISC-V, and a typo bug in test directory. Trivial fix, no risk. - Commit messages: - 8337713: RISC-V: fix typos in macroAssembler_riscv.cpp Changes: https://git.openjdk.org/jdk/pull/21368/files Webrev: https://webrevs.openjdk.org/?repo=jd