This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [PATCH 2/4][AArch64] Add support for FCCMP
- From: "Wilco Dijkstra" <Wilco dot Dijkstra at arm dot com>
- To: "'Evandro Menezes'" <e dot menezes at samsung dot com>
- Cc: <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 13 Nov 2015 17:36:08 -0000
- Subject: RE: [PATCH 2/4][AArch64] Add support for FCCMP
- Authentication-results: sourceware.org; auth=none
- References: <000101d11e2c$ae349150$0a9db3f0$ at arm dot com> <5646158B dot 6030001 at samsung dot com>
> 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?
Wilco