[PATCH] SanitizerCoverage: Use `createSanitizerCtor` to create ctor and call init

Ismail Pazarbasi ismail.pazarbasi at gmail.com
Sat May 2 15:25:41 PDT 2015


================
Comment at: lib/Transforms/Instrumentation/SanitizerCoverage.cpp:163
@@ -168,3 +162,3 @@
 
   SanCovModuleInit = checkInterfaceFunction(M.getOrInsertFunction(
       kSanCovModuleInitName, Type::getVoidTy(*C), Int32PtrTy, IntptrTy,
----------------
samsonov wrote:
> Isn't this function supposed to be created in `createSanitizerCtor`?
Good catch, thanks! I will include it in the next revision. It needs a test to check init function is called from module ctor, as well as renaming `createSanitizerCtor` below.

http://reviews.llvm.org/D8780

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list