Bug 1330306 - silence sa-checker regarding false-positive out-of-bounds access/read. r?jorendorff
MozReview-Commit-ID: 6jtsZUgGheC
--- a/js/src/gc/Statistics.cpp
+++ b/js/src/gc/Statistics.cpp
@@ -958,28 +958,17 @@ LongestPhaseSelfTime(const Statistics::P
// Start with total times, including children's times.
for (size_t i = 0; i < PHASE_LIMIT; ++i)
selfTimes[i] = SumPhase(Phase(i), times);
// Subtract out the children's times.
for (size_t i = 0; i < PHASE_LIMIT; ++i) {
Phase parent = phases[i].parent;
- if (parent == PHASE_MULTI_PARENTS) {
- // Subtract out only the time for the children specific to this
- // parent.
- for (auto edge : dagChildEdges) {
- if (edge.parent == parent) {
- size_t dagSlot = phaseExtra[edge.parent].dagSlot;
- CheckSelfTime(parent, edge.child, times, selfTimes, times[dagSlot][edge.child]);
- MOZ_ASSERT(selfTimes[parent] >= times[dagSlot][edge.child]);
- selfTimes[parent] -= times[dagSlot][edge.child];
- }
- }
- } else if (parent != PHASE_NO_PARENT) {
+ if (parent != PHASE_NO_PARENT) {
MOZ_ASSERT(selfTimes[parent] >= selfTimes[i]);
CheckSelfTime(parent, Phase(i), times, selfTimes, selfTimes[i]);
selfTimes[parent] -= selfTimes[i];
}
}
TimeDuration longestTime = 0;
Phase longestPhase = PHASE_NONE;