[llvm] r323140 - asan: allow inline instrumentation for the kernel

[Vedant Kumar via llvm-commits]

Could you please add a test?

vedant

> On Jan 22, 2018, at 11:07 AM, Dmitry Vyukov via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> 
> Author: dvyukov
> Date: Mon Jan 22 11:07:11 2018
> New Revision: 323140
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=323140&view=rev
> Log:
> asan: allow inline instrumentation for the kernel
> 
> Currently ASan instrumentation pass forces callback
> instrumentation when applied to the kernel.
> This patch changes the current behavior to allow
> using inline instrumentation in this case.
> 
> Authored by andreyknvl. Reviewed in:
> https://reviews.llvm.org/D42384
> 
> 
> Modified:
>    llvm/trunk/lib/Transforms/Instrumentation/AddressSanitizer.cpp
> 
> Modified: llvm/trunk/lib/Transforms/Instrumentation/AddressSanitizer.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Instrumentation/AddressSanitizer.cpp?rev=323140&r1=323139&r2=323140&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Transforms/Instrumentation/AddressSanitizer.cpp (original)
> +++ llvm/trunk/lib/Transforms/Instrumentation/AddressSanitizer.cpp Mon Jan 22 11:07:11 2018
> @@ -2494,7 +2494,6 @@ bool AddressSanitizer::runOnFunction(Fun
>   }
> 
>   bool UseCalls =
> -      CompileKernel ||
>       (ClInstrumentationWithCallsThreshold >= 0 &&
>        ToInstrument.size() > (unsigned)ClInstrumentationWithCallsThreshold);
>   const DataLayout &DL = F.getParent()->getDataLayout();
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits