Bug 832070 - LTruncateDToInt32 doesn't need a snapshot (r=sstangl)
authorLuke Wagner <luke@mozilla.com>
Fri, 07 Dec 2012 18:54:05 -0800
changeset 119327 3f5f3b2602e533b19cde84410a1b0713ff477865
parent 119326 dcf3f2446544fdb714cf96fbc9176908de9f839f
child 119328 bfad3bfb241451c44627ffdb2b9de0610b7743fb
push id21615
push userlwagner@mozilla.com
push dateSat, 19 Jan 2013 01:31:32 +0000
treeherdermozilla-inbound@2674257f6117 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssstangl
bugs832070
milestone21.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 832070 - LTruncateDToInt32 doesn't need a snapshot (r=sstangl)
js/src/ion/Lowering.cpp
--- a/js/src/ion/Lowering.cpp
+++ b/js/src/ion/Lowering.cpp
@@ -1287,20 +1287,17 @@ LIRGenerator::visitTruncateToInt32(MTrun
       case MIRType_Undefined:
         return define(new LInteger(0), truncate);
 
       case MIRType_Int32:
       case MIRType_Boolean:
         return redefine(truncate, opd);
 
       case MIRType_Double:
-      {
-        LTruncateDToInt32 *lir = new LTruncateDToInt32(useRegister(opd), tempFloat());
-        return assignSnapshot(lir) && define(lir, truncate);
-      }
+        return define(new LTruncateDToInt32(useRegister(opd), tempFloat()), truncate);
 
       default:
         // Objects might be effectful.
         // Strings are complicated - we don't handle them yet.
         JS_ASSERT(!"unexpected type");
         return false;
     }
 }