[llvm-nm] Fix docs and help text for --print-size

[jhenderson]

Summary

The --print-size help text and documentation claimed that the size was printed instead of the address, but this is incorrect. It is printed as well as the address. This patch fixes this issue.

[ruiu]

LGTM

[MaskRay]

docs/CommandGuide/llvm-nm.rst

124 ↗

(On Diff #204078)

MachO files do not record the size. (There is currently a warning) You may add a note about this if useful.

[jhenderson]

docs/CommandGuide/llvm-nm.rst

124 ↗

(On Diff #204078)

Sure. I'll add a note to the doc indicating this is not applicable for Mach-O.

[jhenderson]

Add note about Mach-O.

llvm/trunk/docs/CommandGuide/llvm-nm.rst

  Sort symbols by address.
 
 .. option:: --print-file-name, -A, -o
 
  Precede each symbol with the file it came from.
 
 .. option:: --print-size, -S
 
- Show symbol size instead of address.
+ Show symbol size as well as address (not applicable for Mach-O).
 
 .. option:: --size-sort
 
  Sort symbols by size.
 
 .. option:: --undefined-only, -u
 
  Print only symbols referenced but not defined in this file.

llvm/trunk/tools/llvm-nm/llvm-nm.cpp

cl::opt<bool> NoDemangle("no-demangle", cl::init(false), cl::ZeroOrMore,
                          cl::cat(NMCat));
 
 cl::opt<bool> ReverseSort("reverse-sort", cl::desc("Sort in reverse order"),
                           cl::cat(NMCat));
 cl::alias ReverseSortr("r", cl::desc("Alias for --reverse-sort"),
                        cl::aliasopt(ReverseSort), cl::Grouping);
 
 cl::opt<bool> PrintSize("print-size",
-                        cl::desc("Show symbol size instead of address"),
+                        cl::desc("Show symbol size as well as address"),
                         cl::cat(NMCat));
 cl::alias PrintSizeS("S", cl::desc("Alias for --print-size"),
                      cl::aliasopt(PrintSize), cl::Grouping);
 bool MachOPrintSizeWarning = false;
 
 cl::opt<bool> SizeSort("size-sort", cl::desc("Sort symbols by size"),
                        cl::cat(NMCat));