diff options
| author | Benjamin Kramer <benny.kra@googlemail.com> | 2015-09-24 14:48:49 +0000 |
|---|---|---|
| committer | Benjamin Kramer <benny.kra@googlemail.com> | 2015-09-24 14:48:49 +0000 |
| commit | 72e64317a7edb385235ed9b5f81f747d545ec1a6 (patch) | |
| tree | 804ebae90a12ebbd3b1183ff5a0bc1125887c40f /clang/lib/Driver/Tools.cpp | |
| parent | e8b764144e5f5b9e01f7bcecb0b6e9d0eb9990e7 (diff) | |
| download | llvm-72e64317a7edb385235ed9b5f81f747d545ec1a6.zip llvm-72e64317a7edb385235ed9b5f81f747d545ec1a6.tar.gz llvm-72e64317a7edb385235ed9b5f81f747d545ec1a6.tar.bz2 | |
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
Diffstat (limited to 'clang/lib/Driver/Tools.cpp')
| -rw-r--r-- | clang/lib/Driver/Tools.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/clang/lib/Driver/Tools.cpp b/clang/lib/Driver/Tools.cpp index 682c7e5..e13144f 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<StringRef, 8> Split; text.split(Split, StringRef("+"), -1, false); - for (const StringRef Feature : Split) { + for (StringRef Feature : Split) { const char *result = llvm::StringSwitch<const char *>(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; |