commit 42be77672a93fb07ece46cd6b253f08dc79740e1
author Warren Hunt <whunt@google.com> Mon Oct 14 20:14:09 2013 +0000
committer Warren Hunt <whunt@google.com> Mon Oct 14 20:14:09 2013 +0000
tree 75c4ac6a74c925ea26aad4675067f1dca8c86d0e
parent a443563ff09cd4a93257326b685167e36935af49

Correctly check for distructors when realizing vtordisps

This patch fixes the distructor test when checking for vtordisp requirements in 
microsoft record layout.  A test case is also included.

Addresses:
http://llvm.org/bugs/show_bug.cgi?id=16406#c7

llvm-svn: 192616

clang/lib/AST/RecordLayoutBuilder.cpp

diff --git a/clang/lib/AST/RecordLayoutBuilder.cpp b/clang/lib/AST/RecordLayoutBuilder.cpp
index 10c7b5d..05eeae1 100644
--- a/clang/lib/AST/RecordLayoutBuilder.cpp
+++ b/clang/lib/AST/RecordLayoutBuilder.cpp
@@ -2989,7 +2989,7 @@
     for (CXXRecordDecl::method_iterator i = RD->method_begin(),
                                         e = RD->method_end();
          i != e; ++i)
-      if ((*i)->isVirtual() && (*i) != RD->getDestructor())
+      if ((*i)->isVirtual() && !isa<CXXDestructorDecl>(*i))
         Work.insert(*i);
     while (!Work.empty()) {
       const CXXMethodDecl *MD = *Work.begin();