RFR: 8264018: AArch64: NEON loadV2 and storeV2 addressing is wrong

Andrew Dinn adinn at openjdk.java.net
Wed Mar 24 10:29:41 UTC 2021


On Wed, 24 Mar 2021 10:24:04 GMT, Andrew Haley <aph at openjdk.org> wrote:

>> AArch64 c2 loadV2 and storeV2 are used for Vector API mask value load/store with size of halfword. Their load/store immediate offset should be similar to other vector load/store, e.g. loadV4/storeV4, valid only for unscaled signed offset of imm9 or scaled unsigned offset of imm12:scale. This patch also groups other loadVs/storeVs together in the ad file.
>> 
>> Verified with tier1 and jtreg vector api cases, though I cannot reproduce this issue.
>
> I admire your decision to tidy up the files, but I can't easily see what exactly has changed.

The change is to the memory operand of the first load and store rules. It was special cased to memory and is now vmem2 (just as the relocated rules used vmem4, vmem8 etc).

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

PR: https://git.openjdk.java.net/jdk/pull/3168


More information about the hotspot-compiler-dev mailing list