This is an archive of the discontinued LLVM Phabricator instance.

[clang][cli] Streamline MarhsallingInfoFlag description

Authored by jansvoboda11 on Jul 27 2020, 10:01 AM.



This replaces the existing MarshallingInfoFlag<...>, IsNegative with simpler MarshallingInfoNegativeFlag.

Diff Detail

Event Timeline

dang created this revision.Jul 27 2020, 10:01 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald Transcript
jansvoboda11 commandeered this revision.Dec 18 2020, 7:10 AM
jansvoboda11 added a reviewer: dang.
jansvoboda11 added a subscriber: jansvoboda11.

Taking over this patch, as Daniel is no longer involved.

jansvoboda11 retitled this revision from Streamline MarhsallingInfoFlag description to [clang][cli] Streamline MarhsallingInfoFlag description.Dec 18 2020, 7:13 AM
jansvoboda11 edited the summary of this revision. (Show Details)

Rebase, remove DefaultValue to avoid confusion with Default used for BoolOptionBase.

jansvoboda11 edited reviewers, added: dexonsmith; removed: jdoerfert.Dec 18 2020, 7:15 AM
This revision is now accepted and ready to land.Dec 18 2020, 1:13 PM
This revision was landed with ongoing or failed builds.Dec 21 2020, 2:33 AM
This revision was automatically updated to reflect the committed changes.