<div dir="ltr"><div dir="ltr"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature">On Mon, Jun 24, 2019 at 8:16 AM James Henderson via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div></div></div><div class="gmail_quote"><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>I just was discussing a couple of features in llvm-readobj and whether we should change them slightly:</div><div><br></div><div>1) llvm-readobj --all currently matches GNU readelf's --all switch in that it is equivalent to specifying a bundle of different options (including --file-headers, --program-headers, --symbols, --version-info etc). However, --all does not actually specify "all" dump types. For example, it doesn't dump ELF section groups or the addrsig section (--elf-section-groups and --addrsig respectively). I'd propose that it should dump all relevant dumping information. Ideally, I'd suggest this difference for both llvm-readobj and llvm-readelf, but I can understand nervousness in the additional dumps that this would produce in llvm-readelf versus GNU readelf.</div><div><br></div><div>2) llvm-readobj --notes prints the SHT_NOTE sections unless the ELF type is ET_CORE, in which case it prints the contents of the PT_NOTE segments and NOT the SHT_NOTE sections. It seems to me like there's no good reason to not try both for all file formats. For example, it's perfectly reasonable to have PT_NOTE segments in an ET_EXEC or ET_DYN object file. What would people think about removing this distinction? In this case, I'd propose printing all PT_NOTE segments and all SHT_NOTE sections not in PT_NOTE segments. In all likelihood, this would produce no different behaviour to the old behaviour.</div><div><br></div><div>Thoughts?</div><div><br></div><div>James<br></div></div><br></blockquote><div><br></div><div>I'm fine with both changes.</div><div><br></div><div>For the first, I prefer llvm-readelf --all mimics gnu readelf.</div><div><br></div><div>- Michael Spencer </div></div></div>