[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