This is an archive of the discontinued LLVM Phabricator instance.

[Driver] Add --config= as canonical spelling of --config
ClosedPublic

Authored by MaskRay on Sep 27 2022, 11:05 PM.

Details

Summary

Driver options usually use Joined instead of Separate. It is also weird that
--config-system-dir=/etc exist while --config= did not exist.

Diff Detail

Event Timeline

MaskRay created this revision.Sep 27 2022, 11:05 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 27 2022, 11:05 PM
Herald added a subscriber: StephenFan. · View Herald Transcript
MaskRay requested review of this revision.Sep 27 2022, 11:05 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 27 2022, 11:05 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript

--config is deliberately not a CoreOption as we want to discourage the form. --config was not CoreOption in previous releases.

mgorny accepted this revision.Sep 29 2022, 12:37 AM

I don't know the fancy option magic you're using here, I haven't tested it but it seems to make sense to me.

This revision is now accepted and ready to land.Sep 29 2022, 12:37 AM
This revision was automatically updated to reflect the committed changes.