RFR: 8259948: Aarch64: Add cast nodes for Aarch64 Neon backend [v5]
Wu Yan
wuyan at openjdk.java.net
Tue Sep 28 01:57:07 UTC 2021
On Wed, 22 Sep 2021 07:58:40 GMT, Andrew Haley <aph at openjdk.org> wrote:
>>> One is _Advanced SIMD copy_, the other _Advanced SIMD scalar copy_. The _Advanced SIMD copy_ version is missing a comment to that effect. Maybe one day someone could go through assembler_aarch64.hpp and add the missing titles from the A64 instruction set encoding section of the ARM.
>>
>> Yes, a comment will be helpful. But for the scalar copy, we should use SIMD_RegVariant instead of SIMD_Arrangement?
>
> It looks like the official doc uses _SIMD_RegVariant_ for the scalar copy, so we should do the same. I don't think you'll ever find me suggesting we should deviate from what the Arm reference manuals do, unless we can't avoid it.
> Yes, a comment will be helpful. But for the scalar copy, we should use SIMD_RegVariant instead of SIMD_Arrangement?
Thanks, we have fixed it.
-------------
PR: https://git.openjdk.java.net/jdk/pull/4839
More information about the hotspot-compiler-dev
mailing list