diff options
author | Rui Ueyama <ruiu@google.com> | 2015-01-30 01:52:23 +0000 |
---|---|---|
committer | Rui Ueyama <ruiu@google.com> | 2015-01-30 01:52:23 +0000 |
commit | afbe58f88b7d4710e1b921f1437b45f243322802 (patch) | |
tree | 1dc97eb8a0afea91108d22e7e675bbd8f0d48316 | |
parent | 3084aff4988ccabc7e8f604edcb5cf1acfa13e56 (diff) | |
download | bcm5719-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.h | 16 |
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); } }; |