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