diff options
author | Richard Smith <richard-llvm@metafoo.co.uk> | 2017-02-18 01:14:43 +0000 |
---|---|---|
committer | Richard Smith <richard-llvm@metafoo.co.uk> | 2017-02-18 01:14:43 +0000 |
commit | 0aef305f352235fe66d44e8f253b3b09d27a1b10 (patch) | |
tree | 9fabfb4edc99fe647b3f8e0ef67aafdd89e65da9 | |
parent | a84404c7a46f0a6a3d5be6cf707b08d6ab80f517 (diff) | |
download | bcm5719-llvm-0aef305f352235fe66d44e8f253b3b09d27a1b10.tar.gz bcm5719-llvm-0aef305f352235fe66d44e8f253b3b09d27a1b10.zip |
Cleanup: use range-based for rather than separate calls to begin and end.
llvm-svn: 295524
-rw-r--r-- | clang/lib/Driver/Driver.cpp | 17 | ||||
-rw-r--r-- | clang/tools/driver/cc1as_main.cpp | 10 |
2 files changed, 12 insertions, 15 deletions
diff --git a/clang/lib/Driver/Driver.cpp b/clang/lib/Driver/Driver.cpp index 0dcab7c0c6c..b8ae7e4c312 100644 --- a/clang/lib/Driver/Driver.cpp +++ b/clang/lib/Driver/Driver.cpp @@ -1456,16 +1456,15 @@ void Driver::BuildInputs(const ToolChain &TC, DerivedArgList &Args, ? types::TY_C : types::TY_CXX; - arg_iterator it = - Args.filtered_begin(options::OPT__SLASH_TC, options::OPT__SLASH_TP); - const arg_iterator ie = Args.filtered_end(); - Arg *Previous = *it++; + Arg *Previous = nullptr; bool ShowNote = false; - while (it != ie) { - Diag(clang::diag::warn_drv_overriding_flag_option) - << Previous->getSpelling() << (*it)->getSpelling(); - Previous = *it++; - ShowNote = true; + for (Arg *A : Args.filtered(options::OPT__SLASH_TC, options::OPT__SLASH_TP)) { + if (Previous) { + Diag(clang::diag::warn_drv_overriding_flag_option) + << Previous->getSpelling() << A->getSpelling(); + ShowNote = true; + } + Previous = A; } if (ShowNote) Diag(clang::diag::note_drv_t_option_is_global); diff --git a/clang/tools/driver/cc1as_main.cpp b/clang/tools/driver/cc1as_main.cpp index 263751346b8..2fa8edb81ae 100644 --- a/clang/tools/driver/cc1as_main.cpp +++ b/clang/tools/driver/cc1as_main.cpp @@ -212,13 +212,11 @@ bool AssemblerInvocation::CreateFromArgs(AssemblerInvocation &Opts, // Frontend Options if (Args.hasArg(OPT_INPUT)) { bool First = true; - for (arg_iterator it = Args.filtered_begin(OPT_INPUT), - ie = Args.filtered_end(); - it != ie; ++it, First = false) { - const Arg *A = it; - if (First) + for (const Arg *A : Args.filtered(OPT_INPUT)) { + if (First) { Opts.InputFile = A->getValue(); - else { + First = false; + } else { Diags.Report(diag::err_drv_unknown_argument) << A->getAsString(Args); Success = false; } |