This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR61375: cancel bswap optimization when value doesn't fit in a HOST_WIDE_INT
- From: Jakub Jelinek <jakub at redhat dot com>
- To: "Thomas Preud'homme" <thomas dot preudhomme at arm dot com>
- Cc: Richard Biener <richard dot guenther at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 1 Aug 2014 08:59:39 +0200
- Subject: Re: [PATCH] Fix PR61375: cancel bswap optimization when value doesn't fit in a HOST_WIDE_INT
- Authentication-results: sourceware.org; auth=none
- References: <002c01cf8453$e32b4710$a981d530$ at arm dot com> <CAFiYyc3F==SQgWMWp2NOXAQvdB5UNQ1j6pJGDC6ebs-e9rnduw at mail dot gmail dot com> <002301cf8c74$3a934a20$afb9de60$ at arm dot com> <CAFiYyc0Y6do+ahNngtoE=RzVutBw3=5MXcNr0fXpM6xaOo+YUw at mail dot gmail dot com> <20140623083630 dot GO31640 at tucnak dot redhat dot com> <002a01cf8ec0$3d351df0$b79f59d0$ at arm dot com> <20140623085831 dot GP31640 at tucnak dot redhat dot com> <004e01cf90db$837c1b20$8a745160$ at arm dot com> <002001cfac8c$b6985830$23c90890$ at arm dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Thu, Jul 31, 2014 at 02:57:34PM +0800, Thomas Preud'homme wrote:
> Now that GCC 4.9 branch is opened again and GCC 4.8 branch still open, is
> the following backported patch ok for both branches?
Ok.
> > Ok, what about the following patch and associated ChangeLog entries?
> >
> > 2014-06-24 Thomas Preud'homme <thomas.preudhomme@arm.com>
> >
> > PR tree-optimization/61375
> > * tree-ssa-math-opts.c (find_bswap_or_nop_1): Cancel optimization
> > if
> > symbolic number cannot be represented in an unsigned
> > HOST_WIDE_INT.
> > (execute_optimize_bswap): Cancel optimization if CHAR_BIT != 8.
> >
> > 2014-06-24 Thomas Preud'homme <thomas.preudhomme@arm.com>
> >
> > PR tree-optimization/61375
> > * gcc.c-torture/execute/pr61375-1.c: New test.
Jakub