RFR: 8282711: Accelerate Math.signum function for AVX and AVX512 target. [v6]
Jatin Bhateja
jbhateja at openjdk.java.net
Thu Apr 14 05:57:58 UTC 2022
On Wed, 13 Apr 2022 18:54:49 GMT, Sandhya Viswanathan <sviswanathan at openjdk.org> wrote:
>> Jatin Bhateja has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains seven commits:
>>
>> - 8282711: Review comments resolutions.
>> - Merge branch 'master' of http://github.com/openjdk/jdk into JDK-8282711
>> - Merge branch 'master' of http://github.com/openjdk/jdk into JDK-8282711
>> - 8282711: Replacing vector length based predicate.
>> - 8282711: Making the changes more generic (removing AVX512DQ restriction), adding new IR level test.
>> - 8282711: Review comments resolved.
>> - 8282711: Accelerate Math.signum function for AVX and AVX512 target.
>
> src/hotspot/cpu/x86/x86.ad line 6118:
>
>> 6116: match(Set dst (SignumVD src (Binary zero one)));
>> 6117: effect(TEMP dst, TEMP ktmp1);
>> 6118: format %{ "vector_signum_evex $dst, $src\t! using $one, $zero and $ktmp1 as TEMP" %}
>
> $one and $zero are inputs and not temps per the IR.
Corrected it.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7717
More information about the hotspot-compiler-dev
mailing list