r331621 - Remove now-unnecessary check for non-zero nvsize in addition to

Richard Smith via cfe-commits cfe-commits at lists.llvm.org
Sun May 6 23:43:31 PDT 2018


Author: rsmith
Date: Sun May  6 23:43:31 2018
New Revision: 331621

URL: http://llvm.org/viewvc/llvm-project?rev=331621&view=rev
Log:
Remove now-unnecessary check for non-zero nvsize in addition to
emptyness in MS record layout.

Modified:
    cfe/trunk/lib/AST/RecordLayoutBuilder.cpp

Modified: cfe/trunk/lib/AST/RecordLayoutBuilder.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/RecordLayoutBuilder.cpp?rev=331621&r1=331620&r2=331621&view=diff
==============================================================================
--- cfe/trunk/lib/AST/RecordLayoutBuilder.cpp (original)
+++ cfe/trunk/lib/AST/RecordLayoutBuilder.cpp Sun May  6 23:43:31 2018
@@ -2617,8 +2617,8 @@ void MicrosoftRecordLayoutBuilder::layou
   }
 
   if (!FoundBase) {
-    if (MDCUsesEBO && BaseDecl->isEmpty() &&
-        BaseLayout.getNonVirtualSize() == CharUnits::Zero()) {
+    if (MDCUsesEBO && BaseDecl->isEmpty()) {
+      assert(BaseLayout.getNonVirtualSize() == CharUnits::Zero());
       BaseOffset = CharUnits::Zero();
     } else {
       // Otherwise, lay the base out at the end of the MDC.




More information about the cfe-commits mailing list