Bug 921470 - Remove hasFallbackStub_ check in resetMonitorStubChain. r=djvj, a=lsblakk
authorJan de Mooij <jdemooij@mozilla.com>
Tue, 10 Dec 2013 09:31:53 +0100
changeset 175273 86e26e292344d26897b1377903405a6f17bd4a64
parent 175272 423edfffe821b905a07645955afdd178dcb3bfb0
child 175274 cc2ab73484b25cb21077d33679193d8297cc5b15
push id445
push userffxbld
push dateMon, 10 Mar 2014 22:05:19 +0000
treeherdermozilla-release@dc38b741b04e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdjvj, lsblakk
bugs921470
milestone28.0a2
Bug 921470 - Remove hasFallbackStub_ check in resetMonitorStubChain. r=djvj, a=lsblakk
js/src/jit/BaselineIC.cpp
--- a/js/src/jit/BaselineIC.cpp
+++ b/js/src/jit/BaselineIC.cpp
@@ -491,20 +491,18 @@ ICFallbackStub::unlinkStubsWithKind(JSCo
 
 void
 ICTypeMonitor_Fallback::resetMonitorStubChain(Zone *zone)
 {
     if (zone->needsBarrier()) {
         // We are removing edges from monitored stubs to gcthings (IonCode).
         // Perform one final trace of all monitor stubs for incremental GC,
         // as it must know about those edges.
-        if (hasFallbackStub_) {
-            for (ICStub *s = firstMonitorStub_; !s->isTypeMonitor_Fallback(); s = s->next())
-                s->trace(zone->barrierTracer());
-        }
+        for (ICStub *s = firstMonitorStub_; !s->isTypeMonitor_Fallback(); s = s->next())
+            s->trace(zone->barrierTracer());
     }
 
     firstMonitorStub_ = this;
     numOptimizedMonitorStubs_ = 0;
 
     if (hasFallbackStub_) {
         lastMonitorStubPtrAddr_ = nullptr;