[vectorIntrinsics] RFR: 8258072: Add Reinterpret nodes implementation for Arm SVE [v2]

Wang Huang whuang at openjdk.java.net
Wed Dec 16 08:27:07 UTC 2020


On Wed, 16 Dec 2020 08:02:42 GMT, Xiaohong Gong <xgong at openjdk.org> wrote:

>> Wang Huang has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR.
>
> src/hotspot/cpu/aarch64/aarch64_sve.ad line 252:
> 
>> 250: // All SVE instructions
>> 251: 
>> 252: // vector load/store/reinterpret
> 
> Can you please add a separate comment for "reinterpret" before the related rules?

This comment is copied from neon implementation. We will add a  separate comment for "reinterpret"  in revised patch.

> src/hotspot/cpu/aarch64/aarch64_sve.ad line 342:
> 
>> 340:   ins_pipe(pipe_slow);
>> 341: %}
>> 342: instruct reinterpret(vReg dst) %{
> 
> Could you please try to add one more line between these different rules?

ok. Thanks.

-------------

PR: https://git.openjdk.java.net/panama-vector/pull/29


More information about the panama-dev mailing list