commit 86dc6f9a7953f45d9c4791b0cb6cdceef8ca00ee
author Alexey Samsonov <samsonov@google.com> Fri Jul 19 08:55:18 2013 +0000
committer Alexey Samsonov <samsonov@google.com> Fri Jul 19 08:55:18 2013 +0000
tree 4b908187395ff8d30af83bc2421cd56c5bb10370
parent df5ed3f642ff5dfd19634c80c4106ace247dc1bf

Fix uninitialized memory read found by MemorySanitizer: always set output parameter of ConvergingScheduler::SchedBoundary::getOtherResourceCount

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@186658 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/MachineScheduler.cpp

diff --git a/lib/CodeGen/MachineScheduler.cpp b/lib/CodeGen/MachineScheduler.cpp
index 314f621..2ee8775 100644
--- a/lib/CodeGen/MachineScheduler.cpp
+++ b/lib/CodeGen/MachineScheduler.cpp
@@ -1593,6 +1593,7 @@
 // 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 @@
     + (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];