<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Mar 19, 2018 at 2:24 PM Rafael Avila de Espindola via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">espindola added inline comments.<br>
<br>
<br>
================<br>
Comment at: test/ELF/undef.s:37<br>
<br>
+# CHECK: warning: parsing line table prologue at 0x00000000 should have ended at 0x00000038 but it ended at 0x00000037<br>
 # CHECK: error: undefined symbol: zed6<br>
----------------<br>
I think I agree with George about producing an error on corrupted output. In practice we expect to never see it, so we may as well make the code simpler.<br></blockquote><div><br>Seems like it'd be unfortunate if you had a shipped binary library with some bogus DWARF (or, say, we had a bug in the DWARF parser that caused a parse failure?) & you had linker errors you couldn't diagnose because the linker just told you about the bogus DWARF but not about the missing symbol you could act on (by, say, linking in some other library you forgot to pass to the linker?)? (if I'm understanding correctly here that you're suggesting erroring on the bogus DWARF and stopping there/not providing the undefined symbol error?)<br><br>- Dave<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
Repository:<br>
  rLLD LLVM Linker<br>
<br>
<a href="https://reviews.llvm.org/D44562" rel="noreferrer" target="_blank">https://reviews.llvm.org/D44562</a><br>
<br>
<br>
<br>
</blockquote></div></div>