[aarch64-port-dev ] RFR: 8169697: aarch64: vectorized MLA instruction not generated for some test cases

[Vladimir Kozlov]

On 12/8/16 5:56 PM, Ningsheng Jian wrote:
> Hi Roland,
>
> Thanks for the review.
>
>>> NEON vector MLA instructions can not be generated in some simple
>>> multiply-add cases.
>>>
>>> Bug:
>>> https://bugs.openjdk.java.net/browse/JDK-8169697
>>>
>>
>> This is a bug fix and bug fixes can still be pushed without restrictions
>> so I see no reason to not go with this:
>>
>>> In the mean time, Yang also has a more generic fix for this issue,
>>> which patches share code and could also fix this issue:
>>>
>>> http://cr.openjdk.java.net/~njian/8169697/webrev.share/
>>
>> That looks good to me. Anyone on the compiler side can take a look (and
>> sponsor)?
>
> Compared to aarch64 only version, I also prefer this patch.

What about SubV* and MulV* nodes?

I prefer shared code change but we would need to test on all platforms 
which support vectors.

Thanks,
Vladimir

>
> Thanks,
> Ningsheng
>