[PATCH] D25518: [ELF] - Handle broken size field of compressed sections header.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 12 09:07:24 PDT 2016


grimar added a comment.

In https://reviews.llvm.org/D25518#568300, @davide wrote:

> Independently from this patch. Are we really trying to fully support 32-bit hosts? We `mmap()` the whole set of input  which I think very likely causes shortage of VA on 32-bit platforms.


Do you think we should check if 32 bits host is used to link 64 bit app and just error out before trying to link ?


https://reviews.llvm.org/D25518





More information about the llvm-commits mailing list