COMMIT_MESSAGE

 1[JSC][32bit] Remove a bunch of compiler warnings
 2https://bugs.webkit.org/show_bug.cgi?id=231091
 3
 4Reviewed by NOBODY (OOPS!).
 5
 6* bytecode/CodeBlock.cpp:
 7(JSC::CodeBlock::isConstantOwnedByUnlinkedCodeBlock const): make
 8the function return a default value. Also assert in the default
 9case for the switch, since it covers all the possibilites (as of
 10now).
 11* dfg/DFGOSRExit.cpp:
 12(JSC::DFG::OSRExit::compileExit): silence warning about unused
 13variable, it's not used in 32-bit.
 14* jit/JITOpcodes32_64.cpp:
 15(JSC::JIT::emit_op_enter): use an unsigned int for the loop count.

Source/JavaScriptCore/ChangeLog

 12021-10-01 Xan López <xan@igalia.com>
 2
 3 [JSC][32bit] Remove a bunch of compiler warnings
 4 https://bugs.webkit.org/show_bug.cgi?id=231091
 5
 6 Reviewed by NOBODY (OOPS!).
 7
 8 * bytecode/CodeBlock.cpp:
 9 (JSC::CodeBlock::isConstantOwnedByUnlinkedCodeBlock const): make
 10 the function return a default value. Also assert in the default
 11 case for the switch, since it covers all the possibilites (as of
 12 now).
 13 * dfg/DFGOSRExit.cpp:
 14 (JSC::DFG::OSRExit::compileExit): silence warning about unused
 15 variable, it's not used in 32-bit.
 16 * jit/JITOpcodes32_64.cpp:
 17 (JSC::JIT::emit_op_enter): use an unsigned int for the loop count.
 18
1192021-10-01 Olivier Blin <olivier.blin@softathome.com>
220
321 [JSC][GLib] Fix typo in WrapperMap creation methods

Source/JavaScriptCore/bytecode/CodeBlock.cpp

@@bool CodeBlock::isConstantOwnedByUnlinkedCodeBlock(VirtualRegister reg) const
949949 }
950950 case SourceCodeRepresentation::LinkTimeConstant:
951951 return false;
 952 default:
 953 ASSERT_NOT_REACHED();
952954 }
 955
 956 return false;
953957}
954958
955959Vector<unsigned> CodeBlock::setConstantRegisters(const FixedVector<WriteBarrier<Unknown>>& constants, const FixedVector<SourceCodeRepresentation>& constantsSourceCodeRepresentation)

Source/JavaScriptCore/dfg/DFGOSRExit.cpp

@@void OSRExit::compileExit(CCallHelpers& jit, VM& vm, const OSRExit& exit, const
606606 jit.store64(GPRInfo::regT0, scratch + index);
607607 }
608608#else // not USE(JSVALUE64)
 609 UNUSED_VARIABLE(firstTmpToRestoreEarly);
609610 jit.store32(
610611 AssemblyHelpers::TrustedImm32(recovery.constant().tag()),
611612 &bitwise_cast<EncodedValueDescriptor*>(scratch + index)->asBits.tag);

Source/JavaScriptCore/jit/JITOpcodes32_64.cpp

@@void JIT::emit_op_enter(const Instruction* currentInstruction)
10931093 // Even though JIT code doesn't use them, we initialize our constant
10941094 // registers to zap stale pointers, to avoid unnecessarily prolonging
10951095 // object lifetime and increasing GC pressure.
1096  for (int i = CodeBlock::llintBaselineCalleeSaveSpaceAsVirtualRegisters(); i < m_profiledCodeBlock->numVars(); ++i)
 1096 for (unsigned i = CodeBlock::llintBaselineCalleeSaveSpaceAsVirtualRegisters(); i < m_profiledCodeBlock->numVars(); ++i)
10971097 emitStore(virtualRegisterForLocal(i), jsUndefined());
10981098
10991099 JITSlowPathCall slowPathCall(this, currentInstruction, slow_path_enter);