Title: [283608] trunk/Source/_javascript_Core
Revision
283608
Author
[email protected]
Date
2021-10-06 02:35:17 -0700 (Wed, 06 Oct 2021)

Log Message

[JSC][32bit] Remove a bunch of compiler warnings
https://bugs.webkit.org/show_bug.cgi?id=231091

Patch by Xan López <[email protected]> on 2021-10-06
Reviewed by Carlos Alberto Lopez Perez.

* dfg/DFGOSRExit.cpp:
(JSC::DFG::OSRExit::compileExit): silence warning about unused
variable, it's not used in 32-bit.
* jit/JITOpcodes32_64.cpp:
(JSC::JIT::emit_op_enter): use an unsigned int for the loop count.

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (283607 => 283608)


--- trunk/Source/_javascript_Core/ChangeLog	2021-10-06 09:25:15 UTC (rev 283607)
+++ trunk/Source/_javascript_Core/ChangeLog	2021-10-06 09:35:17 UTC (rev 283608)
@@ -1,3 +1,16 @@
+2021-10-06  Xan López  <[email protected]>
+
+        [JSC][32bit] Remove a bunch of compiler warnings
+        https://bugs.webkit.org/show_bug.cgi?id=231091
+
+        Reviewed by Carlos Alberto Lopez Perez.
+
+        * dfg/DFGOSRExit.cpp:
+        (JSC::DFG::OSRExit::compileExit): silence warning about unused
+        variable, it's not used in 32-bit.
+        * jit/JITOpcodes32_64.cpp:
+        (JSC::JIT::emit_op_enter): use an unsigned int for the loop count.
+
 2021-10-06  Carlos Garcia Campos  <[email protected]>
 
         [WPE][GTK] TestJSC incorrectly expects garbage collector to collect variables still on the stack

Modified: trunk/Source/_javascript_Core/dfg/DFGOSRExit.cpp (283607 => 283608)


--- trunk/Source/_javascript_Core/dfg/DFGOSRExit.cpp	2021-10-06 09:25:15 UTC (rev 283607)
+++ trunk/Source/_javascript_Core/dfg/DFGOSRExit.cpp	2021-10-06 09:35:17 UTC (rev 283608)
@@ -606,6 +606,7 @@
                 jit.store64(GPRInfo::regT0, scratch + index);
             }
 #else // not USE(JSVALUE64)
+            UNUSED_VARIABLE(firstTmpToRestoreEarly);
             jit.store32(
                 AssemblyHelpers::TrustedImm32(recovery.constant().tag()),
                 &bitwise_cast<EncodedValueDescriptor*>(scratch + index)->asBits.tag);

Modified: trunk/Source/_javascript_Core/jit/JITOpcodes32_64.cpp (283607 => 283608)


--- trunk/Source/_javascript_Core/jit/JITOpcodes32_64.cpp	2021-10-06 09:25:15 UTC (rev 283607)
+++ trunk/Source/_javascript_Core/jit/JITOpcodes32_64.cpp	2021-10-06 09:35:17 UTC (rev 283608)
@@ -1083,7 +1083,7 @@
     // Even though JIT code doesn't use them, we initialize our constant
     // registers to zap stale pointers, to avoid unnecessarily prolonging
     // object lifetime and increasing GC pressure.
-    for (int i = CodeBlock::llintBaselineCalleeSaveSpaceAsVirtualRegisters(); i < m_profiledCodeBlock->numVars(); ++i)
+    for (unsigned i = CodeBlock::llintBaselineCalleeSaveSpaceAsVirtualRegisters(); i < m_profiledCodeBlock->numVars(); ++i)
         emitStore(virtualRegisterForLocal(i), jsUndefined());
 
     JITSlowPathCall slowPathCall(this, currentInstruction, slow_path_enter);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to