[webkit-changes] [WebKit/WebKit] 7da972: [WASM] Fix prefixed-opcode code origins for relaxe...

David Degazio noreply at github.com
Wed Jul 31 19:31:23 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 7da9729a2e2a44dd001f07f1acd000dda7a24e5f
      https://github.com/WebKit/WebKit/commit/7da9729a2e2a44dd001f07f1acd000dda7a24e5f
  Author: David Degazio <d_degazio at apple.com>
  Date:   2024-07-31 (Wed, 31 Jul 2024)

  Changed paths:
    M Source/JavaScriptCore/wasm/WasmOpcodeOrigin.h

  Log Message:
  -----------
  [WASM] Fix prefixed-opcode code origins for relaxed SIMD opcodes
https://bugs.webkit.org/show_bug.cgi?id=277446
rdar://problem/132927699

Reviewed by Keith Miller and Yijia Huang.

Removes the assertion that the prefixed opcode in an OpcodeOrigin
fits in 8 bits. This is violated by the relaxed SIMD proposal, which
currently reserves up to 0x12F. Instead, this patch just lets the
opcode use all of the upper 24 bits of the origin.

* Source/JavaScriptCore/wasm/WasmOpcodeOrigin.h:
(JSC::Wasm::OpcodeOrigin::OpcodeOrigin):
(JSC::Wasm::OpcodeOrigin::ext1Opcode const):
(JSC::Wasm::OpcodeOrigin::simdOpcode const):
(JSC::Wasm::OpcodeOrigin::gcOpcode const):
(JSC::Wasm::OpcodeOrigin::atomicOpcode const):

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



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


More information about the webkit-changes mailing list