From 64c391dbe48062a857a75148ad399e3a8835bfb9 Mon Sep 17 00:00:00 2001 From: Alexey Samsonov Date: Fri, 19 Jul 2013 08:55:18 +0000 Subject: [PATCH] Fix uninitialized memory read found by MemorySanitizer: always set output parameter of ConvergingScheduler::SchedBoundary::getOtherResourceCount llvm-svn: 186658 --- llvm/lib/CodeGen/MachineScheduler.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/llvm/lib/CodeGen/MachineScheduler.cpp b/llvm/lib/CodeGen/MachineScheduler.cpp index 314f621..2ee8775 100644 --- a/llvm/lib/CodeGen/MachineScheduler.cpp +++ b/llvm/lib/CodeGen/MachineScheduler.cpp @@ -1593,6 +1593,7 @@ findMaxLatency(ArrayRef ReadySUs) { // resource index, or zero if the zone is issue limited. unsigned ConvergingScheduler::SchedBoundary:: getOtherResourceCount(unsigned &OtherCritIdx) { + OtherCritIdx = 0; if (!SchedModel->hasInstrSchedModel()) return 0; @@ -1600,7 +1601,6 @@ getOtherResourceCount(unsigned &OtherCritIdx) { + (RetiredMOps * SchedModel->getMicroOpFactor()); DEBUG(dbgs() << " " << Available.getName() << " + Remain MOps: " << OtherCritCount / SchedModel->getMicroOpFactor() << '\n'); - OtherCritIdx = 0; for (unsigned PIdx = 1, PEnd = SchedModel->getNumProcResourceKinds(); PIdx != PEnd; ++PIdx) { unsigned OtherCount = getResourceCount(PIdx) + Rem->RemainingCounts[PIdx]; -- 2.7.4