Hao Sun
Hao Sun
@nick-arm Could you help take a look at this patch when you have spare time? Thanks.
Thanks for your reviews! @theRealAph and @nick-arm /integrate
I wonder if we can remove the implicit `= noreg` arguments for `load_sized_value()` and `store_sized_value()` as well?
> > I wonder if we can remove the implicit `= noreg` arguments for `load_sized_value()` and `store_sized_value()` as well? > > The argument can be removed all together. `Register dst2`...
Hi, 1) For the following two files, the copyright year should be updated to 2022. ``` src/hotspot/share/opto/phasetype.hpp test/hotspot/jtreg/compiler/lib/ir_framework/driver/FlagVMProcess.java ``` 2) I tested this PR on one SVE supporting machine and...
Hi, I tested this PR based on the latest code in master branch, and found the following IR verification error on AArch64 machine. ``` test/hotspot/jtreg/compiler/c2/irTests/TestVectorConditionalMove.java:178: error: cannot find symbol @IR(failOn...
> > The build fails on Windows: > > Argh! Thanks. I'm afraid the compiler warnings on Windows still exist. See the GHA test results. Besides, the copyright year of...
Hi. Not a review. Here lists our internal test result. ## 1. VM crashed for arm32 I created one armhf build via cross-compilation and ran `java --version` via Qemu. Here...
> @shqking I hope the arm build will now work ok. Thanks for your fix. Yes. It works now.
Thanks for your explanation. Sounds reasonable to me. @Bhavana-Kilambi