<div dir="ltr">Hi Guys,<div><br></div><div>Yeah - this is definitely an improvement. Tests would be great though. It looks like we need a few malformed objects: (1) Duplicate <span style="font-size:13px">SymtabLoadCmd, (2) Duplicate </span><span style="font-size:13px">DysymtabLoadCmd, (3) Duplicate </span><span style="font-size:13px">DataInCodeLoadCmd, (4) Duplicate DyldInfoLoadCmd, and (5) Duplicate </span><span style="font-size:13px">UuidLoadCmd.</span><div><span style="font-size:13px"><br></span></div><div><span style="font-size:13px">I've filed </span><a href="http://llvm.org/PR22343">http://llvm.org/PR22343</a> - if anyone wants to dive on it feel free, otherwise I'll try to get to it (time permitting) in the next few days.</div><div><br></div><div>Cheers,</div><div>Lang.</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 23, 2015 at 3:52 PM, 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"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On 23 January 2015 at 17:14, Kevin Enderby <span dir="ltr"><<a href="mailto:enderby@apple.com" target="_blank">enderby@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Added Lang to the To: line and he maybe better to comment.<div><br></div><div>While the change looks cleaner I’m not sure what the error handling should be for malformed Mach-O files in llvm’s libObject.  If malformed Mach-O files cause asserts it is difficult to build tools to print and diagnose bad files.  For example in Mac OS otool(1) is used both to display information about good Mach-O files and do its best to display what it can with warning about  malformed Mach-O files.  For the guy producing Mach-O files which some times are broken this is a valuable approach for have otool(1) be able so show what it can instead of just stopping.</div></div></blockquote><div><br></div></span><div>I agree that it should not assert.</div><div><br></div><div>The intention was to point out that the original change was (and still is) untested.</div><div><br></div><div>Cheers,</div><div>Rafael</div><div><br></div></div></div></div>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>