Re: Change double_int calls to new interface.

2012-09-12 Thread Lawrence Crowl
On 9/12/12, Mark Kettenis wrote: >> Date: Tue, 11 Sep 2012 17:03:39 -0700 >> From: Ian Lance Taylor >> >> On Tue, Sep 11, 2012 at 3:12 PM, Lawrence Crowl >> wrote: >> > On 9/11/12, Andreas Schwab wrote: >> >> Mark Kettenis writes: >> >>> In file included from ../../../src/gcc/gcc/mcf.c:47:0: >

Re: Change double_int calls to new interface.

2012-09-12 Thread Mark Kettenis
> Date: Tue, 11 Sep 2012 17:03:39 -0700 > From: Ian Lance Taylor > > On Tue, Sep 11, 2012 at 3:12 PM, Lawrence Crowl wrote: > > On 9/11/12, Andreas Schwab wrote: > >> Mark Kettenis writes: > >>> In file included from ../../../src/gcc/gcc/mcf.c:47:0: > >>> ../../../src/gcc/gcc/mcf.c: In functio

Re: Change double_int calls to new interface.

2012-09-11 Thread Ian Lance Taylor
On Tue, Sep 11, 2012 at 3:12 PM, Lawrence Crowl wrote: > On 9/11/12, Andreas Schwab wrote: >> Mark Kettenis writes: >>> In file included from ../../../src/gcc/gcc/mcf.c:47:0: >>> ../../../src/gcc/gcc/mcf.c: In function 'void dump_fixup_edge(FILE*, >>> fixup_graph_type*, fixup_edge_p)': >>> ../..

Re: Change double_int calls to new interface.

2012-09-11 Thread Lawrence Crowl
On 9/11/12, Andreas Schwab wrote: > Mark Kettenis writes: >> In file included from ../../../src/gcc/gcc/mcf.c:47:0: >> ../../../src/gcc/gcc/mcf.c: In function 'void dump_fixup_edge(FILE*, >> fixup_graph_type*, fixup_edge_p)': >> ../../../src/gcc/gcc/system.h:288:78: error: integer overflow in >>

Re: Change double_int calls to new interface.

2012-09-11 Thread Andreas Schwab
Mark Kettenis writes: > In file included from ../../../src/gcc/gcc/mcf.c:47:0: > ../../../src/gcc/gcc/mcf.c: In function 'void dump_fixup_edge(FILE*, > fixup_graph_type*, fixup_edge_p)': > ../../../src/gcc/gcc/system.h:288:78: error: integer overflow in expression > [-Werror=overflow] This is

Re: Change double_int calls to new interface.

2012-09-11 Thread Mark Kettenis
> Index: gcc/ChangeLog > > 2012-09-04 Lawrence Crowl > > * double-int.h (double_int::operator &=): New. > (double_int::operator ^=): New. > (double_int::operator |=): New. > (double_int::mul_with_sign): Modify overflow parameter to bool*. > (do

Re: Change double_int calls to new interface.

2012-09-06 Thread Richard Guenther
On Wed, 5 Sep 2012, Lawrence Crowl wrote: > On 9/5/12, Richard Guenther wrote: > > On Tue, 4 Sep 2012, Lawrence Crowl wrote: > >> Modify gcc/*.[hc] double_int call sites to use the new interface. > >> This change entailed adding a few new methods to double_int. > >> > >> Other changes will happen

Re: Change double_int calls to new interface.

2012-09-05 Thread Lawrence Crowl
On 9/5/12, Marc Glisse wrote: > On Wed, 5 Sep 2012, Lawrence Crowl wrote: >> On 9/5/12, Richard Guenther wrote: >>> The tmin.cmp (tmax, uns) > 0 kind of things look odd - definitely >>> methods like tmin.gt (tmax, uns) would be nice to have. Or even >>> better, get rid of the 'uns' parameters an

Re: Change double_int calls to new interface.

2012-09-05 Thread Marc Glisse
On Wed, 5 Sep 2012, Lawrence Crowl wrote: On 9/5/12, Richard Guenther wrote: The tmin.cmp (tmax, uns) > 0 kind of things look odd - definitely methods like tmin.gt (tmax, uns) would be nice to have. Or even better, get rid of the 'uns' parameters and provide a struct double_int_with_signedne

Re: Change double_int calls to new interface.

2012-09-05 Thread Lawrence Crowl
On 9/5/12, Richard Guenther wrote: > On Tue, 4 Sep 2012, Lawrence Crowl wrote: >> Modify gcc/*.[hc] double_int call sites to use the new interface. >> This change entailed adding a few new methods to double_int. >> >> Other changes will happen in separate patches. Once all uses of >> the old inte