RFR: 8283091: Support type conversion between different data sizes in SLP [v3]
Vladimir Kozlov
kvn at openjdk.java.net
Fri May 27 22:21:40 UTC 2022
On Wed, 25 May 2022 01:13:36 GMT, Fei Gao <fgao at openjdk.org> wrote:
>> @fg1417 Thank you for suggesting this optimization. I see that it was not updated for some time. Do you still intend to work on it?
>>
>> Please update to latest JDK (Loom was integrated) and run performance again. Also include % of changes.
>>
>> I have the same concern as @DamonFool about regression when vectorizing some conversions. May be we should have additional `Matcher` property we could consult when trying to **auto-vectorize**. I understand that we need `vcvt2Dto2I` when VectorAPI specifically asking to generate it but we should not enforce auto-generation.
>
>> Please update to latest JDK (Loom was integrated) and run performance again. Also include % of changes.
>>
>> I have the same concern as @DamonFool about regression when vectorizing some conversions. May be we should have additional `Matcher` property we could consult when trying to **auto-vectorize**. I understand that we need `vcvt2Dto2I` when VectorAPI specifically asking to generate it but we should not enforce auto-generation.
>
> @vnkozlov thanks for your review and kind suggestion! I'll update the patch to resolve the potential performance regression.
@fg1417 I don't see new update in this PR. Please also show performance numbers with new changes
-------------
PR: https://git.openjdk.java.net/jdk/pull/7806
More information about the hotspot-compiler-dev
mailing list