<div class="__aliyun_email_body_block"><div  style="line-height:1.7;font-family:Tahoma,Arial,STHeiti,SimSun;font-size:14.0px;color:#000000;"><div  style="clear:both;">Hi Vladimir,</div><div  style="clear:both;"><br ></div><div  style="clear:both;">Thank you for the suggestion and that sounds nice as well to me - more unified.</div><div  style="clear:both;"><br ></div><div  style="clear:both;">Pushed a new commit to fit your pre-review comment.</div><div  style="clear:both;"><br ></div><div  style="clear:both;">Best,</div><div  style="clear:both;">Xiaolin</div><div  style="clear:both;"><br ></div><div  style="clear:both;">[1] <span ><a  href="https://github.com/zhengxiaolinX/jdk/commit/312462e83ea3dcbd884e121ca16b2209b7a6c5c4" target="_blank">https://github.com/zhengxiaolinX/jdk/commit/312462e83ea3dcbd884e121ca16b2209b7a6c5c4</a></span></div><div  style="clear:both;"><br ></div><div  style="clear:both;"><br ></div><div  style="clear:both;"><br /></div><blockquote  style="margin-right:0;margin-top:0;margin-bottom:0;font-family:Tahoma,Arial,STHeiti,SimSun;font-size:14.0px;color:#000000;"><div  style="clear:both;">------------------------------------------------------------------</div><div  style="clear:both;">From:Vladimir Kempik <vladimir.kempik@gmail.com></div><div  style="clear:both;">Send Time:2022年11月7日(星期一) 17:05</div><div  style="clear:both;">To:郑孝林(云矅) <yunyao.zxl@alibaba-inc.com></div><div  style="clear:both;">Cc:riscv-port-dev <riscv-port-dev@openjdk.org></div><div  style="clear:both;">Subject:Re: RVC by default (cont'd)</div><div  style="clear:both;"><br /></div>Hello<br >Recently commit [1]  introduced support for cpu profiles, for example RVA20U64, RVA22U64.<br >And UseRVC is already a part of UseRVA20U64.<br ><br >Maybe it would be good to go another way. For example make  RVA20U64 non experimental and enable it by default, that will enable RVC automatically. Also make UseRVC non-experimental.<br ><br >Regards, Vladimir<br ><br >[1] https://github.com/openjdk/jdk/commit/e0c29307f7b35149aacae0bb935aa9fe524cff72<br ><br >> 7 нояб. 2022 г., в 07:17, Xiaolin Zheng <yunyao.zxl@alibaba-inc.com> написал(а):<br >> <br >> Hi team,<br >> <br >> As RVC's proposed patches have been merged into the mainline, in response to the former thread[1] I would like to turn it on by default before the December RDP 1 deadline, for currently the hardware feature C extension has been ratified and implemented by mainstream RISC-V hardware like boards produced by Hifive, meaning we can test and verify our implementation on physical boards.<br >> <br >> Opening another thread to refresh the content.<br >> <br >> We can turn RVC on for now by using `-XX:+UnlockExperimentalVMOptions -XX:+UseRVC`. In addition we can examine the generated code by using options `-XX:+UnlockExperimentalVMOptions -XX:+UseRVC -XX:+UnlockDiagnosticVMOptions -XX:+PrintAssembly -XX:PrintAssemblyOptions=no-aliases,numeric,show-bytes` combined.<br >> <br >> I have pushed a simple proposed patch[2] to turn it as default true.<br >> <br >> The only thing I shall mention here is as we know there is a known issue that may relate to the opensbi lib. Please see previous discussions[3][4]. The pattern of that issue is very easy to be distinguished, which is an uncommon case and which turns out to be bugs hidden in underlying libraries at last. I think it should be users' responsibility to update their outdated libs, and such issue shall not stop our pace.<br >> <br >> I have opened an JBS issue[5] to record this, marking it as "Won't fix".<br >> <br >> If there's any suggestion or objection, please let me know. If not, I will file a patch around Nov 15 (may be next week since the deadline is looming) if everything looks okay.<br >> <br >> Best Regards,<br >> Xiaolin<br >> <br >> [1] https://mail.openjdk.org/pipermail/riscv-port-dev/2022-September/000609.html<br >> [2] https://github.com/zhengxiaolinX/jdk/commit/b5b9c64529c27c40542f8cda720652fabf70682d<br >> [3] https://mail.openjdk.org/pipermail/riscv-port-dev/2022-September/000618.html<br >> [4] https://github.com/riscv-collab/riscv-openjdk/issues/23<br >> [5] https://bugs.openjdk.org/browse/JDK-8296350<br ></blockquote></div></div>