<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>249149</bug_id>
          
          <creation_ts>2022-12-12 08:29:22 -0800</creation_ts>
          <short_desc>[Wasm-GC] Adjust struct.new opcode</short_desc>
          <delta_ts>2022-12-13 18:24:16 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebAssembly</component>
          <version>Other</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>247394</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Asumu Takikawa">asumu</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1918496</commentid>
    <comment_count>0</comment_count>
    <who name="Asumu Takikawa">asumu</who>
    <bug_when>2022-12-12 08:29:22 -0800</bug_when>
    <thetext>The current implementation for Wasm GC struct operations is using an opcode for `struct.new` that is compatible with current V8 and Firefox (https://docs.google.com/document/d/14xWenN0lUoHa9TiDJd8yYR8ZKjgOhgO7iEmfrAuA1_M/edit#heading=h.9dwoku9340md), but not with the actual spec https://github.com/WebAssembly/gc/blob/main/proposals/gc/MVP.md.

The difference is using 0xfb01 or 0xfb07. The opcode numbers are likely to get renumbered eventually as part of a &quot;cleanup&quot; phase of the proposal.

I think we should potentially just support both opcode variations in the parser for now, which will allow us to run wasm programs produced with V8 in mind while also passing tests originating from the spec.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1918697</commentid>
    <comment_count>1</comment_count>
    <who name="Asumu Takikawa">asumu</who>
    <bug_when>2022-12-12 19:07:31 -0800</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/7532</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1919014</commentid>
    <comment_count>2</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2022-12-13 18:23:23 -0800</bug_when>
    <thetext>Committed 257821@main (67cbb39ef6cc): &lt;https://commits.webkit.org/257821@main&gt;

Reviewed commits have been landed. Closing PR #7532 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1919015</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2022-12-13 18:24:16 -0800</bug_when>
    <thetext>&lt;rdar://problem/103330695&gt;</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>