r198906 - CGRecordLayoutBuilder.cpp: Clarify if-else. [-Wdangling-else]

NAKAMURA Takumi geek4civic at gmail.com
Thu Jan 9 16:54:50 PST 2014


Author: chapuni
Date: Thu Jan  9 18:54:50 2014
New Revision: 198906

URL: http://llvm.org/viewvc/llvm-project?rev=198906&view=rev
Log:
CGRecordLayoutBuilder.cpp: Clarify if-else. [-Wdangling-else]

Modified:
    cfe/trunk/lib/CodeGen/CGRecordLayoutBuilder.cpp

Modified: cfe/trunk/lib/CodeGen/CGRecordLayoutBuilder.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGRecordLayoutBuilder.cpp?rev=198906&r1=198905&r2=198906&view=diff
==============================================================================
--- cfe/trunk/lib/CodeGen/CGRecordLayoutBuilder.cpp (original)
+++ cfe/trunk/lib/CodeGen/CGRecordLayoutBuilder.cpp Thu Jan  9 18:54:50 2014
@@ -825,12 +825,13 @@ bool CGRecordLayoutBuilder::LayoutFields
   const ASTRecordLayout &Layout = Types.getContext().getASTRecordLayout(D);
 
   const CXXRecordDecl *RD = dyn_cast<CXXRecordDecl>(D);
-  if (RD)
+  if (RD) {
     if (Types.getTarget().getCXXABI().isMicrosoft()) {
       if (!MSLayoutNonVirtualBases(RD, Layout))
         return false;
     } else if (!LayoutNonVirtualBases(RD, Layout))
       return false;
+  }
 
   unsigned FieldNo = 0;
   





More information about the cfe-commits mailing list