Re: [PATCH] Add offset_int to guard HOST_WIDE_INT overflow (PR tree-optimization/82042).

2017-11-10 Thread Martin Sebor
On 10/19/2017 05:58 AM, Richard Biener wrote: On Thu, Oct 19, 2017 at 1:21 PM, Martin Liška wrote: On 09/20/2017 05:47 PM, Richard Biener wrote: On Wed, Sep 20, 2017 at 9:50 AM, Martin Liška wrote: Hi. This is partial fix for the PR, where I calculate 2

Re: [PATCH] Add offset_int to guard HOST_WIDE_INT overflow (PR tree-optimization/82042).

2017-11-02 Thread Martin Liška
On 10/19/2017 01:58 PM, Richard Biener wrote: > In the end I don't think we want to slow down code just for the sake of > UBSAN. IMHO for code invoking undefined behavior (object too large) > it's reasonable for the compiler to invoke undefined behavior Agree that it's very artificial

Re: [PATCH] Add offset_int to guard HOST_WIDE_INT overflow (PR tree-optimization/82042).

2017-10-19 Thread Richard Biener
On Thu, Oct 19, 2017 at 1:21 PM, Martin Liška wrote: > On 09/20/2017 05:47 PM, Richard Biener wrote: >> On Wed, Sep 20, 2017 at 9:50 AM, Martin Liška wrote: >>> Hi. >>> >>> This is partial fix for the PR, where I calculate 2 offsets in offset_int >>> type. >>>

Re: [PATCH] Add offset_int to guard HOST_WIDE_INT overflow (PR tree-optimization/82042).

2017-10-19 Thread Martin Liška
On 09/20/2017 05:47 PM, Richard Biener wrote: > On Wed, Sep 20, 2017 at 9:50 AM, Martin Liška wrote: >> Hi. >> >> This is partial fix for the PR, where I calculate 2 offsets in offset_int >> type. >> >> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. >>

Re: [PATCH] Add offset_int to guard HOST_WIDE_INT overflow (PR tree-optimization/82042).

2017-09-20 Thread Richard Biener
On Wed, Sep 20, 2017 at 9:50 AM, Martin Liška wrote: > Hi. > > This is partial fix for the PR, where I calculate 2 offsets in offset_int > type. > > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > > Ready to be installed? That seems to poorly handle