RFR: 8343689: AArch64: Optimize MulReduction implementation [v8]
Andrew Haley
aph at openjdk.org
Sat Aug 9 07:32:15 UTC 2025
On Fri, 8 Aug 2025 15:13:40 GMT, Mikhail Ablakatov <mablakatov at openjdk.org> wrote:
>> Sorry, the unpredicated version should be in the _SVE Integer Misc - Unpredicated_ section.
>
> Are you asking to move it to another existing section in the file or create a new one? If it's the former, could you point me to the section in the file - I can see neither `sve_ftssel` nor `sve_fexpa` defined. If the latter, in Arm ARM *C4.1.41 SVE Integer Misc - Unpredicated* is followed by *C4.1.42 SVE Element Count*, so the patch places `sve_movprfx` definition right before `sve_cnt*`; I also don't see an opportunity to define an `INSN` for this section as encodings of the instructions within the section do not follow a single pattern.
>
> If it's something else completely, please elaborate.
Please try to organize things the same way as the Decode section of the ARM.
Insert a new section called _SVE Integer Misc - Unpredicated_ after _SVE bitwise shift by immediate (predicated)_ and put this pattern there.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23181#discussion_r2264572038
More information about the hotspot-compiler-dev
mailing list