[14] RFR (L): 8235756: C2: Merge AD instructions for DivV, SqrtV, FmaV, AddReductionV, and MulReductionV nodes

Vladimir Ivanov vladimir.x.ivanov at oracle.com
Thu Dec 12 10:22:06 UTC 2019


Thanks for the reviews, Vladimir & John.

> May be you need to separate Reduction changes into a separate patch so 
> we can have more time to review it. It is more complex.

Sure, I filed JDK-8235824 [1] and removed all reduction-related changes 
from current patch.

Best regards,
Vladimir Ivanov

[1] https://bugs.openjdk.java.net/browse/JDK-8235824

> On 12/11/19 3:53 AM, Vladimir Ivanov wrote:
>> http://cr.openjdk.java.net/~vlivanov/jbhateja/8235756/webrev.00/all/
>> https://bugs.openjdk.java.net/browse/JDK-8235756
>>
>> Merge AD instructions for the following vector nodes:
>>    - DivVF/DivVD
>>    - SqrtVF/SqrtVD
>>    - FmaVF/FmaVD
>>    - AddReductionV*
>>    - MulReductionV*
>>
>> Individual patches:
>>
>> http://cr.openjdk.java.net/~vlivanov/jbhateja/8235756/webrev.00/individual 
>>
>>
>> Testing: tier1-4, test run on different CPU flavors (KNL, SKL, etc)
>>
>> Contributed-by: Jatin Bhateja <jatin.bhateja at intel.com>
>> Reviewed-by: vlivanov, sviswanathan, ?
>>
>> Best regards,
>> Vladimir Ivanov


More information about the hotspot-compiler-dev mailing list