<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 4, 2015 at 6:23 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 4 June 2015 at 03:27, Alexey Samsonov <<a href="mailto:vonosmas@gmail.com">vonosmas@gmail.com</a>> wrote:<br>
> I don't know, why should it?<br>
><br>
> llvm-objdump is not documented in <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_docs_CommandGuide_&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=PHVaDU0UX5G-lW36JBtKsOzMi5hnpywQPp8xR2aPYSY&s=lQ0UUysNKFTGUoKL2N-_opsq3ROZDX-JfsUKhHw2WuU&e=" target="_blank">http://llvm.org/docs/CommandGuide/</a>, and<br>
> now the behavior of the tool is the same (return 0) for these invalid MachO<br>
> files and missing input files. I agree that we should probably decide and<br>
> document the behavior of the tool for invalid input, but that seems<br>
> orthogonal to this change - we just make all the possible failures in<br>
> constructor uniform.<br>
<br>
</span>It is modeled after objdump, and that one exits with non-zero:<br>
<br>
objdump  -d Inputs/corrupt.elf-x86-64<br>
objdump: Inputs/corrupt.elf-x86-64: invalid string offset 1048625 >=<br>
75 for section `.shstrtab'<br>
objdump: Inputs/corrupt.elf-x86-64: invalid string offset 1048625 >=<br>
75 for section `.shstrtab'<br>
objdump: Inputs/corrupt.elf-x86-64: Bad value<br>
$ echo $?<br>
1<br>
<br>
IMHO we should not regress that. It should be fixed to check the error<br>
code first.<br></blockquote><div><br></div><div>Agree. r239065.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Alexey Samsonov<br><a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a></div></div>
</div></div>