
<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">It changes the moment we emit the error (and I agree that the earlier the better usually, even though there may be cases where you consider a module but don’t include it in the link for instance), but I’m not sure why you think that the current error does not easily allow to diagnose which file has the problem, since it returns an error on the call to `LTO.add(InputFile)` ?<div class=""><br class=""><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Dec 13, 2016, at 8:43 PM, Peter Collingbourne <<a href="mailto:peter@pcc.me.uk" class="">peter@pcc.me.uk</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="auto" class=""><div class="">It could be more useful to do this in InputFile::create, that way the client can easily diagnose which file has the problem.<div dir="auto" class=""><br class=""></div><div dir="auto" class="">Peter</div><br class=""><div class="gmail_extra"><br class=""><div class="gmail_quote">On Dec 13, 2016 8:36 PM, "Mehdi AMINI via Phabricator" <<a href="mailto:reviews@reviews.llvm.org" class="">reviews@reviews.llvm.org</a>> wrote:<br type="attribution" class=""><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">mehdi_amini accepted this revision.<br class="">

mehdi_amini added a comment.<br class="">

This revision is now accepted and ready to land.<br class="">

<br class="">

LGTM.<br class="">

<br class="">

<br class="">

<a href="https://reviews.llvm.org/D27082" rel="noreferrer" target="_blank" class="">https://reviews.llvm.org/<wbr class="">D27082</a><br class="">

<br class="">

<br class="">

<br class="">

</blockquote></div><br class=""></div></div></div>

</div></blockquote></div><br class=""></div></div></body></html>