[Aarch64][SVE] Add copysign and xorsign support

Alejandro Martinez Vicente Alejandro.MartinezVicente@arm.com
Wed Jan 9 12:06:00 GMT 2019


Hi,

I updated the patch to address Wilco's comment and style issues.

Alejandro


> -----Original Message-----
> From: Wilco Dijkstra <Wilco.Dijkstra@arm.com>
> Sent: 08 January 2019 16:58
> To: GCC Patches <gcc-patches@gcc.gnu.org>; Alejandro Martinez Vicente
> <Alejandro.MartinezVicente@arm.com>
> Cc: nd <nd@arm.com>; Richard Sandiford <Richard.Sandiford@arm.com>
> Subject: Re: [Aarch64][SVE] Add copysign and xorsign support
> 
> Hi Alejandro,
> 
> +    emit_move_insn (mask,
> +		    aarch64_simd_gen_const_vector_dup
> (<V_INT_EQUIV>mode,
> +						       HOST_WIDE_INT_M1U
> +						       << bits));
> +
> +    emit_insn (gen_and<v_int_equiv>3 (sign, arg2, mask));
> 
> Is there a reason to emit separate moves and then requiring the optimizer to
> combine them? The result of aarch64_simd_gen_const_vector_dup can be
> used directly in the gen_and for all supported floating point types.
> 
> Cheers,
> Wilco
-------------- next part --------------
A non-text attachment was scrubbed...
Name: copysign_2.patch
Type: application/octet-stream
Size: 9022 bytes
Desc: copysign_2.patch
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20190109/c3e223eb/attachment.obj>


More information about the Gcc-patches mailing list