[PATCH] D16668: ELF: Remove fatal() and add the second parameter to the linker's entry point.

Rafael Ávila de Espíndola via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 1 05:36:44 PST 2016


rafael added a comment.

I don't think so.

The problem is that we were not at 100% coverage with fatal. There are many cases where we would just follow a broken offset and crash. If we can produce a segmentation fault, we can call fatal.

If we cannot produce a segmentation fault, we will need another way of fixing this.


http://reviews.llvm.org/D16668





More information about the llvm-commits mailing list