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

Evandro Menezes e.menezes@samsung.com
Wed Jan 20 23:10:00 GMT 2016


On 01/06/16 14:44, Evandro Menezes wrote:
> Hi, Wilco.
>
> On 01/06/2016 06:04 AM, Wilco Dijkstra wrote:
>>> Here's what I had in mind when I inquired about distinguishing FCMP 
>>> from
>>> FCCMP.  As you can see in the patch, Exynos is the only target that
>>> cares about it, but I wonder if ThunderX or Xgene would too.
>>>
>>> What do you think?
>> The new attributes look fine (I've got a similar outstanding change), 
>> however
>> please don't add them to non-AArch64 cores. We only need it for 
>> thunderx.md,
>> cortex-a53.md, cortex-a57.md, xgene1.md and exynos-m1.md.
>
>         Add support for the FCCMP insn types
>
>         2016-01-04  Evandro Menezes  <e.menezes@samsung.com>
>
>         gcc/
>             * config/aarch64/aarch64.md (fccmp): Change insn type.
>             (fccmpe): Likewise.
>             * config/aarch64/thunderx.md (thunderx_fcmp): Add
>    "fccmp{s,d}" types.
>             * config/arm/cortex-a53.md (cortex_a53_fpalu): Likewise.
>             * config/arm/cortex-a57.md (cortex_a57_fp_cmp): Likewise.
>             * config/arm/xgene1.md (xgene1_fcmp): Likewise.
>             * config/arm/exynos-m1.md (exynos_m1_fp_ccmp): New insn
>    reservation.
>             * config/arm/types.md (fccmps): Add new insn type.
>             (fccmpd): Likewise.
>
> Got it.  Here's an updated patch.  Again, assuming that your original 
> patch is in place.  Perhaps you can build on it.
>
> Thank you,
>

Ping.

-- 
Evandro Menezes



More information about the Gcc-patches mailing list