This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Support asan-instrumentation-with-call-threshold in GCC
- From: Konstantin Serebryany <konstantin dot s dot serebryany at gmail dot com>
- To: Yury Gribov <y dot gribov at samsung dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Konstantin Serebryany <kcc at gcc dot gnu dot org>, Jakub Jelinek <jakub at redhat dot com>, Viacheslav Garbuzov <v dot garbuzov at samsung dot com>, Yuri Gribov <tetra2005 at gmail dot com>, Dmitry Vyukov <dvyukov at google dot com>, Alexey Preobrazhensky <preobr at google dot com>
- Date: Tue, 27 May 2014 09:43:31 +0400
- Subject: Re: [PATCH] Support asan-instrumentation-with-call-threshold in GCC
- Authentication-results: sourceware.org; auth=none
- References: <53836499 dot 3000501 at samsung dot com> <CAGQ9bdyvDrWiG5w84XEcdsxXyWvfNkQPEKBvevdr2_0dqq_sfw at mail dot gmail dot com> <53842551 dot 6090000 at samsung dot com>
On Tue, May 27, 2014 at 9:40 AM, Yury Gribov <y.gribov@samsung.com> wrote:
>> - using instrumentation via calls adds extra 1.5x-2.x slowdown
>
> On x64.
Interesting. can you share your ARM numbers?
>
>
>> - it would be nice to have the name prefix configurable from command
>> line (${PREFIX}_load1 instead of __asan_load1) because kasan uses
>> different names already.
>
> Yeah, I noticed corresponding option in LLVM. AFAIK standard GCC parameters
> infrastructure (--param) only supports integral values so we'll need a
> separate flag for this feature. I'm curious, why do you need separate names
> for Kasan?
Well, maybe we don't. It'll just cause us a minor headache to change the names.
Dmitry?
>
> -Y