This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [PATCH][GCC][mid-end] Correct subreg no-op handling for big-endian vec_select.
- From: Tamar Christina <Tamar dot Christina at arm dot com>
- To: Tamar Christina <Tamar dot Christina at arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Cc: nd <nd at arm dot com>, "law at redhat dot com" <law at redhat dot com>, "rguenther at suse dot de" <rguenther at suse dot de>, "ian at airs dot com" <ian at airs dot com>
- Date: Wed, 27 Jun 2018 08:05:16 +0000
- Subject: RE: [PATCH][GCC][mid-end] Correct subreg no-op handling for big-endian vec_select.
- References: <20180619141104.GA28745@arm.com>
Ping.
> -----Original Message-----
> From: gcc-patches-owner@gcc.gnu.org <gcc-patches-owner@gcc.gnu.org>
> On Behalf Of Tamar Christina
> Sent: Tuesday, June 19, 2018 15:11
> To: gcc-patches@gcc.gnu.org
> Cc: nd <nd@arm.com>; law@redhat.com; rguenther@suse.de; ian@airs.com
> Subject: [PATCH][GCC][mid-end] Correct subreg no-op handling for big-
> endian vec_select.
>
> Hi All,
>
> Previously GCC's no-op detection could would consider something a no-op
> even when the mode change is not directly possible. This caused subregs
> that shouldn't be removed to be treated as a no-op and deleted.
>
> Regtested on armeb-none-eabi and no regressions.
> Bootstrapped on arm-none-linux-gnueabihf and no issues.
>
> Ok for trunk? and for backport to GCC 8?
>
> Thanks,
> Tamar
>
> gcc/
> 2018-06-19 Tamar Christina <tamar.christina@arm.com>
>
> PR target/84711
> * rtlanal.c (set_noop_p): Constrain on mode change,
> include hard-reg-set.h
>
> --