Re: [wide-int] Fix mode choice in convert_modes

2013-09-18 Thread Kenneth Zadeck
this is fine with me. kenny On 09/18/2013 03:27 AM, Richard Sandiford wrote: Ping Richard Sandiford writes: Enabling the CONST_INT assert showed that this branch-local code was passing the wrong mode to std::make_pair. The mode of X is OLDMODE, and we're supposed to be converting it to MODE

Re: [wide-int] Fix mode choice in convert_modes

2013-09-18 Thread Richard Sandiford
Ping Richard Sandiford writes: > Enabling the CONST_INT assert showed that this branch-local code was > passing the wrong mode to std::make_pair. The mode of X is OLDMODE, > and we're supposed to be converting it to MODE. > > In the case where OLDMODE is VOIDmode, I think we have to assume > tha

Re: [wide-int] Fix mode choice in convert_modes

2013-09-09 Thread Kenneth Zadeck
On 09/09/2013 09:56 AM, Richard Sandiford wrote: Richard Biener writes: On Mon, Sep 9, 2013 at 3:11 PM, Richard Sandiford wrote: Enabling the CONST_INT assert showed that this branch-local code was passing the wrong mode to std::make_pair. The mode of X is OLDMODE, and we're supposed to be c

Re: [wide-int] Fix mode choice in convert_modes

2013-09-09 Thread Richard Sandiford
Richard Biener writes: > On Mon, Sep 9, 2013 at 3:11 PM, Richard Sandiford > wrote: >> Enabling the CONST_INT assert showed that this branch-local code was >> passing the wrong mode to std::make_pair. The mode of X is OLDMODE, >> and we're supposed to be converting it to MODE. >> >> In the case

Re: [wide-int] Fix mode choice in convert_modes

2013-09-09 Thread Richard Biener
On Mon, Sep 9, 2013 at 3:11 PM, Richard Sandiford wrote: > Enabling the CONST_INT assert showed that this branch-local code was > passing the wrong mode to std::make_pair. The mode of X is OLDMODE, > and we're supposed to be converting it to MODE. > > In the case where OLDMODE is VOIDmode, I thin