Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 6c59818feb768577fbbf3545ab321e1a36c44e26
      
https://github.com/WebKit/WebKit/commit/6c59818feb768577fbbf3545ab321e1a36c44e26
  Author: Yusuke Suzuki <[email protected]>
  Date:   2026-01-28 (Wed, 28 Jan 2026)

  Changed paths:
    M Source/JavaScriptCore/wasm/WasmOMGIRGenerator.cpp

  Log Message:
  -----------
  [JSC] Consolidate vm access in OMG into one value
https://bugs.webkit.org/show_bug.cgi?id=306413
rdar://169076114

Reviewed by Sosuke Suzuki.

VM* is used for mutatorFence and write-barrier in OMG.
But since it looks like just a load from instance, we end up loading it
each time. Let's define it m_vmValue just after obtaining instanceValue.
Since this is immutable load, if nobody is using it, it will be removed
in DCE phase.

* Source/JavaScriptCore/wasm/WasmOMGIRGenerator.cpp:
(JSC::Wasm::OMGIRGenerator::vmValue):
(JSC::Wasm::OMGIRGenerator::OMGIRGenerator):
(JSC::Wasm::OMGIRGenerator::setGlobal):
(JSC::Wasm::OMGIRGenerator::emitWriteBarrierForJSWrapper):
(JSC::Wasm::OMGIRGenerator::emitWriteBarrier):
(JSC::Wasm::OMGIRGenerator::emitArraySetUnchecked):
(JSC::Wasm::OMGIRGenerator::addStructSet):
(JSC::Wasm::OMGIRGenerator::mutatorFence):

Canonical link: https://commits.webkit.org/306344@main



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications

Reply via email to