[PATCH] D50240: [RuntimeDyld] Fix a calculation of common symbols size

[Lang Hames via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL340125: [RuntimeDyld] Fix a bug in RuntimeDyld::loadObjectImpl that was over-allocating (authored by lhames, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D50240?vs=158970&id=161377#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D50240

Files:
  llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp


Index: llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp
===================================================================
--- llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp
+++ llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp
@@ -277,7 +277,7 @@
           uint64_t Size = I->getCommonSize();
           if (!CommonAlign)
             CommonAlign = Align;
-          CommonSize += alignTo(CommonSize, Align) + Size;
+          CommonSize = alignTo(CommonSize, Align) + Size;
           CommonSymbolsToAllocate.push_back(*I);
         }
       } else


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50240.161377.patch
Type: text/x-patch
Size: 584 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180818/77d0c7cb/attachment.bin>