[PATCH] D12022: Refactored dtor sanitizing into EHScopeStack

Evgeniy Stepanov via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 21 15:54:35 PDT 2015


eugenis added inline comments.

================
Comment at: lib/CodeGen/CGCXX.cpp:41
@@ +40,3 @@
+  if (getCodeGenOpts().SanitizeMemoryUseAfterDtor && Layout.getFieldCount() > 0
+      && HasTrivialDestructorBody(Context, D->getParent(), D->getParent())) {
+        return true;
----------------
I'm not sure this is correct. It says not to use an alias if D has trivial body. Should not it be the other way around?


http://reviews.llvm.org/D12022





More information about the cfe-commits mailing list