[aarch64-port-dev ] RFR: 8169697: aarch64: vectorized MLA instruction not generated for some test cases
Yang Zhang
yang.zhang at linaro.org
Tue Dec 20 10:02:51 UTC 2016
Hi Vladimir, Roland
The patch has been updated in
http://cr.openjdk.java.net/~njian/8169697/webrev.01/.
Could you please help to review it again?
Regards
Yang
On 12 December 2016 at 11:42, Yang Zhang <yang.zhang at linaro.org> wrote:
> Hi Vladimir,
>
> Thanks for your review.
>
>>
>> What about SubV* and MulV* nodes?
>>
>
> After checking commut_op_list, I think MulV*, AndV, OrV and XorV nodes
> can also be added. Should I add them all?
>
> SubV* nodes are not commutative, so I don't think they need to be
> considered here.
>
>>
>> I prefer shared code change but we would need to test on all platforms which support vectors.
>>
> Yeah, we need to test on all platforms. We already tested previous
> patch on x86 and aarch64 platforms, but we don't have other platforms.
>
> Regards
> Yang
More information about the aarch64-port-dev
mailing list