[llvm] 676c296 - Inline debug variable.

Dmitri Gribenko via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 30 01:31:52 PST 2020


Author: Michael Forster
Date: 2020-01-30T10:29:24+01:00
New Revision: 676c29694c5444ca3c63067770dfac0f37158797

URL: https://github.com/llvm/llvm-project/commit/676c29694c5444ca3c63067770dfac0f37158797
DIFF: https://github.com/llvm/llvm-project/commit/676c29694c5444ca3c63067770dfac0f37158797.diff

LOG: Inline debug variable.

Summary:
In a release build this variable becomes unused and may break the build
with `-Werror,-Wunused-variable`.

Reviewers: gribozavr2, jdoerfert, sstefan1

Reviewed By: gribozavr2

Subscribers: hiraditya, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D73683

Added: 
    

Modified: 
    llvm/lib/Transforms/IPO/Attributor.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/IPO/Attributor.cpp b/llvm/lib/Transforms/IPO/Attributor.cpp
index b23df5499b7a..1e45d3c08ddc 100644
--- a/llvm/lib/Transforms/IPO/Attributor.cpp
+++ b/llvm/lib/Transforms/IPO/Attributor.cpp
@@ -13,7 +13,7 @@
 //
 //===----------------------------------------------------------------------===//
 
-#include "llvm/Transforms/IPO/Attributor.h"
+#include "llvm/Transforms/IPO/Attributor.h" 
 
 #include "llvm/ADT/DepthFirstIterator.h"
 #include "llvm/ADT/STLExtras.h"
@@ -4992,7 +4992,6 @@ struct AAPrivatizablePtrArgument final : public AAPrivatizablePtrImpl {
     // Helper to check if for the given call site the associated argument is
     // passed to a callback where the privatization would be 
diff erent.
     auto IsCompatiblePrivArgOfCallback = [&](CallSite CS) {
-      Value *CSArgOp = CS.getArgOperand(ArgNo);
       SmallVector<const Use *, 4> CBUses;
       AbstractCallSite::getCallbackUses(CS, CBUses);
       for (const Use *U : CBUses) {
@@ -5010,7 +5009,8 @@ struct AAPrivatizablePtrArgument final : public AAPrivatizablePtrImpl {
                    "callback ("
                 << CBArgNo << "@" << CBACS.getCalledFunction()->getName()
                 << ")\n[AAPrivatizablePtr] " << CBArg << " : "
-                << CBACS.getCallArgOperand(CBArg) << " vs " << CSArgOp << "\n"
+                << CBACS.getCallArgOperand(CBArg) << " vs "
+                << CS.getArgOperand(ArgNo) << "\n"
                 << "[AAPrivatizablePtr] " << CBArg << " : "
                 << CBACS.getCallArgOperandNo(CBArg) << " vs " << ArgNo << "\n";
           });


        


More information about the llvm-commits mailing list