[PATCH] D47602: Correct aligment computation for shared object symbols
Han Shen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 1 14:25:20 PDT 2018
shenhan added inline comments.
================
Comment at: ELF/InputFiles.cpp:910
if (Ret > UINT32_MAX)
error(toString(this) + ": alignment too large: " +
CHECK(Sym.getName(this->StringTable), this));
----------------
pcc wrote:
> I was thinking that instead of erroring here you would return 0. Then you wouldn't need the special case for `SHN_ABS`.
Yup, done.
https://reviews.llvm.org/D47602
More information about the llvm-commits
mailing list