This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] Lno branch merge part 12 -- induction variable optimizations
- From: Zdenek Dvorak <rakdver at atrey dot karlin dot mff dot cuni dot cz>
- To: Mark Mitchell <mark at codesourcery dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Thu, 2 Sep 2004 17:19:46 +0200
- Subject: Re: [patch] Lno branch merge part 12 -- induction variable optimizations
- References: <20040828080737.GA16110@atrey.karlin.mff.cuni.cz> <4136D7D3.1010404@codesourcery.com>
Hello,
> >5) build_int_cst is rewritten to decide whether to sign extend or to
> > zero extend depending on the signedness of the type of the constant,
> > and to mask out bits outside of the precision, which should be what
> > we want most of the time. Few places where this is incorrect should
> > use build_int_csts instead.
> >
> >
>
> Like Joseph, I think (5) should be completely separated from this patch.
>
> The patch to
> stor-layout.c is unncessary, and should not be included.
the patch to stor-layout is necessary for 5), since otherwise
build_int_cst is called before the bitsize of the type is set up.
Zdenek