<div dir="ltr">My perspective is that llvm-readobj is a testing tool whose goal should be to make testing LLVM easy, not 100% GNU readelf compatibility, but I think that ship has sailed.<div><br></div><div>I don't really care if you decide to move the functionality, I just want to have some binary in LLVM for exploratory dumpers, where I don't have to care too much about consistency with another tool that makes different arbitrary formatting decisions. So, if you want to move it, I guess we need some new binary where we can put experimental code. "llvm-objutil" mirroring llvm-pdbutil?</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 27, 2019 at 3:03 AM James Henderson via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Hi all,</div><div><br></div><div>llvm-readelf is an alias for llvm-readobj which aims for GNU compatibility and is likely the tool that most people migrating to the LLVM binutils will adopt instead of llvm-readobj. Because it is just an alias, it has inherited the functionality provided by llvm-readobj, including for non-ELF targets, with Mach-O and COFF-specific switches available in its interface. People migrating from GNU readelf won't need these switches.</div><div><br></div><div>I have a change up for review to expand the documentation for llvm-readelf to list all its supported switches (see <a href="https://reviews.llvm.org/D63826" target="_blank">https://reviews.llvm.org/D63826</a>). This patch currently includes the Mach-O and COFF specific switches. It was suggested that these could be surprising and unnecessary, and could be removed from the patch. If we go ahead and remove them from the documentation, I think we should remove them from the help text, or possibly even the available options entirely. I'd be happy to do the work for this, if the community agrees that llvm-readelf should only support ELF-related options.</div><div><br></div><div>Thoughts?</div><div><br></div><div>James<br></div></div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>