diff options
author | Lang Hames <lhames@gmail.com> | 2018-08-18 18:38:37 +0000 |
---|---|---|
committer | Lang Hames <lhames@gmail.com> | 2018-08-18 18:38:37 +0000 |
commit | 8e296229b53691d5bfeb268ad405a21dcd0a0292 (patch) | |
tree | 5a9d24561577ef22c95810c262f0aec33bd0a84b /llvm/lib/ExecutionEngine | |
parent | 9c1761a6fd289ef104a1146a1648175924ad736b (diff) | |
download | bcm5719-llvm-8e296229b53691d5bfeb268ad405a21dcd0a0292.tar.gz bcm5719-llvm-8e296229b53691d5bfeb268ad405a21dcd0a0292.zip |
[RuntimeDyld] Fix a bug in RuntimeDyld::loadObjectImpl that was over-allocating
space for common symbols.
Patch by Dmitry Sidorov. Thanks Dmitry!
Differential revision: https://reviews.llvm.org/D50240
llvm-svn: 340125
Diffstat (limited to 'llvm/lib/ExecutionEngine')
-rw-r--r-- | llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp b/llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp index af5427ee7b8..df4bd5728a6 100644 --- a/llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp +++ b/llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp @@ -277,7 +277,7 @@ RuntimeDyldImpl::loadObjectImpl(const object::ObjectFile &Obj) { uint64_t Size = I->getCommonSize(); if (!CommonAlign) CommonAlign = Align; - CommonSize += alignTo(CommonSize, Align) + Size; + CommonSize = alignTo(CommonSize, Align) + Size; CommonSymbolsToAllocate.push_back(*I); } } else |