summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRui Ueyama <ruiu@google.com>2015-01-30 01:52:23 +0000
committerRui Ueyama <ruiu@google.com>2015-01-30 01:52:23 +0000
commitafbe58f88b7d4710e1b921f1437b45f243322802 (patch)
tree1dc97eb8a0afea91108d22e7e675bbd8f0d48316
parent3084aff4988ccabc7e8f604edcb5cf1acfa13e56 (diff)
downloadbcm5719-llvm-afbe58f88b7d4710e1b921f1437b45f243322802.tar.gz
bcm5719-llvm-afbe58f88b7d4710e1b921f1437b45f243322802.zip
Explicitly write type instead of auto.
Now it is clear that std::move() is meaningless here. llvm-svn: 227543
-rw-r--r--lld/include/lld/Core/Simple.h16
1 files changed, 8 insertions, 8 deletions
diff --git a/lld/include/lld/Core/Simple.h b/lld/include/lld/Core/Simple.h
index 789fd44f939..e13c0f459a1 100644
--- a/lld/include/lld/Core/Simple.h
+++ b/lld/include/lld/Core/Simple.h
@@ -80,14 +80,14 @@ protected:
class SimpleFileWrapper : public SimpleFile {
public:
SimpleFileWrapper(const File &file) : SimpleFile(file.path()) {
- for (auto definedAtom : file.defined())
- _definedAtoms._atoms.push_back(std::move(definedAtom));
- for (auto undefAtom : file.undefined())
- _undefinedAtoms._atoms.push_back(std::move(undefAtom));
- for (auto shlibAtom : file.sharedLibrary())
- _sharedLibraryAtoms._atoms.push_back(std::move(shlibAtom));
- for (auto absAtom : file.absolute())
- _absoluteAtoms._atoms.push_back(std::move(absAtom));
+ for (const DefinedAtom *atom : file.defined())
+ _definedAtoms._atoms.push_back(atom);
+ for (const UndefinedAtom *atom : file.undefined())
+ _undefinedAtoms._atoms.push_back(atom);
+ for (const SharedLibraryAtom *atom : file.sharedLibrary())
+ _sharedLibraryAtoms._atoms.push_back(atom);
+ for (const AbsoluteAtom *atom : file.absolute())
+ _absoluteAtoms._atoms.push_back(atom);
}
};
OpenPOWER on IntegriCloud