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

Luke Drummond via lldb-commits lldb-commits at lists.llvm.org
Thu Jan 21 09:27:41 PST 2016


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 appear to be any side-effects from removing this.

http://reviews.llvm.org/D16412

Files:
  source/Plugins/ExpressionParser/Clang/IRForTarget.cpp

Index: source/Plugins/ExpressionParser/Clang/IRForTarget.cpp
===================================================================
--- source/Plugins/ExpressionParser/Clang/IRForTarget.cpp
+++ source/Plugins/ExpressionParser/Clang/IRForTarget.cpp
@@ -163,8 +163,6 @@
 {
     llvm_function.setLinkage(GlobalValue::ExternalLinkage);
 
-    std::string name = llvm_function.getName().str();
-
     return true;
 }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16412.45542.patch
Type: text/x-patch
Size: 411 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20160121/dca822a2/attachment.bin>


More information about the lldb-commits mailing list