This is an archive of the discontinued LLVM Phabricator instance.

[llvm-objdump] Add --no-addresses as an alias for --no-leading-addr
ClosedPublic

Authored by MaskRay on Oct 2 2022, 11:36 AM.

Details

Summary

The output is similar to objdump --no-addresses since binutils 2.35.

Depends on D135039
Close #58088

Diff Detail

Event Timeline

MaskRay created this revision.Oct 2 2022, 11:36 AM
MaskRay requested review of this revision.Oct 2 2022, 11:36 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 2 2022, 11:36 AM
nickdesaulniers accepted this revision.Oct 3 2022, 9:35 AM
This revision is now accepted and ready to land.Oct 3 2022, 9:35 AM
jhenderson added inline comments.Oct 13 2022, 1:01 AM
llvm/tools/llvm-objdump/ObjdumpOpts.td
133

Just a quick note: unless something has changed at some point (I don't have a quick way to build and check llvm-objdump currently, or I'd check myself), because this has no help text, I believe it won't appear in the list of options listed by --help. This means that the documentation and the help text won't agree with one another, which seems less than ideal.

MaskRay added inline comments.Oct 13 2022, 1:03 AM
llvm/tools/llvm-objdump/ObjdumpOpts.td
133

Append , HelpText<"Alias for --no-leading-addr">?

jhenderson added inline comments.Oct 13 2022, 1:10 AM
llvm/tools/llvm-objdump/ObjdumpOpts.td
133

Yep, that would be in keeping with the surrounding code.