[PATCH][AArch64] Replace insn to zero up DF register
Wilco Dijkstra
Wilco.Dijkstra@arm.com
Fri Feb 26 12:37:00 GMT 2016
Evandro Menezes <e.menezes@samsung.com> wrote:
>
> I have a question though: is it necessary to add the "fp" and "simd"
> attributes to both movsf_aarch64 and movdf_aarch64 as well?
You need at least the "simd" attribute, but providing "fp" as well is clearer
(in principle the TARGET_FLOAT check in the pattern condition is
redundant as a result, but the movhf and movtf patterns already do both).
Also you want to use the smallest possible SIMD size as these are
scalar operations and some microarchitectures execute 64-bit operations
more efficiently than 128-bit ones, so:
mov\\t%0.h[0], %w1
+ movi\\t%0.4h, #0
umov\\t%w0, %1.h[0]
fmov\\t%s0, %w1
+ movi\\t%0.2s, #0
fmov\\t%w0, %s1
With those changes it should be ready for commit once you get the OK from James/Marcus.
Wilco
More information about the Gcc-patches
mailing list