[llvm] e9ff53d - [MemProf] Fix buildbot error due to unused variable from bad merge
Teresa Johnson via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 22 13:24:47 PDT 2022
Author: Teresa Johnson
Date: 2022-09-22T13:24:33-07:00
New Revision: e9ff53d42feac7fc157718523275619a8106f2f3
URL: https://github.com/llvm/llvm-project/commit/e9ff53d42feac7fc157718523275619a8106f2f3
DIFF: https://github.com/llvm/llvm-project/commit/e9ff53d42feac7fc157718523275619a8106f2f3.diff
LOG: [MemProf] Fix buildbot error due to unused variable from bad merge
Fix an unused variable warning introduced by a212d8da94d08e229aa8d65283e4b116310bba10
due to a bad merge with a recent change. E.g. in
https://lab.llvm.org/buildbot/#/builders/77/builds/22095
Added:
Modified:
llvm/lib/Transforms/Instrumentation/PGOInstrumentation.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Transforms/Instrumentation/PGOInstrumentation.cpp b/llvm/lib/Transforms/Instrumentation/PGOInstrumentation.cpp
index 3a08186b21dc..22728144fa03 100644
--- a/llvm/lib/Transforms/Instrumentation/PGOInstrumentation.cpp
+++ b/llvm/lib/Transforms/Instrumentation/PGOInstrumentation.cpp
@@ -2093,12 +2093,6 @@ static bool annotateAllFunctions(
// function is actually hot / warm. We will reset the function hot / cold
// attribute and drop all the profile counters.
InstrProfRecord::CountPseudoKind PseudoKind = InstrProfRecord::NotPseudo;
-
- // When AllMinusOnes is true, it means the profile for the function
- // is unrepresentative and this function is actually hot. Set the
- // entry count of the function to be multiple times of hot threshold
- // and drop all its internal counters.
- bool AllMinusOnes = false;
bool AllZeros = false;
if (!Func.readCounters(PGOReader.get(), AllZeros, PseudoKind))
continue;
More information about the llvm-commits
mailing list