RFR: 8300865: C2: product reduction in ProdRed_Double is not vectorized [v4]
Sandhya Viswanathan
sviswanathan at openjdk.org
Wed May 31 17:01:01 UTC 2023
On Wed, 31 May 2023 00:47:56 GMT, Sandhya Viswanathan <sviswanathan at openjdk.org> wrote:
>> This PR fixes the problem with double reduction on x86_64.
>>
>> In the test compiler.loopopts.superword.ProdRed_Double, the product reduction loop in prodReductionImplement() was not getting vectorized when run as follows:
>> jtreg -XX:CompileCommand=PrintAssembly,compiler.loopopts.superword.ProdRed_Double::prodReductionImplement compiler/loopopts/superword/ProdRed_Double.java
>> The print assembly generated in the pid-xxx.log output in JTwork/scratch directory was not showing any vector_reduction_double node.
>>
>> This was happening as the ReductionNode::implemented was passed a vector size of one element. For the vector reduction implemented we need to check with at least vector size of two elements.
>>
>> With this PR the vector_reduction_double node is generated.
>>
>> Please review.
>>
>> Best Regards,
>> Sandhya
>
> Sandhya Viswanathan has updated the pull request incrementally with one additional commit since the last revision:
>
> Add jmh test case
@vnkozlov Could you also please review this PR?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/14065#issuecomment-1570589226
More information about the hotspot-compiler-dev
mailing list