[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