This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCHv2/AARCH64] Remove index from AARCH64_FUSION_PAIR
- From: James Greenhalgh <james dot greenhalgh at arm dot com>
- To: Andrew Pinski <pinskia at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 19 Aug 2015 17:02:56 +0100
- Subject: Re: [PATCHv2/AARCH64] Remove index from AARCH64_FUSION_PAIR
- Authentication-results: sourceware.org; auth=none
- References: <CA+=Sn1kKn7qOC3c3d9157P_ohKdWjRHxOxVpzzeCrgejmBXn+A at mail dot gmail dot com>
On Wed, Aug 19, 2015 at 04:58:22PM +0100, Andrew Pinski wrote:
> Changes from v1:
> Also remove the hack AARCH64_FUSE_ALL.
>
> Instead of doing an explict index in aarch64-fusion-pairs.def, we
> should have an enum which does the index instead. This allows
> you to add/remove them without worrying about the order being
> correct and having holes or worry about merge conficts.
>
> OK? Bootstrapped and tested on aarch64-linux-gnu with no regressions.
OK.
Thanks,
James
>
> ChangeLog:
> * aarch64-fusion-pairs.def: Remove all index to AARCH64_FUSION_PAIR.
> * config/aarch64/aarch64-protos.h (aarch64_fusion_pairs_index): New enum.
> (aarch64_fusion_pairs): Base the shifted value on the index instead
> Rewrite AARCH64_FUSE_ALL to be based on the end index.
> of the argument to AARCH64_FUSION_PAIR.
> * config/aarch64/aarch64.c: Remove the last argument to AARCH64_FUSION_PAIR.