Previously, the llvm-readelf documentation was essentially just a list of differences to llvm-readobj. Since llvm-readelf is the more likely goto tool for many people migrating to the LLVM toolchain, it seems like it would be helpful to document all the switches in the llvm-readelf document too. This change expands the options listed accordingly. Additionally, they are unlikely to care what the differences are to llvm-readobj, since they won't be familiar with the latter as there is no GNU equivalent, so this change moves the "differences" section to llvm-readobj's documentation.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
Overall no objections. Only thing I can think of that could be surprising is the presence of mach-o and coff options in llvm-readelf. A case could be made for leaving these out as people migrating won't need to use them, but I don't have a strong opinion.
Comment Actions
I'd be okay with this, but in that case, I'd recommend hiding the options in the help text too, so that's possibly tangential to this change? I'll post something on the mailing list to see if anybody else has any thoughts.
Comment Actions
Rebase + remove Mach-O and COFF options from the llvm-readelf documentation. I'll upload another review at a later point to remove the same options from llvm-readelf's help text.
Comment Actions
Thanks. I'm committing this now, but I'm happy for more feedback if people want to give any.