three problems with stor-layout.c.
Richard Biener
rguenther@suse.de
Fri Nov 22 11:54:00 GMT 2013
On Thu, 21 Nov 2013, Kenneth Zadeck wrote:
> Richi,
>
> Here is the patch. As you can see, i chose the unsigned option.
> It was bootstrapped and tested on x86 with all languages including ada.
> Ok to commit?
Ok.
Thanks,
Richard.
> kenny
>
> 2013-11-21 zadeck@naturalbridge.com
> * store-layout.c
> (place-field): Fix hwi test and accessor mismatch.
>
>
>
>
> On 11/21/2013 11:07 AM, Eric Botcazou wrote:
> > > I think most of these are because in the past (yes I have fixed that!!)
> > > all 'sizetype' constants were sign-extended (and the signedness,
> > > that is, TYPE_UNSIGNED (sizetype), was frontend dependend (ugh) and
> > > then later true, thus unsigned).
> > >
> > > So I think all _SIZE stuff should check fits_uhwi_p and be used as
> > > uhwi. But that may have ripple-down effects, so consistently
> > > using fits_shwi_p and using as shwi is also fine (it just restricts
> > > the maximum values we accept(?)).
> > And please make sure to test Ada if you're tweaking this, it's fragile
> > stuff.
> >
More information about the Gcc-patches
mailing list