RFR: 8288300: AArch64: Remove the assertion in fmovs/fmovd(FloatRegister, FloatRegister) [v2]

[Hao Sun]

On Wed, 15 Jun 2022 06:53:25 GMT, Andrew Haley <aph at openjdk.org> wrote:

>> Thanks for your review.
>> It's a style issue. I added one whitespace for `fmov/fabs/fneg` and the later `fcvt` so as to align the arguments to those of `fsqrt`. I saw such a style in several other sites in this header.
>> 
>> If you don't like it, I can remove the added whitespace. But I guess you may also want me to remove the whitespace I added for the `fcvt` below.
>
> I see. You're right, we're not consistent about this in the file.
> 
>  There are advantages and disadvantages. On the one hand it's a little easier to read, but on the other it's more work to maintain and tends to get eroded over time.
> 
> Whitespace-only changes as part of a larger patch can be hard to review , because they ca hide the substantive changes.
> 
> Please take out the added whitespace in this patch, and submit just the substantive changes. I will think about what to do to make this file consistent.

Updated in the latest revision. Thanks.

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

PR: https://git.openjdk.org/jdk/pull/9163