Re: PR35503 - warn for restrict pointer (-Wrestrict)

2016-08-31 Thread Prathamesh Kulkarni
On 31 August 2016 at 03:45, Mike Stump wrote: > On Aug 30, 2016, at 4:57 AM, Prathamesh Kulkarni > wrote: >> >> On 30 August 2016 at 17:11, Eric Gallager wrote: >>> On 8/29/16, Jason Merrill wrote: On Mon, Aug 29, 2016 at 10:28 AM, Marek Polacek wrote: > On Mon, Aug 29, 2016 at 09:20

Re: PR35503 - warn for restrict pointer (-Wrestrict)

2016-08-30 Thread Mike Stump
On Aug 30, 2016, at 4:57 AM, Prathamesh Kulkarni wrote: > > On 30 August 2016 at 17:11, Eric Gallager wrote: >> On 8/29/16, Jason Merrill wrote: >>> On Mon, Aug 29, 2016 at 10:28 AM, Marek Polacek wrote: On Mon, Aug 29, 2016 at 09:20:53AM -0400, Eric Gallager wrote: > I tried this pa

Re: PR35503 - warn for restrict pointer (-Wrestrict)

2016-08-30 Thread Prathamesh Kulkarni
On 30 August 2016 at 17:11, Eric Gallager wrote: > On 8/29/16, Jason Merrill wrote: >> On Mon, Aug 29, 2016 at 10:28 AM, Marek Polacek wrote: >>> On Mon, Aug 29, 2016 at 09:20:53AM -0400, Eric Gallager wrote: I tried this patch on my fork of gdb-binutils and got a few warnings from it.

Re: PR35503 - warn for restrict pointer (-Wrestrict)

2016-08-30 Thread Eric Gallager
On 8/29/16, Jason Merrill wrote: > On Mon, Aug 29, 2016 at 10:28 AM, Marek Polacek wrote: >> On Mon, Aug 29, 2016 at 09:20:53AM -0400, Eric Gallager wrote: >>> I tried this patch on my fork of gdb-binutils and got a few warnings >>> from it. Would it be possible to have the caret point to the arg

Re: PR35503 - warn for restrict pointer (-Wrestrict)

2016-08-29 Thread Jason Merrill
On Mon, Aug 29, 2016 at 10:28 AM, Marek Polacek wrote: > On Mon, Aug 29, 2016 at 09:20:53AM -0400, Eric Gallager wrote: >> I tried this patch on my fork of gdb-binutils and got a few warnings >> from it. Would it be possible to have the caret point to the argument >> mentioned, instead of the func

Re: PR35503 - warn for restrict pointer (-Wrestrict)

2016-08-29 Thread Marek Polacek
On Mon, Aug 29, 2016 at 09:20:53AM -0400, Eric Gallager wrote: > I tried this patch on my fork of gdb-binutils and got a few warnings > from it. Would it be possible to have the caret point to the argument > mentioned, instead of the function name? And also print the option > name? E.g., instead of

Re: PR35503 - warn for restrict pointer (-Wrestrict)

2016-08-29 Thread Eric Gallager
On 8/28/16, Prathamesh Kulkarni wrote: > On 26 August 2016 at 21:25, Jason Merrill wrote: >> On Fri, Aug 26, 2016 at 7:39 AM, Prathamesh Kulkarni >> wrote: >>> However with C++FE it appears TYPE_RESTRICT is not set for the >>> parameters (buf and fmt) >>> and hence the warning doesn't get emitte