RFR: 8340079: Modify rearrange/selectFrom Vector API methods to perform wrapIndexes instead of checkIndexes [v2]
Jatin Bhateja
jbhateja at openjdk.org
Sat Sep 14 09:11:06 UTC 2024
On Fri, 13 Sep 2024 19:14:29 GMT, Sandhya Viswanathan <sviswanathan at openjdk.org> wrote:
>> Hi @sviswa7, I was suggesting emitting toShuffle() + toVector() only if it's needed under a target specific hook, since indexes are anyways passed though vector. Please let me know if you find blow explanation too constraining.
>> https://github.com/openjdk/jdk/pull/20508#issuecomment-2349801299
>
> I think VectorLoadShuffle removal optimizations should be a separate PR and well thought out. So far the contract has been that rearrange always gets the shuffle through VectorLoadShuffle and I would like to keep that contract in this PR.
Hi @sviswa7, @PaulSandoz , I will modify PR#20508 accordingly to honor the contract at IR level and address VectorLoadShuffle optimization for both flavors of selectFrom API in a follow up PR.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20634#discussion_r1759701508
More information about the hotspot-compiler-dev
mailing list