[PATCH] D13138: [ELF2] - add support of --noinhibit-exec flag

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 24 11:37:26 PDT 2015


ruiu added a comment.

BTW, is this guaranteed to be safe? I think we currently do not expect any undefined symbols to appear in Writer because we abort early if such symbol exists. This patch changes that assumption.

In COFF, we overwrite undefined symbols with absolute symbols with value 0 if /force (Windows option for -no-inhibit-exec). Maybe we should do that?

Rafael?


http://reviews.llvm.org/D13138





More information about the llvm-commits mailing list