Page MenuHomeFreeBSD

llvm-objdump: initial man page
ClosedPublic

Authored by emaste on Nov 23 2018, 3:58 PM.
Tags
None
Referenced Files
F81623040: D18309.id51018.diff
Fri, Apr 19, 3:35 AM
Unknown Object (File)
Thu, Apr 18, 9:26 AM
Unknown Object (File)
Fri, Mar 29, 10:13 PM
Unknown Object (File)
Mar 18 2024, 7:27 PM
Unknown Object (File)
Mar 18 2024, 7:27 PM
Unknown Object (File)
Mar 18 2024, 7:27 PM
Unknown Object (File)
Mar 18 2024, 7:27 PM
Unknown Object (File)
Mar 18 2024, 7:27 PM

Details

Summary

Based on llvm-objdump's online documentation and usage information.

Also being submitted upstream as http://reviews.llvm.org/D54864. New features and option parsing improvements are being made upstream; I expect to use this one for LLVM 6 and 7, then switch to the upstream one when we import 8.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

yuripv added inline comments.
usr.bin/clang/llvm-objdump/llvm-objdump.1
11

Just IMO: listing all the long options in the synopsis section makes it unreadable (see diff(1), for example); if there are no short ones, just say llvm-objdump [OPTIONS] objfile ... like some of gnu utilities do (yeah, I know...), e.g., gnugrep(1).

emaste added inline comments.
usr.bin/clang/llvm-objdump/llvm-objdump.1
11

There are a mix of long and short options, some short ones not documented.

I had this discussion with Ingo Schwarze about the ld.lld.1 man page as well; he does not like eliminating the options in the synopsis, but also does not like long options at all.

I think you're right that the giant collection of long options is not that helpful, although note that GNU objdump does list all of its options in the synopsis.

Split mach-o options to subsection.

Did you ran the man page through "mandoc -Tlint" and textproc/igor to see if they spot any errors?
I found just one issue. Good to have a man page for it and even better to upstream it!

usr.bin/clang/llvm-objdump/llvm-objdump.1
151

Line break after the sentence stop.

Clean up mandoc -Tlint and/or igor complaints.

OK from manpages, others may have comments on the content itself.

This revision is now accepted and ready to land.Nov 23 2018, 7:28 PM
In D18309#388676, @bcr wrote:

OK from manpages, others may have comments on the content itself.

Thanks @bcr, I'll wait a bit for comments from others. I don't think this is a great man page, but is a starting point. I expect to iterate on it both in our tree and upstream.

This revision was automatically updated to reflect the committed changes.