[vector] Remove unnecessary node AbsV

Vladimir Ivanov vladimir.x.ivanov at oracle.com
Mon Apr 13 08:35:52 UTC 2020


Hi Qi,

> We already have nodes AbsVB/AbsVI/AbsVD/AbsVF/AbsVS/AbsVL for every primitive
> type. These nodes can cover all situations. So AbsV can be removed.
> 
> Please help me review this patch.
> 
> Webrev: http://cr.openjdk.java.net/~qfeng/panama/vectorapi/remove_absv/webrev/

Looks good.

In the longer term, I prefer us to move in the opposite direction: 
replace the family of type-specific nodes per operation with a single 
generic node and during matching rely on vector type info 
(n->bottom_type()->is_vect()) to choose proper implementation.

Best regards,
Vladimir Ivanov


More information about the panama-dev mailing list