Re: [Lldb-commits] [PATCH] D16412: remove unused local string in IRForTarget.cpp

2016-02-26 Thread Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL262043: remove unused local string in IRForTarget.cpp (authored by aidandodds). Changed prior to commit: http://reviews.llvm.org/D16412?vs=48006&id=49206#toc Repository: rL LLVM http://reviews.llvm.

Re: [Lldb-commits] [PATCH] D16412: remove unused local string in IRForTarget.cpp

2016-02-26 Thread Aidan Dodds via lldb-commits
ADodds added a subscriber: ADodds. ADodds accepted this revision. ADodds added a reviewer: ADodds. ADodds added a comment. This revision is now accepted and ready to land. I'm happy to commit this for you. http://reviews.llvm.org/D16412 ___ lldb-com

Re: [Lldb-commits] [PATCH] D16412: remove unused local string in IRForTarget.cpp

2016-02-15 Thread Luke Drummond via lldb-commits
ldrumm updated this revision to Diff 48006. ldrumm added a comment. rebased on current HEAD http://reviews.llvm.org/D16412 Files: source/Plugins/ExpressionParser/Clang/IRForTarget.cpp Index: source/Plugins/ExpressionParser/Clang/IRForTarget.cpp ===

[Lldb-commits] [PATCH] D16412: remove unused local string in IRForTarget.cpp

2016-01-21 Thread Luke Drummond via lldb-commits
ldrumm created this revision. ldrumm added reviewers: spyffe, jingham. ldrumm added a subscriber: lldb-commits. It looks like there's an unused variable in `source/Plugins/Expression/Clang/IRForTarget.cpp` `std::string name` is assigned the function name, then never used again. ; There don't app