Re: [PATCH] analyzer: Fix PR analyzer/101980

2021-08-23 Thread Ankur Saini via Gcc-patches
> On 23-Aug-2021, at 5:00 PM, Martin Liška wrote: > > On 8/23/21 13:04, Ankur Saini wrote: >>> On 23-Aug-2021, at 2:20 PM, Martin Liška wrote: >>> >>> On 8/23/21 10:10, Martin Liška wrote: On 8/20/21 18:47, David Malcolm via Gcc-patches wrote: > On Fri, 2021-08-20 at 21:55 +0530, An

Re: [PATCH] analyzer: Fix PR analyzer/101980

2021-08-23 Thread Martin Liška
On 8/23/21 13:04, Ankur Saini wrote: On 23-Aug-2021, at 2:20 PM, Martin Liška wrote: On 8/23/21 10:10, Martin Liška wrote: On 8/20/21 18:47, David Malcolm via Gcc-patches wrote: On Fri, 2021-08-20 at 21:55 +0530, Ankur Saini wrote: The patch fixes the test failures introduced by : aef703

Re: [PATCH] analyzer: Fix PR analyzer/101980

2021-08-23 Thread Ankur Saini via Gcc-patches
update looks like the problem was with test itself, apparently Deja is quite strict when it comes to spaces and didn’t read the “dg-“ directive on line 72 as there wasn’t an extra space between the statement and the closing brace. Adding it seems to fix the issue - - - diff --git a/gcc/tes

Re: [PATCH] analyzer: Fix PR analyzer/101980

2021-08-23 Thread Ankur Saini via Gcc-patches
> On 23-Aug-2021, at 2:20 PM, Martin Liška wrote: > > On 8/23/21 10:10, Martin Liška wrote: >> On 8/20/21 18:47, David Malcolm via Gcc-patches wrote: >>> On Fri, 2021-08-20 at 21:55 +0530, Ankur Saini wrote: The patch fixes the test failures introduced by : aef703cf982072427e74

Re: [PATCH] analyzer: Fix PR analyzer/101980

2021-08-23 Thread Martin Liška
On 8/23/21 10:10, Martin Liška wrote: On 8/20/21 18:47, David Malcolm via Gcc-patches wrote: On Fri, 2021-08-20 at 21:55 +0530, Ankur Saini wrote: The patch fixes the test failures introduced by : aef703cf982072427e74034f4c460a11c5e04b8e 1b34248527472496ca3fe2a07183beac8cf69041 Thanks - Ankur

Re: [PATCH] analyzer: Fix PR analyzer/101980

2021-08-23 Thread Martin Liška
On 8/20/21 18:47, David Malcolm via Gcc-patches wrote: On Fri, 2021-08-20 at 21:55 +0530, Ankur Saini wrote: The patch fixes the test failures introduced by : aef703cf982072427e74034f4c460a11c5e04b8e 1b34248527472496ca3fe2a07183beac8cf69041 Thanks - Ankur Thanks for fixing this. The patch l

Re: [PATCH] analyzer: Fix PR analyzer/101980

2021-08-20 Thread David Malcolm via Gcc-patches
On Fri, 2021-08-20 at 21:55 +0530, Ankur Saini wrote: > The patch fixes the test failures introduced by : > > aef703cf982072427e74034f4c460a11c5e04b8e > 1b34248527472496ca3fe2a07183beac8cf69041 > > Thanks > - Ankur Thanks for fixing this. The patch looks OK, apart from some minor whitespace is