Bug 607856 - long hang loading page. r=dvander.
authorNicholas Nethercote <nnethercote@mozilla.com>
Tue, 02 Nov 2010 16:16:40 -0700
changeset 57683 1ce507b5289d335c5c7dde6ad57bc0034cf7d737
parent 57682 322fe3e6482e13e682b28f9d84ddcc6b25e27148
child 57684 f70a7b95d2ea0ddf5e2a805ba0b631652f18844b
push id1
push usershaver@mozilla.com
push dateTue, 04 Jan 2011 17:58:04 +0000
reviewersdvander
bugs607856
milestone2.0b8pre
Bug 607856 - long hang loading page. r=dvander.
js/src/nanojit/LIR.cpp
--- a/js/src/nanojit/LIR.cpp
+++ b/js/src/nanojit/LIR.cpp
@@ -2569,17 +2569,19 @@ namespace nanojit
         LIns* ins;
         NanoAssert(isCseOpcode(op));
         uint32_t k;
         ins = find2(op, a, b, k);
         if (!ins) {
             ins = out->ins2(op, a, b);
             addNL(LIns2, ins, k);
         } else if (ins->isCmp()) {
-            if (knownCmpValues.containsKey(ins)) {
+            // XXX: temporarily disabled because it exposed latent problems
+            // that caused bug 607856.  See also bug 609129.
+            if (0 && knownCmpValues.containsKey(ins)) {
                 // We've seen this comparison before, and it was previously
                 // used in a guard, so we know what its value must be at this
                 // point.  Replace it with a constant.
                 NanoAssert(ins->isCmp());
                 bool cmpValue = knownCmpValues.get(ins);
                 return insImmI(cmpValue ? 1 : 0);
             }
         }