r243912 - Dtor callback emitted when msan attribute not repressed for this function.

[Naomi Musgrave]

Author: nmusgrave
Date: Mon Aug  3 17:53:11 2015
New Revision: 243912

URL: http://llvm.org/viewvc/llvm-project?rev=243912&view=rev
Log:
Dtor callback emitted when msan attribute not repressed for this function.

Summary: In addition to checking compiler flags, the front-end also examines the attributes of the destructor definition to ensure that the SanitizeMemory attribute is attached.

Reviewers: eugenis, kcc

Subscribers: cfe-commits

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

refactored test into new file, revised how function attribute examined

modified test to examine default dtor with and without attribute

removed attribute check

Added:
    cfe/trunk/test/CodeGenCXX/sanitize-dtor-fn-attribute.cpp
Modified:
    cfe/trunk/lib/CodeGen/CGClass.cpp

Modified: cfe/trunk/lib/CodeGen/CGClass.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGClass.cpp?rev=243912&r1=243911&r2=243912&view=diff
==============================================================================
--- cfe/trunk/lib/CodeGen/CGClass.cpp (original)
+++ cfe/trunk/lib/CodeGen/CGClass.cpp Mon Aug  3 17:53:11 2015
@@ -1479,7 +1479,8 @@ void CodeGenFunction::EmitDestructorBody
     ExitCXXTryStmt(*cast<CXXTryStmt>(Body), true);
 
   // Insert memory-poisoning instrumentation.
-  if (CGM.getCodeGenOpts().SanitizeMemoryUseAfterDtor)
+  if (CGM.getCodeGenOpts().SanitizeMemoryUseAfterDtor
+      && SanOpts.has(SanitizerKind::Memory))
     EmitDtorSanitizerCallback(*this, Dtor);
 }
 

Added: cfe/trunk/test/CodeGenCXX/sanitize-dtor-fn-attribute.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGenCXX/sanitize-dtor-fn-attribute.cpp?rev=243912&view=auto
==============================================================================
--- cfe/trunk/test/CodeGenCXX/sanitize-dtor-fn-attribute.cpp (added)
+++ cfe/trunk/test/CodeGenCXX/sanitize-dtor-fn-attribute.cpp Mon Aug  3 17:53:11 2015
@@ -0,0 +1,51 @@
+// Test -fsanitize-memory-use-after-dtor
+// RUN: %clang_cc1 -fsanitize=memory -fsanitize-memory-use-after-dtor -std=c++11 -triple=x86_64-pc-linux -emit-llvm -o - %s | FileCheck %s
+// RUN: %clang_cc1 -DATTRIBUTE -fsanitize=memory -fsanitize-memory-use-after-dtor -std=c++11 -triple=x86_64-pc-linux -emit-llvm -o - %s | FileCheck %s --check-prefix=CHECK-ATTR
+
+template <class T> class Vector {
+ public:
+  ~Vector() {}
+};
+
+struct No_San {
+  Vector<int> v;
+  No_San() { }
+#ifdef ATTRIBUTE
+  __attribute__((no_sanitize_memory)) ~No_San() = default;
+#else
+  ~No_San() = default;
+#endif
+};
+
+int main() {
+  No_San *ns = new No_San();
+  ns->~No_San();
+  return 0;
+}
+
+// Repressing the sanitization attribute results in no msan
+// instrumentation of the destructor
+// CHECK: define {{.*}}No_SanD1Ev{{.*}} [[ATTRIBUTE:#[0-9]+]]
+// CHECK: call void {{.*}}No_SanD2Ev
+// CHECK: call void @__sanitizer_dtor_callback
+// CHECK: ret void
+
+// CHECK-ATTR: define {{.*}}No_SanD1Ev{{.*}} [[ATTRIBUTE:#[0-9]+]]
+// CHECK-ATTR: call void {{.*}}No_SanD2Ev
+// CHECK-ATTR-NOT: call void @__sanitizer_dtor_callback
+// CHECK-ATTR: ret void
+
+
+// CHECK: define {{.*}}No_SanD2Ev{{.*}} [[ATTRIBUTE:#[0-9]+]]
+// CHECK: call void {{.*}}Vector
+// CHECK: call void @__sanitizer_dtor_callback
+// CHECK: ret void
+
+// CHECK-ATTR: define {{.*}}No_SanD2Ev{{.*}} [[ATTRIBUTE:#[0-9]+]]
+// CHECK-ATTR: call void {{.*}}Vector
+// CHECK-ATTR-NOT: call void @__sanitizer_dtor_callback
+// CHECK-ATTR: ret void
+
+// When attribute is repressed, the destructor does not emit any tail calls
+// CHECK: attributes [[ATTRIBUTE]] = {{.*}} sanitize_memory
+// CHECK-ATTR-NOT: attributes [[ATTRIBUTE]] = {{.*}} sanitize_memory