RFR: 8282528: AArch64: Incorrect replicate2L_zero rule [v2]
Eric Liu
eliu at openjdk.java.net
Wed Apr 6 03:38:27 UTC 2022
> This patch fixes the wrong matching rule of replicate2L_zero. It was
> matched "ReplicateI" by mistake so that long immediates(not only zero)
> had to be moved to register first and matched to replicate2L finally. To
> fix this trivial bug, this patch fixes the typo and extends the rule of
> replicate2L_zero to replicate2L_imm, which now supports all possible
> long immediate values.
>
> The final code changes are shown as below:
>
> replicate2L_imm:
>
> mov x13, #0xff
> movk x13, #0xff, lsl #16
> movk x13, #0xff, lsl #32
> dup v16.2d, x13
>
> =>
>
> movi v16.2d, #0xff00ff00ff
>
> [Test]
> test/jdk/jdk/incubator/vector, test/hotspot/jtreg/compiler/vectorapi
> passed without failure.
Eric Liu has updated the pull request incrementally with one additional commit since the last revision:
fix comment
Change-Id: Ic51820391d19b61e37847cc04375ecd79fc86779
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/7939/files
- new: https://git.openjdk.java.net/jdk/pull/7939/files/a5201990..aa66f34b
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7939&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7939&range=00-01
Stats: 105 lines in 3 files changed: 49 ins; 41 del; 15 mod
Patch: https://git.openjdk.java.net/jdk/pull/7939.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/7939/head:pull/7939
PR: https://git.openjdk.java.net/jdk/pull/7939
More information about the hotspot-dev
mailing list