Re: [PATCH] relax lower bound for infinite arguments in gimple-ssa-sprinf.c (PR 86274)

2018-07-04 Thread Martin Sebor
On 07/04/2018 08:04 AM, Christophe Lyon wrote: Hi, On Wed, 4 Jul 2018 at 09:26, Richard Biener wrote: On Tue, 3 Jul 2018, Martin Sebor wrote: Committed to trunk in r86274. Jakub/Richard, can you please also review and approve the corresponding fix for the release branches? If it is a

Re: [PATCH] relax lower bound for infinite arguments in gimple-ssa-sprinf.c (PR 86274)

2018-07-04 Thread Christophe Lyon
Hi, On Wed, 4 Jul 2018 at 09:26, Richard Biener wrote: > > On Tue, 3 Jul 2018, Martin Sebor wrote: > > > Committed to trunk in r86274. Jakub/Richard, can you please > > also review and approve the corresponding fix for the release > > branches? > > If it is a regression and the patch was

Re: [PATCH] relax lower bound for infinite arguments in gimple-ssa-sprinf.c (PR 86274)

2018-07-04 Thread Richard Biener
On Tue, 3 Jul 2018, Martin Sebor wrote: > Committed to trunk in r86274. Jakub/Richard, can you please > also review and approve the corresponding fix for the release > branches? If it is a regression and the patch was approved for trunk it is automatically OK for release branches without

Re: [PATCH] relax lower bound for infinite arguments in gimple-ssa-sprinf.c (PR 86274)

2018-07-03 Thread Martin Sebor
Committed to trunk in r86274. Jakub/Richard, can you please also review and approve the corresponding fix for the release branches? Martin On 07/03/2018 06:32 PM, Jeff Law wrote: On 07/03/2018 04:50 PM, Martin Sebor wrote: In computing the size of expected output for non-constant floating

Re: [PATCH] relax lower bound for infinite arguments in gimple-ssa-sprinf.c (PR 86274)

2018-07-03 Thread Jeff Law
On 07/03/2018 04:50 PM, Martin Sebor wrote: > In computing the size of expected output for non-constant floating > arguments the sprintf pass doesn't consider the possibility that > the argument value may be not finite (i.e., it can be infinity or > NaN).  Infinities and NaNs are formatted as 

[PATCH] relax lower bound for infinite arguments in gimple-ssa-sprinf.c (PR 86274)

2018-07-03 Thread Martin Sebor
In computing the size of expected output for non-constant floating arguments the sprintf pass doesn't consider the possibility that the argument value may be not finite (i.e., it can be infinity or NaN). Infinities and NaNs are formatted as "inf" or "infinity" and "nan". As a result, any