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

Wed, 09 Nov 2011 05:32:14 -0800

Title: [99693] trunk/Source/_javascript_Core
Revision
99693
Author
[email protected]
Date
2011-11-09 05:31:59 -0800 (Wed, 09 Nov 2011)

Log Message

Enable the DFG JIT on x86-64 Linux platforms
https://bugs.webkit.org/show_bug.cgi?id=71373

Enable DFG JIT by default on X86 Linux and Mac platforms
https://bugs.webkit.org/show_bug.cgi?id=71686

Buildfix for stricter compilers: -Werror=unused-but-set-variable

Reviewed by Zoltan Herczeg.

* dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compilePutByValForByteArray):
* dfg/DFGSpeculativeJIT32_64.cpp:
(JSC::DFG::SpeculativeJIT::compile):
* dfg/DFGSpeculativeJIT64.cpp:
(JSC::DFG::SpeculativeJIT::compile):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (99692 => 99693)


--- trunk/Source/_javascript_Core/ChangeLog	2011-11-09 13:19:43 UTC (rev 99692)
+++ trunk/Source/_javascript_Core/ChangeLog	2011-11-09 13:31:59 UTC (rev 99693)
@@ -1,3 +1,22 @@
+2011-11-09  Csaba Osztrogonác  <[email protected]>
+
+        Enable the DFG JIT on x86-64 Linux platforms
+        https://bugs.webkit.org/show_bug.cgi?id=71373
+
+        Enable DFG JIT by default on X86 Linux and Mac platforms
+        https://bugs.webkit.org/show_bug.cgi?id=71686
+
+        Buildfix for stricter compilers: -Werror=unused-but-set-variable
+
+        Reviewed by Zoltan Herczeg.
+
+        * dfg/DFGSpeculativeJIT.cpp:
+        (JSC::DFG::SpeculativeJIT::compilePutByValForByteArray):
+        * dfg/DFGSpeculativeJIT32_64.cpp:
+        (JSC::DFG::SpeculativeJIT::compile):
+        * dfg/DFGSpeculativeJIT64.cpp:
+        (JSC::DFG::SpeculativeJIT::compile):
+
 2011-11-09  Sheriff Bot  <[email protected]>
 
         Unreviewed, rolling out r99678.

Modified: trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp (99692 => 99693)


--- trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp	2011-11-09 13:19:43 UTC (rev 99692)
+++ trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp	2011-11-09 13:31:59 UTC (rev 99693)
@@ -731,7 +731,7 @@
         GPRReg scratchReg = scratch.gpr();
         m_jit.move(valueOp.gpr(), scratchReg);
         MacroAssembler::Jump inBounds = m_jit.branch32(MacroAssembler::BelowOrEqual, scratchReg, TrustedImm32(0xff));
-        MacroAssembler::Jump tooBig = m_jit.branch32(MacroAssembler::GreaterThan, scratchReg, TrustedImm32(0xff));
+        m_jit.branch32(MacroAssembler::GreaterThan, scratchReg, TrustedImm32(0xff));
         m_jit.xorPtr(scratchReg, scratchReg);
         MacroAssembler::Jump clamped = m_jit.jump();
         m_jit.move(TrustedImm32(255), scratchReg);
@@ -749,7 +749,7 @@
         value.adopt(result);
         valueGPR = gpr;
     }
-    ASSERT(valueGPR != property);
+    ASSERT_UNUSED(valueGPR, valueGPR != property);
     ASSERT(valueGPR != base);
     GPRTemporary storage(this);
     GPRReg storageReg = storage.gpr();

Modified: trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT32_64.cpp (99692 => 99693)


--- trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT32_64.cpp	2011-11-09 13:19:43 UTC (rev 99692)
+++ trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT32_64.cpp	2011-11-09 13:31:59 UTC (rev 99693)
@@ -1505,7 +1505,7 @@
 
         // Code to handle put beyond array bounds.
         silentSpillAllRegisters(scratchReg);
-        JITCompiler::Call functionCall = callOperation(operationPutByValBeyondArrayBounds, baseReg, propertyReg, valueTagReg, valuePayloadReg);
+        callOperation(operationPutByValBeyondArrayBounds, baseReg, propertyReg, valueTagReg, valuePayloadReg);
         silentFillAllRegisters(scratchReg);
         JITCompiler::Jump wasBeyondArrayBounds = m_jit.jump();
 
@@ -2493,7 +2493,7 @@
 
         structuresNotMatch.link(&m_jit);
         silentSpillAllRegisters(resultTagGPR, resultPayloadGPR);
-        JITCompiler::Call functionCall = callOperation(operationResolveGlobal, resultTagGPR, resultPayloadGPR, resolveInfoGPR, &m_jit.codeBlock()->identifier(data.identifierNumber));
+        callOperation(operationResolveGlobal, resultTagGPR, resultPayloadGPR, resolveInfoGPR, &m_jit.codeBlock()->identifier(data.identifierNumber));
         silentFillAllRegisters(resultTagGPR, resultPayloadGPR);
 
         wasFast.link(&m_jit);

Modified: trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT64.cpp (99692 => 99693)


--- trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT64.cpp	2011-11-09 13:19:43 UTC (rev 99692)
+++ trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT64.cpp	2011-11-09 13:31:59 UTC (rev 99693)
@@ -1549,7 +1549,7 @@
 
         // Code to handle put beyond array bounds.
         silentSpillAllRegisters(scratchReg);
-        JITCompiler::Call functionCall = callOperation(operationPutByValBeyondArrayBounds, baseReg, propertyReg, valueReg);
+        callOperation(operationPutByValBeyondArrayBounds, baseReg, propertyReg, valueReg);
         silentFillAllRegisters(scratchReg);
         JITCompiler::Jump wasBeyondArrayBounds = m_jit.jump();
 
@@ -2468,7 +2468,7 @@
         JITCompiler::Jump structuresMatch = m_jit.branchPtr(JITCompiler::Equal, resultGPR, JITCompiler::Address(globalObjectGPR, JSCell::structureOffset()));
 
         silentSpillAllRegisters(resultGPR);
-        JITCompiler::Call functionCall = callOperation(operationResolveGlobal, resultGPR, resolveInfoGPR, &m_jit.codeBlock()->identifier(data.identifierNumber));
+        callOperation(operationResolveGlobal, resultGPR, resolveInfoGPR, &m_jit.codeBlock()->identifier(data.identifierNumber));
         silentFillAllRegisters(resultGPR);
 
         JITCompiler::Jump wasSlow = m_jit.jump();

_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to