From: Naomi Musgrave Date: Thu, 16 Jul 2015 00:25:45 +0000 (+0000) Subject: updated tests for correct commit X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=cdc607c2398cb353af7cf751a4066dc99be698d8;p=clang updated tests for correct commit git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@242364 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/test/CodeGenCXX/sanitize-dtor-callback.cpp b/test/CodeGenCXX/sanitize-dtor-callback.cpp index 233561e4e0..a3ecbcbdd8 100644 --- a/test/CodeGenCXX/sanitize-dtor-callback.cpp +++ b/test/CodeGenCXX/sanitize-dtor-callback.cpp @@ -1,58 +1,39 @@ // Test -fsanitize-memory-use-after-dtor -// RUN: %clang_cc1 -fsanitize=memory -fsanitize-memory-use-after-dtor -triple=x86_64-pc-linux -emit-llvm -o - %s | FileCheck %s -// RUN: %clang_cc1 -fsanitize=memory -triple=x86_64-pc-linux -emit-llvm -o - %s | FileCheck %s -check-prefix=NO-DTOR-CHECK - -// RUN: %clang_cc1 -std=c++11 -fsanitize=memory -fsanitize-memory-use-after-dtor -triple=x86_64-pc-linux -emit-llvm -o - %s | FileCheck %s -check-prefix=STD11 -// RUN: %clang_cc1 -std=c++11 -fsanitize=memory -triple=x86_64-pc-linux -emit-llvm -o - %s | FileCheck %s -check-prefix=NO-DTOR-STD11-CHECK +// RUN: %clang_cc1 -fsanitize=memory -fsanitize-memory-use-after-dtor -std=c++11 -triple=x86_64-pc-linux -emit-llvm -o - %s | FileCheck %s struct Simple { ~Simple() {} }; Simple s; // Simple internal member is poisoned by compiler-generated dtor -// CHECK-LABEL: @_ZN6SimpleD2Ev +// CHECK-LABEL: define {{.*}}SimpleD1Ev // CHECK: call void @__sanitizer_dtor_callback +// CHECK-NOT: call void @__sanitizer_dtor_callback // CHECK: ret void -// Compiling without the flag does not generate member-poisoning dtor -// NO-DTOR-CHECK-LABEL: @_ZN6SimpleD2Ev -// NO-DTOR-CHECK-NOT: call void @__sanitizer_dtor_callback -// NO-DTOR-CHECK: ret void - - struct Inlined { inline ~Inlined() {} }; -Inlined in; -// Dtor that is inlined where invoked poisons object -// CHECK-LABEL: @_ZN7InlinedD2Ev +Inlined i; +// Simple internal member is poisoned by compiler-generated dtor +// CHECK-LABEL: define {{.*}}InlinedD1Ev // CHECK: call void @__sanitizer_dtor_callback +// CHECK-NOT: call void @__sanitizer_dtor_callback // CHECK: ret void -// Compiling without the flag does not generate member-poisoning dtor -// NO-DTOR-CHECK-LABEL: @_ZN7InlinedD2Ev -// NO-DTOR-CHECK-NOT: call void @__sanitizer_dtor_callback -// NO-DTOR-CHECK: ret void - - struct Defaulted_Trivial { ~Defaulted_Trivial() = default; }; -int main() { +void create_def_trivial() { Defaulted_Trivial def_trivial; } // The compiler is explicitly signalled to handle object cleanup. -// No complex member attributes ensures that the compiler destroys -// the memory inline. However, it must still poison this memory. -// STD11-CHECK-LABEL: alloca %struct.Defaulted_Trivial -// STD11: call void @__sanitizer_dtor_callback -// STD11: ret void - -// Compiling without the flag does not generate member-poisoning dtor -// NO-DTOR-STD11-CHECK-LABEL: alloca %struct.Defaulted_Trivial -// NO-DTOR-STD11-CHECK-NOT: call void @__sanitizer_dtor_callback -// NO-DTOR-STD11-CHECK: ret void - +// No complex member attributes. Compiler destroys inline, so +// no destructor defined. +// CHECK-LABEL: define {{.*}}create_def_trivial +// CHECK-NOT: call {{.*}}Defaulted_Trivial +// CHECK-NOT: call void @__sanitizer_dtor_callback +// CHECK: ret void struct Defaulted_Non_Trivial { Simple s; @@ -61,12 +42,28 @@ struct Defaulted_Non_Trivial { Defaulted_Non_Trivial def_non_trivial; // Explicitly compiler-generated dtor poisons object. // By including a Simple member in the struct, the compiler is -// forced to generate a non-trivial destructor.. -// STD11-CHECK-LABEL: @_ZN21Defaulted_Non_TrivialD2Ev -// STD11: call void @__sanitizer_dtor_callback -// STD11: ret void +// forced to generate a non-trivial destructor. +// CHECK-LABEL: define {{.*}}Defaulted_Non_TrivialD1Ev +// CHECK: call void @__sanitizer_dtor_callback +// CHECK-NOT: call void @__sanitizer_dtor_callback +// CHECK: ret void + + +// Note: ordering is important. In the emitted bytecode, these +// second dtors defined after the first. Explicitly checked here +// to confirm that all invoked dtors have member poisoning +// instrumentation inserted. +// CHECK-LABEL: define {{.*}}SimpleD2Ev +// CHECK: call void @__sanitizer_dtor_callback +// CHECK-NOT: call void @__sanitizer_dtor_callback +// CHECK: ret void -// Compiling without the flag does not generate member-poisoning dtor -// NO-DTOR-STD11-CHECK-LABEL: @_ZN21Defaulted_Non_TrivialD2Ev -// NO-DTOR-STD11-CHECK-NOT: call void @__sanitizer_dtor_callback -// NO-DTOR-STD11-CHECK: ret void +// CHECK-LABEL: define {{.*}}InlinedD2Ev +// CHECK: call void @__sanitizer_dtor_callback +// CHECK-NOT: call void @__sanitizer_dtor_callback +// CHECK: ret void + +// CHECK-LABEL: define {{.*}}Defaulted_Non_TrivialD2Ev +// CHECK: call void @__sanitizer_dtor_callback +// CHECK-NOT: call void @__sanitizer_dtor_callback +// CHECK: ret void diff --git a/test/CodeGenCXX/sanitize-no-dtor-callback.cpp b/test/CodeGenCXX/sanitize-no-dtor-callback.cpp new file mode 100644 index 0000000000..2c35576621 --- /dev/null +++ b/test/CodeGenCXX/sanitize-no-dtor-callback.cpp @@ -0,0 +1,23 @@ +// Test without the flag -fsanitize-memory-use-after-dtor, to ensure that +// instrumentation is not erroneously inserted +// RUN: %clang_cc1 -fsanitize=memory -triple=x86_64-pc-linux -emit-llvm -o - %s | FileCheck %s + +struct Simple { + ~Simple() {} +}; +Simple s; +// CHECK-LABEL: define {{.*}}SimpleD1Ev +// CHECK-NOT: call void @__sanitizer_dtor_callback + +struct Inlined { + inline ~Inlined() {} +}; +Inlined i; +// CHECK-LABEL: define {{.*}}InlinedD1Ev +// CHECK-NOT: call void @__sanitizer_dtor_callback + +// CHECK-LABEL: define {{.*}}SimpleD2Ev +// CHECK-NOT: call void @__sanitizer_dtor_callback + +// CHECK-LABEL: define {{.*}}InlinedD2Ev +// CHECK-NOT: call void @__sanitizer_dtor_callback