Integrated: 8290154: [JVMCI] partially implement JVMCI for RISC-V

2022-10-07 Thread Sacha Coppey
On Thu, 21 Jul 2022 10:18:05 GMT, Sacha Coppey wrote: > This patch adds a partial JVMCI implementation for RISC-V, to allow using the > GraalVM Native Image RISC-V LLVM backend, which does not use JVMCI for code > emission. > It creates the jdk.vm.ci.riscv64 and jdk.vm.ci.hot

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v13]

2022-10-07 Thread Sacha Coppey
ments a part of jvmciCodeInstaller_riscv64.cpp. To check for > correctness, it enables JVMCI code installation tests on RISC-V. More testing > is performed in Native Image. Sacha Coppey has updated the pull request incrementally with one additional commit since the last revision: Upda

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v10]

2022-10-06 Thread Sacha Coppey
On Thu, 6 Oct 2022 12:27:11 GMT, Fei Yang wrote: > The current version does not build. I will take another look after this is > rebased on the latest jdk master. Sorry for the delay, I rebased the PR and fixed the building issue. - PR: https://git.openjdk.org/jdk/pull/9587

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v12]

2022-10-06 Thread Sacha Coppey
ments a part of jvmciCodeInstaller_riscv64.cpp. To check for > correctness, it enables JVMCI code installation tests on RISC-V. More testing > is performed in Native Image. Sacha Coppey has updated the pull request incrementally with one additional commit since the last revision: Replace RegisterImpl

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v11]

2022-10-06 Thread Sacha Coppey
ments a part of jvmciCodeInstaller_riscv64.cpp. To check for > correctness, it enables JVMCI code installation tests on RISC-V. More testing > is performed in Native Image. Sacha Coppey has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains ten commits

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v10]

2022-10-05 Thread Sacha Coppey
On Wed, 14 Sep 2022 09:39:07 GMT, Sacha Coppey wrote: >> This patch adds a partial JVMCI implementation for RISC-V, to allow using >> the GraalVM Native Image RISC-V LLVM backend, which does not use JVMCI for >> code emission. >> It creates the jdk.vm.ci.riscv64 and j

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v10]

2022-09-14 Thread Sacha Coppey
ments a part of jvmciCodeInstaller_riscv64.cpp. To check for > correctness, it enables JVMCI code installation tests on RISC-V. More testing > is performed in Native Image. Sacha Coppey has updated the pull request incrementally with one additional commit since the last revision: Remove noinline att

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v7]

2022-08-25 Thread Sacha Coppey
On Wed, 24 Aug 2022 11:33:01 GMT, Fei Yang wrote: > > > I see you added more changes in hotspot file sharedRuntime_riscv.cpp > > > guarded by macro INCLUDE_JVMCI. Searching for INCLUDE_JVMCI or > > > COMPILER2_OR_JVMCI in src/hotspot/cpu/aarch64, I see several more places > > > checking for th

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v8]

2022-08-24 Thread Sacha Coppey
On Wed, 24 Aug 2022 08:08:51 GMT, Yadong Wang wrote: >> Sacha Coppey has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Add a comments for the change in deoptimization.hpp > > src/hotspot/cpu/riscv/jvmciCo

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v9]

2022-08-24 Thread Sacha Coppey
ments a part of jvmciCodeInstaller_riscv64.cpp. To check for > correctness, it enables JVMCI code installation tests on RISC-V. More testing > is performed in Native Image. Sacha Coppey has updated the pull request incrementally with one additional commit since the last revision: Remove vector registers

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v8]

2022-08-24 Thread Sacha Coppey
ments a part of jvmciCodeInstaller_riscv64.cpp. To check for > correctness, it enables JVMCI code installation tests on RISC-V. More testing > is performed in Native Image. Sacha Coppey has updated the pull request incrementally with one additional commit since the last revision: Add a comme

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v7]

2022-08-24 Thread Sacha Coppey
On Tue, 23 Aug 2022 11:29:45 GMT, Sacha Coppey wrote: >> This patch adds a partial JVMCI implementation for RISC-V, to allow using >> the GraalVM Native Image RISC-V LLVM backend, which does not use JVMCI for >> code emission. >> It creates the jdk.vm.ci.riscv64 and j

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v5]

2022-08-23 Thread Sacha Coppey
On Tue, 23 Aug 2022 07:59:32 GMT, Fei Yang wrote: > So I give it a second try with fastdebug build on my unmatched board [1], I > see the error has changed into [2]. Is there anything I missed? Thanks. I don't know why I did not get this assertion error on my end before. I fixed the issue and

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v7]

2022-08-23 Thread Sacha Coppey
ments a part of jvmciCodeInstaller_riscv64.cpp. To check for > correctness, it enables JVMCI code installation tests on RISC-V. More testing > is performed in Native Image. Sacha Coppey has updated the pull request incrementally with one additional commit since the last revision: Fix error when emi

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v5]

2022-08-22 Thread Sacha Coppey
On Mon, 22 Aug 2022 14:05:49 GMT, Fei Yang wrote: >> Sacha Coppey has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Add space in switch > > src/jdk.internal.vm.ci/share/classes/jdk.vm.ci.riscv64/src/jdk/

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v5]

2022-08-22 Thread Sacha Coppey
On Wed, 27 Jul 2022 11:25:22 GMT, Sacha Coppey wrote: >> This patch adds a partial JVMCI implementation for RISC-V, to allow using >> the GraalVM Native Image RISC-V LLVM backend, which does not use JVMCI for >> code emission. >> It creates the jdk.vm.ci.riscv64 and j

Re: RFR: 8290154: [JVMCI] partially implement JVMCI for RISC-V [v6]

2022-08-22 Thread Sacha Coppey
ments a part of jvmciCodeInstaller_riscv64.cpp. To check for > correctness, it enables JVMCI code installation tests on RISC-V. It will be > tested soon in Native Image as well. Sacha Coppey has updated the pull request incrementally with one additional commit since the last revision: Ensure all JVMCI

Re: RFR: 8290154: [JVMCI] Implement JVMCI for RISC-V [v4]

2022-07-27 Thread Sacha Coppey
On Tue, 26 Jul 2022 19:11:23 GMT, Andrey Turbanov wrote: >> Sacha Coppey has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Avoid using set_destination when call is not jal > > src/jdk.internal.vm.ci/share/cla

Re: RFR: 8290154: [JVMCI] Implement JVMCI for RISC-V [v5]

2022-07-27 Thread Sacha Coppey
ments a part of jvmciCodeInstaller_riscv64.cpp. To check for > correctness, it enables JVMCI code installation tests on RISC-V. It will be > tested soon in Native Image as well. Sacha Coppey has updated the pull request incrementally with one additional commit since the last revision:

Re: RFR: 8290154: [JVMCI] Implement JVMCI for RISC-V

2022-07-25 Thread Sacha Coppey
On Mon, 25 Jul 2022 07:30:08 GMT, Fei Yang wrote: > Hi, I see some JVM crash when I try the following test with fastdebug build > with your patch: > test/hotspot/jtreg/compiler/jvmci/jdk.vm.ci.code.test/src/jdk/vm/ci/code/test/SimpleDebugInfoTest.java Thank you for pointing this out, I did not

Re: RFR: 8290154: [JVMCI] Implement JVMCI for RISC-V [v4]

2022-07-25 Thread Sacha Coppey
It should be > tested soon in GraalVM Native Image as well. Sacha Coppey has updated the pull request incrementally with one additional commit since the last revision: Avoid using set_destination when call is not jal - Changes: - all: https://git.openjdk.org/jdk/pull/9587/files

Re: RFR: 8290154: [JVMCI] Implement JVMCI for RISC-V [v3]

2022-07-25 Thread Sacha Coppey
It should be > tested soon in GraalVM Native Image as well. Sacha Coppey has updated the pull request incrementally with one additional commit since the last revision: Use nativeInstruction_at instead of nativeCall_at to avoid wrongly initializating a call - Changes: - all: https://

Re: RFR: 8290154: [JVMCI] Implement JVMCI for RISC-V [v2]

2022-07-25 Thread Sacha Coppey
It should be > tested soon in GraalVM Native Image as well. Sacha Coppey has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since th

RFR: 8290154: [JVMCI] Implement JVMCI for RISC-V

2022-07-21 Thread Sacha Coppey
This patch adds a JVMCI implementation for RISC-V. It creates the jdk.vm.ci.riscv64 and jdk.vm.ci.hotspot.riscv64 packages, as well as implements a part of jvmciCodeInstaller_riscv64.cpp. To check for correctness, it enables JVMCI code installation tests on RISC-V. It should be tested soon in Gr