[PATCH] D12022: Refactored dtor sanitizing into EHScopeStack

Evgeniy Stepanov via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 25 13:30:23 PDT 2015


eugenis added inline comments.

================
Comment at: lib/CodeGen/CGCXX.cpp:31
@@ -30,1 +30,3 @@
 
+static bool HasTrivialField(CodeGenModule &CGM, const CXXDestructorDecl *D) {
+  for (const auto *Field : D->getParent()->fields())
----------------
I think this should be called "HasFieldWithTrivialDestructor".
Also, pass D->getParent() to this function instead of D.


http://reviews.llvm.org/D12022





More information about the cfe-commits mailing list