This is the mail archive of the
mailing list for the GCC project.
Re: [PATCH, i386, Pointer Bounds Checker 12/x] Recognize instrumented special functions
- From: Ilya Enkovich <enkovich dot gnu at gmail dot com>
- To: Jeff Law <law at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Mon, 2 Jun 2014 14:38:13 +0400
- Subject: Re: [PATCH, i386, Pointer Bounds Checker 12/x] Recognize instrumented special functions
- Authentication-results: sourceware.org; auth=none
- References: <20140529110002 dot GA30323 at msticlxl57 dot ims dot intel dot com> <5388B5FB dot 50309 at redhat dot com>
On 30 May 10:46, Jeff Law wrote:
> On 05/29/14 05:00, Ilya Enkovich wrote:
> >This patch allows to recognize instrumented call to special function by using the original function name for recognition.
> >Bootstrapped and tested on linux-x86_64.
> >2014-05-29 Ilya Enkovich <firstname.lastname@example.org>
> > * calls.c (special_function_p): Use original decl name
> > when analyzing instrumentation clone.
> OK for the trunk when the rest of the patches are approved.
> Presumably we have to use the original decl because we twiddle the
> name in the clones, right? Just want to make sure I understand why
> we're dong this :-)
That's right. We add ".chkp" prefix for all instrumented functions to keep assembler name unique during compilation. Otherwise it becomes a mess with LTO.