eugenis added inline comments.

================
Comment at: lib/CodeGen/CGClass.cpp:1459
@@ +1458,3 @@
+
+  if (CGM.getCodeGenOpts().SanitizeMemoryUseAfterDtor) {
+    // insert destructor clean up here
----------------
Please remove this block completely. It does nothing.

================
Comment at: lib/Driver/SanitizerArgs.cpp:420
@@ -418,1 +419,3 @@
       }
+      MsanUseAfterDtor =
+          Args.hasArg(options::OPT_fsanitize_memory_use_after_dtor);
----------------
It looks like this variable is only assigned but never read.

================
Comment at: test/Driver/fsanitize.c:126
@@ +125,3 @@
+// RUN: %clang -target x86_64-linux-gnu -fsanitize=memory 
-fsanitize-memory-use-after-dtor -pie %s -### 2>&1 | FileCheck %s 
--check-prefix=CHECK-MSAN-USE-AFTER-DTOR-0
+// CHECK-MSAN-USE-AFTER-DTOR-0: -fsanitize-memory-use-after-dtor
+
----------------
remove "-0"


http://reviews.llvm.org/D11092




_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to