This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] enhance buffer overflow warnings (and c/53562)
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Martin Sebor <msebor at gmail dot com>
- Cc: gcc-patches at gcc dot gnu dot org, Tobias Burnus <tobias dot burnus at physik dot fu-berlin dot de>
- Date: Tue, 1 Nov 2016 15:10:25 +0100
- Subject: Re: [PATCH] enhance buffer overflow warnings (and c/53562)
- Authentication-results: sourceware.org; auth=none
- References: <20161031123909.GA9233@physik.fu-berlin.de> <334666bc-6308-aa5f-f63f-40697695152f@gmail.com> <904d9d3b-8662-e714-cc82-e08c72c54c0e@gmail.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Mon, Oct 31, 2016 at 08:39:15PM -0600, Martin Sebor wrote:
> Attached is an updated patch that works around the problem with
> the definition of the NOTE_DATA macro discussed below. I've
> raised bug 78174 for it and temporarily worked around it in
> the patch. I'll see if I can come up with a patch to fix the
> macro the "right way" but would prefer to do that separately.
> The NOTE_DATA macro is implemented in terms of the RTL_CHECK1
> macro that will need to change and that macro is used in many
> others, so I would rather not mess around with those as part
> of this patch.
No, you just shouldn't use __bos (*, 1) in the warning code
for mem* like builtins.
Jakub