[PATCH] D35793: [ELF] - Change way how we handle --noinhibit-exec

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 25 00:47:01 PDT 2017


>The intention is to prevent almost all errors, no? So maybe you want to
>check NoinhibitExec in error() and the places that do have to fail can use
>fatal()?
>
>Cheers,
>Rafael

I tried to do something like this in initial diff of: https://reviews.llvm.org/D35724?id=107665,
but changed to current approach right after that, because I found it to much aggressive strategy.
Option saying "Retain the executable output file whenever it is still usable" but we currently
often do early returns when have errorCount>0 and it hard to say what logic is skipped and whether
output is 'usable'.

And I think we were trying to avoid 'fatal' errors as much as possible, I would probably not change
this strategy because of --noinhibit-exec.

George.


More information about the llvm-commits mailing list