[PATCH 2/4][AArch64] Add support for FCCMP

Andrew Pinski pinskia@gmail.com
Fri Nov 13 21:47:00 GMT 2015


On Sat, Nov 14, 2015 at 1:36 AM, Wilco Dijkstra <Wilco.Dijkstra@arm.com> wrote:
>> Evandro Menezes wrote:
>> Hi, Wilco.
>>
>> It looks good to me, but FCMP is quite different from FCCMP on Exynos M1,
>> so it'd be helpful to have distinct types for them. Say, "fcmp{s,d}"
>> and "fccmp{s,d}".  Would it be acceptable to add this with this patch or later?
>
> It would be easy to add fccmps/d as new attributes, I prefer to do that as a separate
> patch. Are there any other attributes that you think are missing besides the ones I know
> about (extr, 64-bit mul/mla, mulh, bfi)?
>
> Also would we need a new entry in the cost tables?


For ThunderX they can treated as the same.


Thanks,
Andrew

>
> Wilco
>
>



More information about the Gcc-patches mailing list