Page MenuHomePhabricator

[docs][llvm-nm] Add missing options to documentation
ClosedPublic

Authored by jhenderson on Jun 21 2019, 9:03 AM.

Details

Summary

There were several options missing from the documentation. This patch adds them as well as improving some wording and separating the Mach-O only options into a separate section.

Fixes https://bugs.llvm.org/show_bug.cgi?id=42234.

Diff Detail

Event Timeline

jhenderson created this revision.Jun 21 2019, 9:03 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 21 2019, 9:03 AM
MaskRay added inline comments.Fri, Jun 21, 6:58 PM
docs/CommandGuide/llvm-nm.rst
72 ↗(On Diff #206014)

-B is the default, even on macOS (explicit -m to enable the Darwin style)

jhenderson marked an inline comment as done.Mon, Jun 24, 1:58 AM
jhenderson added inline comments.
docs/CommandGuide/llvm-nm.rst
72 ↗(On Diff #206014)

Since -B is an alias for --format=bsd, we don't need to mention it in both places. Note the following bit from the --format description: The default is *bsd*.

MaskRay accepted this revision.Mon, Jun 24, 2:37 AM
This revision is now accepted and ready to land.Mon, Jun 24, 2:37 AM

Thanks @MaskRay. I'm going to land this shortly, to reduce the number of in-flight patches I have on the docs, but I'll happily take further feedback for improvements if anybody has any.

This revision was automatically updated to reflect the committed changes.