From 72e64317a7edb385235ed9b5f81f747d545ec1a6 Mon Sep 17 00:00:00 2001 From: Benjamin Kramer Date: Thu, 24 Sep 2015 14:48:49 +0000 Subject: Drop useless const in for-range loops. StringRefs always point to immutable memory so the const doesn't add value here. Also quiets clang's -Wrange-loop-analysis which warns about the implicit copying. llvm-svn: 248496 --- clang/lib/Driver/Tools.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'clang/lib/Driver/Tools.cpp') diff --git a/clang/lib/Driver/Tools.cpp b/clang/lib/Driver/Tools.cpp index 682c7e5255e..e13144ffebf 100644 --- a/clang/lib/Driver/Tools.cpp +++ b/clang/lib/Driver/Tools.cpp @@ -1935,7 +1935,7 @@ static bool DecodeAArch64Features(const Driver &D, StringRef text, SmallVector Split; text.split(Split, StringRef("+"), -1, false); - for (const StringRef Feature : Split) { + for (StringRef Feature : Split) { const char *result = llvm::StringSwitch(Feature) .Case("fp", "+fp-armv8") .Case("simd", "+neon") @@ -2344,7 +2344,7 @@ static void CollectArgsForIntegratedAssembler(Compilation &C, Args.filtered(options::OPT_Wa_COMMA, options::OPT_Xassembler)) { A->claim(); - for (const StringRef Value : A->getValues()) { + for (StringRef Value : A->getValues()) { if (TakeNextArg) { CmdArgs.push_back(Value.data()); TakeNextArg = false; -- cgit v1.2.3