This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Inline asm asan instrumentation
- From: Evgeniy Stepanov <eugeni dot stepanov at gmail dot com>
- To: Konstantin Serebryany <konstantin dot s dot serebryany at gmail dot com>
- Cc: Marat Zakirov <m dot zakirov at samsung dot com>, 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 <y dot gribov at samsung dot com>, Marat Zakirov <marat61 at gmail dot com>
- Date: Thu, 29 May 2014 13:58:16 +0400
- Subject: Re: [PATCH] Inline asm asan instrumentation
- Authentication-results: sourceware.org; auth=none
- References: <000001cf7a79$745ee000$5d1ca000$ at samsung dot com> <CAGQ9bdxVSgrmRW+aBijJOkG+-ozDXyR7pDXZyccoNmCaqT=mFg at mail dot gmail dot com>
Cool, we don't have this in LLVM-ASan, but we have plans to instrument
inline asm soon (not just constraints).
asm-struct-1.c test looks like a false positive though - the code does
not access any invalid memory, it only does a harmless pointer cast.
On Wed, May 28, 2014 at 10:36 PM, Konstantin Serebryany
<konstantin.s.serebryany@gmail.com> wrote:
> On Wed, May 28, 2014 at 5:33 PM, Marat Zakirov <m.zakirov@samsung.com> wrote:
>> Hi all,
>>
>> Here's a patch for optional Asan instrumentation of inline assembly.
>>
>> This version scans gimple for GIMPLE_ASMs and performs usual instrumentation
>> of arguments with memory constraints ("m", "o", etc.) with fixed size.
>>
>> Instrumentation is turned off by default.
>>
>> This was successfully bootstrapped and regtested on x64. I have also
>> instrumented and ran ffmpeg regression testsuite (it seems to have quite
>> some inline asm).
>>
>> --Marat