RFR: 8342103: C2 compiler support for Float16 type and associated operations

Hao Sun haosun at openjdk.org
Mon Nov 25 06:24:24 UTC 2024


On Mon, 25 Nov 2024 01:07:43 GMT, Hao Sun <haosun at openjdk.org> wrote:

> > Hi @shqking , thanks for your review. I am currently working on adding the aarch64 port for these operations. It's being done here - [jatin-bhateja#6](https://github.com/jatin-bhateja/jdk/pull/6). Do you think it's ok to keep the code (regarding aarch64) in this patch as is for some more time until my patch is rebased and merged?
> 
> Hi @Bhavana-Kilambi , I would suggest making this patch as a clean one, i.e. better to move AArch64 related code to as one separate PR mainly because it may still take some time to review/merge your patch and we'd better **not** merge this PR with known jtreg failure. I noticed @jatin-bhateja has uploaded the cleanup commit and I will check the jtreg test on AArch64+SVE side. Will report the result back when the test finishes.

Previous test failure in file `TestFloat16VectorOperations.java` is gone now.
tier1~3 passed on AArch64+SVE side.

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

PR Comment: https://git.openjdk.org/jdk/pull/21490#issuecomment-2496961595


More information about the hotspot-compiler-dev mailing list