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