[lworld+vector] RFR: Merge lworld [v2]

Xiaohong Gong xgong at openjdk.org
Thu Aug 31 06:38:29 UTC 2023


On Thu, 31 Aug 2023 06:10:43 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

> Hi @XiaohongGong ,
> 
> I verified that validation status is intact and other failures with additional -XX:+DeoptimizeALot option are also fixed with the merge.
> 
> Best Regards, Jatin

Thanks for the review and testing @jatin-bhateja ! Did you see any other regressions on Vector API tests with this option? I also ran the tests with `-XX:+DeoptimizeALot`, but some regressions that I met before are still there on NEON.  I remembered that some x86 specific code was changed (i.e. https://github.com/openjdk/valhalla/commit/d9f744fe6e816a91aa7c8d5e84ce464e8a9d3921#diff-a47d36f3d2e8997bd3d2320ef348dd17b91f388d2cb6e91e9d99d66df5ccd897), which is missing on AArch64 platform now. Seems this can also influence the result in interpreter, right? I will have a test by adding the similar code on AArch64 again. 

Best Regards,
Xiaohong

-------------

PR Comment: https://git.openjdk.org/valhalla/pull/921#issuecomment-1700445882



More information about the valhalla-dev mailing list