CXXInfo memory should be released after calling the destructor

instead of before. The wrong order had no effect since Deallocate()
does nothing right now, but we may replace allocator in the future.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@209567 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/AST/RecordLayout.cpp b/lib/AST/RecordLayout.cpp
index 38e28d44d0ff06545b24e216d1e9244666838466..b2c244e3790ef5c05648797d15514b3ddacb699a 100644 (file)
--- a/lib/AST/RecordLayout.cpp
+++ b/lib/AST/RecordLayout.cpp
@@ -21,8 +21,8 @@ void ASTRecordLayout::Destroy(ASTContext &Ctx) {
   if (FieldOffsets)
     Ctx.Deallocate(FieldOffsets);
   if (CXXInfo) {
-    Ctx.Deallocate(CXXInfo);
     CXXInfo->~CXXRecordLayoutInfo();
+    Ctx.Deallocate(CXXInfo);
   }
   this->~ASTRecordLayout();
   Ctx.Deallocate(this);