Refactor a bit and remove some FIXME audit markers, now that the code

has been audited for correctness.

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

diff --git a/lib/AST/RecordLayoutBuilder.cpp b/lib/AST/RecordLayoutBuilder.cpp
index 2a7477c4493d9b1b5e7050ef26ffa809a3e46432..91a66a3253f44bc8f0682713377ca4515369a72f 100644 (file)
--- a/lib/AST/RecordLayoutBuilder.cpp
+++ b/lib/AST/RecordLayoutBuilder.cpp
@@ -27,8 +27,7 @@ ASTRecordLayoutBuilder::ASTRecordLayoutBuilder(ASTContext &Ctx)
 
 /// LayoutVtable - Lay out the vtable and set PrimaryBase.
 void ASTRecordLayoutBuilder::LayoutVtable(const CXXRecordDecl *RD) {
-  // FIXME: audit indirect virtual bases
-  if (!RD->isPolymorphic() && !RD->getNumVBases()) {
+  if (!RD->isDynamicClass()) {
     // There is no primary base in this case.
     setPrimaryBase(0, false);
     return;
@@ -141,7 +140,6 @@ void ASTRecordLayoutBuilder::SelectPrimaryBase(const CXXRecordDecl *RD) {
 
   // If we have no virtual bases at this point, bail out as the searching below
   // is expensive.
-  // FIXME: audit indirect virtual bases
   if (RD->getNumVBases() == 0) {
     return;
   }