[PATCH] [Commented On] D25082: [ELF] - Do not crash on invalid section alignment.

Eugene Leviant via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 30 02:17:26 PDT 2016


evgeny777 added inline comments.


> InputSection.cpp:48
>    // no alignment constraits.
> +  if (Header->sh_addralign > UINT32_MAX)
> +    fatal(getFilename(File) + ": section sh_addralign is too large");

This doesn't make any sense for ELF32, does it? What about using MaxPageSize?

https://reviews.llvm.org/D25082





More information about the llvm-commits mailing list