diff options
| author | Chandler Carruth <chandlerc@gmail.com> | 2016-03-07 14:05:09 +0000 |
|---|---|---|
| committer | Chandler Carruth <chandlerc@gmail.com> | 2016-03-07 14:05:09 +0000 |
| commit | 9ca96384f34c6dd3c63be5482616e50500fb1f43 (patch) | |
| tree | 8a6a87d95b645c7edd49db2fab2801a5f1339e12 /llvm/test/Instrumentation | |
| parent | 3e7cafe26710ebd42d2007eec29edb3293f347b0 (diff) | |
| download | bcm5719-llvm-9ca96384f34c6dd3c63be5482616e50500fb1f43.tar.gz bcm5719-llvm-9ca96384f34c6dd3c63be5482616e50500fb1f43.zip | |
[DFSan] Remove an overly aggressive assert reported in PR26068.
This code has been successfully used to bootstrap libc++ in a no-asserts
mode for a very long time, so the code that follows cannot be completely
incorrect. I've added a test that shows the current behavior for this
kind of code with DFSan. If it is desirable for DFSan to do something
special when processing an invoke of a variadic function, it can be
added, but we shouldn't keep an assert that we've been ignoring due to
release builds anyways.
llvm-svn: 262829
Diffstat (limited to 'llvm/test/Instrumentation')
| -rw-r--r-- | llvm/test/Instrumentation/DataFlowSanitizer/call.ll | 43 |
1 files changed, 40 insertions, 3 deletions
diff --git a/llvm/test/Instrumentation/DataFlowSanitizer/call.ll b/llvm/test/Instrumentation/DataFlowSanitizer/call.ll index dadb40fdb33..f196d1be4ea 100644 --- a/llvm/test/Instrumentation/DataFlowSanitizer/call.ll +++ b/llvm/test/Instrumentation/DataFlowSanitizer/call.ll @@ -2,13 +2,16 @@ target datalayout = "e-p:64:64:64-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-v64:64:64-v128:128:128-a0:0:64-s0:64:64-f80:128:128-n8:16:32:64-S128" target triple = "x86_64-unknown-linux-gnu" -; CHECK: @__dfsan_arg_tls = external thread_local(initialexec) global [64 x i16] -; CHECK: @__dfsan_retval_tls = external thread_local(initialexec) global i16 +; CHECK-LABEL: @__dfsan_arg_tls +; CHECK: = external thread_local(initialexec) global [64 x i16] + +; CHECK-LABEL: @__dfsan_retval_tls +; CHECK: = external thread_local(initialexec) global i16 declare i32 @f(i32) declare float @llvm.sqrt.f32(float) -; CHECK: @"dfs$call" +; CHECK-LABEL: @"dfs$call" define i32 @call() { ; CHECK: store{{.*}}__dfsan_arg_tls ; CHECK: call{{.*}}@"dfs$f" @@ -22,3 +25,37 @@ define i32 @call() { ; CHECK: ret i32 ret i32 %r } + +declare i32 @__gxx_personality_v0(...) + +declare i8* @__cxa_begin_catch(i8*) + +declare void @__cxa_end_catch() + +declare void @g(...) + +; CHECK-LABEL: @"dfs$h" +; CHECK: personality {{.*}} @"dfs$__gxx_personality_v0" {{.*}} { +define i32 @h() personality i8* bitcast (i32 (...)* @__gxx_personality_v0 to i8*) { +entry: +; CHECK: invoke void (...) @"dfs$g"(i32 42) + invoke void (...) @g(i32 42) + to label %try.cont unwind label %lpad + +lpad: + %0 = landingpad { i8*, i32 } + catch i8* null + %1 = extractvalue { i8*, i32 } %0, 0 + + ; CHECK: store {{.*}} @__dfsan_arg_tls + ; CHECK: call {{.*}} @"dfs$__cxa_begin_catch" + ; CHECK: load {{.*}} @__dfsan_retval_tls + %2 = tail call i8* @__cxa_begin_catch(i8* %1) + + ; CHECK: call {{.*}} @"dfs$__cxa_end_catch" + tail call void @__cxa_end_catch() + br label %try.cont + +try.cont: + ret i32 0 +} |

