RFR: 8262916: Merge LShiftCntV and RShiftCntV into a single node
Vladimir Ivanov
vlivanov at openjdk.java.net
Thu Apr 8 11:08:25 UTC 2021
On Thu, 8 Apr 2021 05:14:52 GMT, Dean Long <dlong at openjdk.org> wrote:
> You should be able to do this without introducing a new node type. You could change the shift rules to match a vector register like x86.ad and aarch64_sve.ad already do.
Not sure what you refer to in x86.ad: vector shifts with variable scalar count require the scalar to be placed in XMM register. `ShiftCntV` handles register-to-register move between different register classes (`RegI` and `Vec*`).
Do you suggest reusing some existing vector node (like `Replicate`) to covert the scalar index to vector first? It would have slightly different behavior on x86.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3371
More information about the hotspot-compiler-dev
mailing list