This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

[ping * 2] PR35503 - warn for restrict

From: Prathamesh Kulkarni <prathamesh dot kulkarni at linaro dot org>
To: gcc Patches <gcc-patches at gcc dot gnu dot org>, Marek Polacek <polacek at redhat dot com>, "Joseph S. Myers" <joseph at codesourcery dot com>
Date: Wed, 19 Oct 2016 19:46:19 +0530
Subject: [ping * 2] PR35503 - warn for restrict
Authentication-results: sourceware.org; auth=none

Hi,
The attached patch is a rebased version of the patch posted at:
https://gcc.gnu.org/ml/gcc-patches/2016-10/msg00446.html

The C++ changes are approved by Jason, other parts still require approval.
Bootstrapped and tested on x86_64-unknown-linux-gnu.
OK to commit ?

I was wondering if the warning should go in Wall or Wextra ?
The patch can give false positives because
it only checks whether parameters are qualified with restrict, not how
parameters are used inside the function. For instance it warned for example 10
mentioned in n1570 under section 6.7.3.1 - "Formal definition of restrict".
Currently I have kept the warning in Wall.

Thanks,
Prathamesh

Attachment: pr35503-13.txt
Description: Text document

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]