[webkit-changes] [WebKit/WebKit] 326030: Re-enable B3 hoistLoopInvariantValues

[David Degazio]

  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 326030ada54fec81647abc20977d3f2b38717a91
      https://github.com/WebKit/WebKit/commit/326030ada54fec81647abc20977d3f2b38717a91
  Author: David Degazio <d_degazio at apple.com>
  Date:   2024-07-23 (Tue, 23 Jul 2024)

  Changed paths:
    M Source/JavaScriptCore/b3/B3Generate.cpp
    M Source/JavaScriptCore/runtime/OptionsList.h

  Log Message:
  -----------
  Re-enable B3 hoistLoopInvariantValues
https://bugs.webkit.org/show_bug.cgi?id=212651
rdar://132334359

Reviewed by Yijia Huang and Yusuke Suzuki.

Re-enables the hoistLoopInvariantValues phase in B3. This was
previously disabled due to (as I understand it) performance
concerns, but recent benchmarking shows it as neutral, if not
a slight progression on Speedometer 3. LICM in B3 specifically
is also more relevant now that WASM is more common - JS isn't
likely to benefit quite as much since we already do LICM in
DFG. But WASM, targeting B3 directly, is currently missing out
on this important optimization.

* Source/JavaScriptCore/runtime/OptionsList.h:

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



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