RFR: 8355585: Aarch64: Add aarch64 backend for Float16 vector operations [v4]

Bhavana Kilambi bkilambi at openjdk.org
Wed May 21 12:16:55 UTC 2025


On Wed, 21 May 2025 07:49:10 GMT, Bhavana Kilambi <bkilambi at openjdk.org> wrote:

>> Looks good. I'm assuming you've tested both SVE and Neon.
>
>> Looks good. I'm assuming you've tested both SVE and Neon.
> 
> Yes, this was tested on both SVE and Neon (N1/V1/V2 architectures).

> @Bhavana-Kilambi I'm getting timeouts with your new test: `compiler/vectorization/TestFloat16VectorOperations.java`
> 
> At least on `linux-aarch64-debug` and `windows-x64-debug`, but not all tests have completed yet, so more could be failing.
> 
> Not sure if it is relevant, but both had extra flag `-XX:-UseTLAB`, we add this flag in our additional stress testing.

Thanks for letting me know. This test by default takes a very long time to finish. As i added more flags (to test various vector sizes) to be tested, it probably ran way too long than anticipated and resulted in the timeout error. I will update with a patch to remove testing these extra flags for now. I did try to increase the default timeout value but the test continued to run for more than 20 min after which I had to terminate. I feel the best for now is to remove the additional tests for various vector sizes that I have added. I will update the patch soon.

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

PR Comment: https://git.openjdk.org/jdk/pull/25096#issuecomment-2897732641


More information about the hotspot-compiler-dev mailing list