[vectorIntrinsics] RFR: 8265321: Add Rearrange nodes implementation for Arm SVE

[Xiaohong Gong]

On Mon, 19 Apr 2021 08:18:14 GMT, Ningsheng Jian <njian at openjdk.org> wrote:

>>> > * I think that the comment is the node's comment (or rule's comment)  instead of single instruction's comment. This rule is `rearrangeL` which means rearranging long type so I think that the comment should be `L` instead of `D`.
>>> > * In `aarch64_neon.ad`, the comments are ` B`I`F`D `` and so on.
>>> 
>>> I assume you mean comment in format section. In aarch64_neon, some are S while some are F, but I don't think there's a need to generate two rules just for different format comments. If there's any, I would suggest to fix them as well.
>> 
>> Of course, in some case `L` and `D` can be zipped in one rule. However, in some case, `L` and `D` is difference. `D` is float type.  That is what I mentioned before `the comment is for the whole rule`.  In the comment, we should show the data type. ;-)
>
> Some cases are different, but I don't think we need to distinguish floating point data types for this operation as they are actually the same thing and generate the same code.

Agree with @nsjian . I prefer to merge all these rules into one as we have discussed too much times before, as there is no instruction difference for different types.  Anyway, we can have a separate patch to unify the styles in ad file.

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

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