This is an archive of the discontinued LLVM Phabricator instance.

Accept not only --foo but also -foo.
ClosedPublic

Authored by ruiu on Sep 13 2017, 1:22 PM.

Details

Summary

GNU ld manual says that multi-letter long option can be prefixed with
either -- or -. Therefore, we should accept not only --subsystem but
also -subsystem, for example.

There is one exception. If an option starts with "o", it should only be
prefixed with -- to avoid ambiguity with -o<filename> option.

Diff Detail

Repository
rL LLVM

Event Timeline

ruiu created this revision.Sep 13 2017, 1:22 PM
emaste accepted this revision.Sep 13 2017, 6:21 PM
This revision is now accepted and ready to land.Sep 13 2017, 6:21 PM
mstorsjo accepted this revision.Sep 13 2017, 9:38 PM
This revision was automatically updated to reflect the committed changes.