Re: wide int patch #6: Replacement of hwi extraction from int-csts.

2012-10-22 Thread Lawrence Crowl
On 10/19/12, Richard Biener wrote: > The existing tree_low_cst function performs checking, so > tree_to_hwi should as well. > > I don't think mismatch of signedness of the variable assigned to > with the sign we use for hwi extraction is any good. C++ isn't > type-safe here for the return value b

Re: wide int patch #6: Replacement of hwi extraction from int-csts.

2012-10-19 Thread Richard Biener
On Thu, Oct 18, 2012 at 4:00 PM, Kenneth Zadeck wrote: > you know richi, i did not know who i was actually talking to. i said who > is this richard beiner person and then i saw the email address. ;) > On 10/18/2012 08:58 AM, Richard Biener wrote: >> >> On Thu, Oct 18, 2012 at 2:52 PM, Kenneth

Re: wide int patch #6: Replacement of hwi extraction from int-csts.

2012-10-18 Thread Kenneth Zadeck
you know richi, i did not know who i was actually talking to. i said who is this richard beiner person and then i saw the email address. On 10/18/2012 08:58 AM, Richard Biener wrote: On Thu, Oct 18, 2012 at 2:52 PM, Kenneth Zadeck wrote: On 10/18/2012 06:22 AM, Richard Biener wrote: On We

Re: wide int patch #6: Replacement of hwi extraction from int-csts.

2012-10-18 Thread Richard Biener
On Thu, Oct 18, 2012 at 2:52 PM, Kenneth Zadeck wrote: > > On 10/18/2012 06:22 AM, Richard Biener wrote: >> >> On Wed, Oct 17, 2012 at 11:47 PM, Kenneth Zadeck >> wrote: >>> >>> Richi, >>> >>> I apologize for the size of this patch, but it only does one very small >>> thing, it is just that it do

Re: wide int patch #6: Replacement of hwi extraction from int-csts.

2012-10-18 Thread Kenneth Zadeck
On 10/18/2012 06:22 AM, Richard Biener wrote: On Wed, Oct 17, 2012 at 11:47 PM, Kenneth Zadeck wrote: Richi, I apologize for the size of this patch, but it only does one very small thing, it is just that it does it all over the middle end. This patch introduces a new api for extracting a sig

Re: wide int patch #6: Replacement of hwi extraction from int-csts.

2012-10-18 Thread Richard Biener
On Wed, Oct 17, 2012 at 11:47 PM, Kenneth Zadeck wrote: > Richi, > > I apologize for the size of this patch, but it only does one very small > thing, it is just that it does it all over the middle end. > > This patch introduces a new api for extracting a signed or unsigned hwi from > an integer cs