[PATCH] D84670: [clang][cli] Port HeaderSearch simple string options to new option parsing system

2020-12-18 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 added a comment. Committing with whitespace fix and with `std::string` constructors removed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84670/new/ https://reviews.llvm.org/D84670 ___ cfe-

[PATCH] D84670: [clang][cli] Port HeaderSearch simple string options to new option parsing system

2020-12-18 Thread Jan Svoboda via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGf4511aec2bf4: [clang][cli] Port HeaderSearch simple string options to new option parsing… (authored by jansvoboda11). Changed prior to commit: htt

[PATCH] D84670: [clang][cli] Port HeaderSearch simple string options to new option parsing system

2020-12-17 Thread Michael Spencer via Phabricator via cfe-commits
Bigcheese accepted this revision. Bigcheese added a comment. This revision is now accepted and ready to land. lgtm with nit. Comment at: clang/include/clang/Driver/Options.td:1815 + HelpText<"Specify the interval (in seconds) after which a module file will be considered unuse

[PATCH] D84670: [clang][cli] Port HeaderSearch simple string options to new option parsing system

2020-12-16 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 updated this revision to Diff 312176. jansvoboda11 added a comment. Rebase, undo unnecessary move of options Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84670/new/ https://reviews.llvm.org/D84670 Files: clang/include/clang/Driver/