[PATCH] D67298: [ASan] Only run dlopen-mixed-c-cxx.c with static runtime

Jonas Hahnfeld via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 7 03:36:11 PDT 2019


Hahnfeld updated this revision to Diff 219227.
Hahnfeld retitled this revision from "[ASan] Fix test case dlopen-mixed-c-cxx.c without C++ stdlib" to "[ASan] Only run dlopen-mixed-c-cxx.c with static runtime".
Hahnfeld edited the summary of this revision.
Hahnfeld added a comment.
Herald added a subscriber: srhines.

I think I just figured out that the original problem is about static runtime, so it doesn't make sense to run the test with dynamic asan.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67298/new/

https://reviews.llvm.org/D67298

Files:
  compiler-rt/test/asan/TestCases/Linux/dlopen-mixed-c-cxx.c


Index: compiler-rt/test/asan/TestCases/Linux/dlopen-mixed-c-cxx.c
===================================================================
--- compiler-rt/test/asan/TestCases/Linux/dlopen-mixed-c-cxx.c
+++ compiler-rt/test/asan/TestCases/Linux/dlopen-mixed-c-cxx.c
@@ -3,9 +3,10 @@
 //
 // RUN: { env ASAN_OPTIONS=verbosity=1 %t.out %t.so || : ; } 2>&1 | FileCheck %s
 //
-// CHECK: {{.*}}AddressSanitizer: failed to intercept '__cxa_{{.*}}throw{{.*}}'
+// CHECK: AddressSanitizer: failed to intercept '__cxa_throw'
 //
-// REQUIRES: x86_64-target-arch && !android
+// This tests assumes static linking of the asan runtime.
+// UNSUPPORTED: asan-dynamic-runtime
 
 #ifdef __cplusplus
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67298.219227.patch
Type: text/x-patch
Size: 681 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190907/8a85149a/attachment.bin>


More information about the llvm-commits mailing list