[ASan] Support the asan_loadN/asan_storeN functions in the DLL thunk library
authorEhsan Akhgari <ehsan.akhgari@gmail.com>
Fri, 1 Aug 2014 18:04:53 +0000 (18:04 +0000)
committerEhsan Akhgari <ehsan.akhgari@gmail.com>
Fri, 1 Aug 2014 18:04:53 +0000 (18:04 +0000)
Summary:
This is required for linking DLLs with large functions exceeding
san-instrumentation-with-call-threshold.  One such function is
vp9_fdct16x16_sse2 in libvpx.

Reviewers: timurrrr

Subscribers: llvm-commits

Differential Revision: http://reviews.llvm.org/D4758

llvm-svn: 214532

compiler-rt/lib/asan/asan_dll_thunk.cc
compiler-rt/test/asan/TestCases/Windows/dll_large_function.cc [new file with mode: 0644]

index b281fb3..63b6e91 100644 (file)
@@ -235,6 +235,20 @@ INTERFACE_FUNCTION(__asan_report_load8)
 INTERFACE_FUNCTION(__asan_report_load16)
 INTERFACE_FUNCTION(__asan_report_load_n)
 
+INTERFACE_FUNCTION(__asan_store1)
+INTERFACE_FUNCTION(__asan_store2)
+INTERFACE_FUNCTION(__asan_store4)
+INTERFACE_FUNCTION(__asan_store8)
+INTERFACE_FUNCTION(__asan_store16)
+INTERFACE_FUNCTION(__asan_storeN)
+
+INTERFACE_FUNCTION(__asan_load1)
+INTERFACE_FUNCTION(__asan_load2)
+INTERFACE_FUNCTION(__asan_load4)
+INTERFACE_FUNCTION(__asan_load8)
+INTERFACE_FUNCTION(__asan_load16)
+INTERFACE_FUNCTION(__asan_loadN)
+
 INTERFACE_FUNCTION(__asan_memcpy);
 INTERFACE_FUNCTION(__asan_memset);
 INTERFACE_FUNCTION(__asan_memmove);
diff --git a/compiler-rt/test/asan/TestCases/Windows/dll_large_function.cc b/compiler-rt/test/asan/TestCases/Windows/dll_large_function.cc
new file mode 100644 (file)
index 0000000..bd20a7b
--- /dev/null
@@ -0,0 +1,11 @@
+// Make sure we can link a DLL with large functions which would mean
+// functions such as __asan_loadN and __asan_storeN will be called
+// from the DLL.  We simulate the large function with
+// -mllvm -asan-instrumentation-with-call-threshold=0.
+// RUN: %clang_cl_asan %s -c -Fo%t.obj -mllvm -asan-instrumentation-with-call-threshold=0
+// RUN: link /nologo /DLL /OUT:%t.dll %t.obj %asan_dll_thunk
+
+void f(long* foo, long* bar) {
+  // One load and one store
+  *foo = *bar;
+}