This is an archive of the discontinued LLVM Phabricator instance.

llvm-nm: Add -radix option
ClosedPublic

Authored by khemant on Feb 2 2016, 2:02 PM.

Details

Summary

nm can print symbol values in different radixes. This option enables llvm-nm to output similarly.

Diff Detail

Repository
rL LLVM

Event Timeline

khemant updated this revision to Diff 46698.Feb 2 2016, 2:02 PM
khemant retitled this revision from to Add -radix option.
khemant updated this object.
khemant added reviewers: llvm-commits, echristo.
khemant set the repository for this revision to rL LLVM.
khemant retitled this revision from Add -radix option to llvm-nm: Add -radix option.
khemant added subscribers: llvm-commits, davide, echristo.
echristo accepted this revision.Feb 4 2016, 2:04 PM
echristo edited edge metadata.

LGTM.

-eric

This revision is now accepted and ready to land.Feb 4 2016, 2:04 PM

I will modify the test to have source instead of binary and assemble it.

Any reason? (Personally I don't mind and prefer that, but I'm also not
going to insist on it because other people like it the other way)

-eric

This revision was automatically updated to reflect the committed changes.