[webkit-changes] [103139] trunk/Source/JavaScriptCore

[commit-queue]

Title: [103139] trunk/Source/_javascript_Core

Revision

103139

Author

commit-qu...@webkit.org

Date

2011-12-16 19:46:17 -0800 (Fri, 16 Dec 2011)

Log Message

Remove unused variable after r74747 (buildfix)
https://bugs.webkit.org/show_bug.cgi?id=74767

Patch by Rafael Brandao <rafael.l...@openbossa.org> on 2011-12-16
Reviewed by Darin Adler.

* dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compilePutByValForByteArray):
(JSC::DFG::SpeculativeJIT::compilePutByValForIntTypedArray):

Modified Paths

• trunk/Source/_javascript_Core/ChangeLog
• trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (103138 => 103139)

--- trunk/Source/_javascript_Core/ChangeLog	2011-12-17 02:40:35 UTC (rev 103138)
+++ trunk/Source/_javascript_Core/ChangeLog	2011-12-17 03:46:17 UTC (rev 103139)
@@ -1,3 +1,14 @@
+2011-12-16  Rafael Brandao  <rafael.l...@openbossa.org>
+
+        Remove unused variable after r74747 (buildfix)
+        https://bugs.webkit.org/show_bug.cgi?id=74767
+
+        Reviewed by Darin Adler.
+
+        * dfg/DFGSpeculativeJIT.cpp:
+        (JSC::DFG::SpeculativeJIT::compilePutByValForByteArray):
+        (JSC::DFG::SpeculativeJIT::compilePutByValForIntTypedArray):
+
 2011-12-16  Gavin Barraclough  <barraclo...@apple.com>
 
         Reverted r103120, this breaks v8 on ARMv7 DFG.

Modified: trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp (103138 => 103139)

--- trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp	2011-12-17 02:40:35 UTC (rev 103138)
+++ trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp	2011-12-17 03:46:17 UTC (rev 103139)
@@ -1553,7 +1553,6 @@
     if (!isByteArrayPrediction(m_state.forNode(baseIndex).m_type))
         speculationCheck(BadType, JSValueSource::unboxedCell(base), baseIndex, m_jit.branchPtr(MacroAssembler::NotEqual, MacroAssembler::Address(base, JSCell::classInfoOffset()), MacroAssembler::TrustedImmPtr(&JSByteArray::s_info)));
     GPRTemporary value;
-    GPRReg valueGPR;
 
     if (at(valueIndex).isConstant()) {
         JSValue jsValue = valueOfJSConstant(valueIndex);
@@ -1572,7 +1571,6 @@
         GPRReg scratchReg = scratch.gpr();
         m_jit.move(Imm32((int)d), scratchReg);
         value.adopt(scratch);
-        valueGPR = scratchReg;
     } else if (!at(valueIndex).shouldNotSpeculateInteger()) {
         SpeculateIntegerOperand valueOp(this, valueIndex);
         GPRTemporary scratch(this);
@@ -1587,7 +1585,6 @@
         clamped.link(&m_jit);
         inBounds.link(&m_jit);
         value.adopt(scratch);
-        valueGPR = scratchReg;
     } else {
         SpeculateDoubleOperand valueOp(this, valueIndex);
         GPRTemporary result(this);
@@ -1596,7 +1593,6 @@
         GPRReg gpr = result.gpr();
         compileClampDoubleToByte(m_jit, gpr, fpr, floatScratch.fpr());
         value.adopt(result);
-        valueGPR = gpr;
     }
     ASSERT_UNUSED(valueGPR, valueGPR != property);
     ASSERT(valueGPR != base);
@@ -1710,7 +1706,6 @@
     if (speculationRequirements != NoTypedArrayTypeSpecCheck)
         speculationCheck(BadType, JSValueSource::unboxedCell(base), baseIndex, m_jit.branchPtr(MacroAssembler::NotEqual, MacroAssembler::Address(base, JSCell::classInfoOffset()), MacroAssembler::TrustedImmPtr(descriptor.m_classInfo)));
     GPRTemporary value;
-    GPRReg valueGPR;
     
     if (at(valueIndex).isConstant()) {
         JSValue jsValue = valueOfJSConstant(valueIndex);
@@ -1724,14 +1719,12 @@
         GPRReg scratchReg = scratch.gpr();
         m_jit.move(Imm32((int)d), scratchReg);
         value.adopt(scratch);
-        valueGPR = scratchReg;
     } else if (!at(valueIndex).shouldNotSpeculateInteger()) {
         SpeculateIntegerOperand valueOp(this, valueIndex);
         GPRTemporary scratch(this);
         GPRReg scratchReg = scratch.gpr();
         m_jit.move(valueOp.gpr(), scratchReg);
         value.adopt(scratch);
-        valueGPR = scratchReg;
     } else {
         SpeculateDoubleOperand valueOp(this, valueIndex);
         GPRTemporary result(this);
@@ -1748,7 +1741,6 @@
             m_jit.truncateDoubleToUint32(fpr, gpr);
         fixed.link(&m_jit);
         value.adopt(result);
-        valueGPR = gpr;
     }
     ASSERT_UNUSED(valueGPR, valueGPR != property);
     ASSERT(valueGPR != base);

_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes