[llvm] r300137 - Fix initialization order of class members.

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 12 16:27:44 PDT 2017


Author: zturner
Date: Wed Apr 12 18:27:43 2017
New Revision: 300137

URL: http://llvm.org/viewvc/llvm-project?rev=300137&view=rev
Log:
Fix initialization order of class members.

Modified:
    llvm/trunk/include/llvm/DebugInfo/PDB/UDTLayout.h
    llvm/trunk/lib/DebugInfo/PDB/UDTLayout.cpp

Modified: llvm/trunk/include/llvm/DebugInfo/PDB/UDTLayout.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/DebugInfo/PDB/UDTLayout.h?rev=300137&r1=300136&r2=300137&view=diff
==============================================================================
--- llvm/trunk/include/llvm/DebugInfo/PDB/UDTLayout.h (original)
+++ llvm/trunk/include/llvm/DebugInfo/PDB/UDTLayout.h Wed Apr 12 18:27:43 2017
@@ -112,10 +112,10 @@ protected:
 
   void addChildToLayout(std::unique_ptr<StorageItemBase> Child);
 
-  uint32_t SizeOf = 0;
+  const PDBSymbol &SymbolBase;
   std::string Name;
+  uint32_t SizeOf = 0;
 
-  const PDBSymbol &SymbolBase;
   BitVector UsedBytes;
   std::vector<std::unique_ptr<PDBSymbol>> NonStorageItems;
   std::vector<std::unique_ptr<StorageItemBase>> ChildStorage;

Modified: llvm/trunk/lib/DebugInfo/PDB/UDTLayout.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/DebugInfo/PDB/UDTLayout.cpp?rev=300137&r1=300136&r2=300137&view=diff
==============================================================================
--- llvm/trunk/lib/DebugInfo/PDB/UDTLayout.cpp (original)
+++ llvm/trunk/lib/DebugInfo/PDB/UDTLayout.cpp Wed Apr 12 18:27:43 2017
@@ -42,8 +42,8 @@ StorageItemBase::StorageItemBase(const U
                                  const PDBSymbol &Symbol,
                                  const std::string &Name,
                                  uint32_t OffsetInParent, uint32_t Size)
-    : Parent(Parent), Symbol(Symbol), Name(Name), SizeOf(Size),
-      OffsetInParent(OffsetInParent) {
+    : Parent(Parent), Symbol(Symbol), Name(Name),
+      OffsetInParent(OffsetInParent), SizeOf(Size) {
   UsedBytes.resize(SizeOf, true);
 }
 




More information about the llvm-commits mailing list