This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][RFC] Fix PR30364, reassociation not allowed for undefined overflow
- From: Rask Ingemann Lambertsen <rask at sygehus dot dk>
- To: Richard Guenther <richard dot guenther at gmail dot com>
- Cc: Richard Guenther <rguenther at suse dot de>, Ian Lance Taylor <iant at google dot com>, gcc-patches at gcc dot gnu dot org
- Date: Sat, 23 Jun 2007 13:49:01 +0200
- Subject: Re: [PATCH][RFC] Fix PR30364, reassociation not allowed for undefined overflow
- References: <Pine.LNX.4.64.0702221058390.18923@zhemvz.fhfr.qr> <m3irdu89tn.fsf@localhost.localdomain> <Pine.LNX.4.64.0702242256150.18923@zhemvz.fhfr.qr> <20070622173214.GB5690@sygehus.dk> <84fc9c000706221116v3c4c420fg9b32c9c7505b9845@mail.gmail.com>
On Fri, Jun 22, 2007 at 08:16:06PM +0200, Richard Guenther wrote:
> On 6/22/07, Rask Ingemann Lambertsen <rask@sygehus.dk> wrote:
> >On Sat, Feb 24, 2007 at 10:58:41PM +0100, Richard Guenther wrote:
> >>
> >> * gcc.dg/torture/pr30364-1.c: New testcase.
> >> * gcc.dg/torture/pr30364-2.c: Likewise.
> >> * gcc.dg/torture/pr30364-3.c: Likewise.
> >
> > 1) This tests nothing on hosts where int is 64 bits.
>
> Which would be?
I don't know the target triplet for it, but
gcc/config/alpha/unicosmk.h:#define INT_TYPE_SIZE 64
> > 2) This will always fail on hosts where int is 16 bits.
>
> Do they all have SImode? In which case we could use a
> SImode type. Feel free to do so or file a bugreport to remind me
> (you can assign it to me).
SImode isn't the complete answer either, because:
gcc/config/c4x/c4x.h:#define BITS_PER_UNIT 32
Another strategy would be to adjust the constants to the range of int.
Anyway, filed as bug testsuite/32471.
--
Rask Ingemann Lambertsen