On Fri, 13 Sep 2024 19:14:29 GMT, Sandhya Viswanathan <sviswanat...@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
