This is an archive of the discontinued LLVM Phabricator instance.

[llvm-symbolizer] Remove one-dash long options
ClosedPublic

Authored by MaskRay on Jul 20 2021, 9:39 AM.

Details

Summary

Most modern tools only accept two-dash long options. Remove one-dash
long options which are not recognized by GNU style getopt_long.
This ensures long options cannot collide with grouped short options.

Note: llvm-symbolizer has -demangle={true,false} for pprof compatibility
(for a while). They are kept.

Diff Detail

Event Timeline

MaskRay created this revision.Jul 20 2021, 9:39 AM
MaskRay requested review of this revision.Jul 20 2021, 9:39 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 20 2021, 9:39 AM
MaskRay updated this revision to Diff 360365.Jul 20 2021, 11:39 PM

fix tests

jhenderson accepted this revision.Jul 23 2021, 1:53 AM

Looks fine to me.

This revision is now accepted and ready to land.Jul 23 2021, 1:53 AM
This revision was landed with ongoing or failed builds.Jul 23 2021, 8:35 AM
This revision was automatically updated to reflect the committed changes.