https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88560

--- Comment #6 from Wilco <wilco at gcc dot gnu.org> ---
(In reply to Vladimir Makarov from comment #5)
> We have too many tests checking expected generated code.  We should more
> focus on overall effect of the change.  SPEC would be a good criterium
> although it is hard to check SPEC for each patch.
> 
> I've checked the generated code of arm8_2-fp16-move-1.c and found that in
> most cases GCC generates better code with the patch:
> 
> @@ -80,7 +80,6 @@ test_load_store_1:
>         @ frame_needed = 0, uses_anonymous_args = 0
>         @ link register save eliminated.
>         lsl     r1, r1, #1
> -       vmov.f16        s0, r3  @ __fp16
>         ldrh    r3, [r2, r1]    @ __fp16
>         strh    r3, [r0, r1]    @ __fp16
>         bx      lr

When I tested it, this test added that vmov, not removed it - see comment #2.

Reply via email to