<?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>274154</bug_id>
          
          <creation_ts>2024-05-14 07:29:31 -0700</creation_ts>
          <short_desc>[Win] Switch from MASM to clang-cl for assembling offlineasm</short_desc>
          <delta_ts>2024-05-14 13:34:36 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>JavaScriptCore</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows 10</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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ian Grunert">ian.grunert</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>fujii</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2035243</commentid>
    <comment_count>0</comment_count>
    <who name="Ian Grunert">ian.grunert</who>
    <bug_when>2024-05-14 07:29:31 -0700</bug_when>
    <thetext>This brings Windows in line with every other platform in how they build offlineasm using the LLIntAssembly.h output.

After this change we can rip out the intel syntax generation from offlineasm x86.rb, as clang-cl inline asm supports AT&amp;T syntax.

I also wonder if it&apos;ll open up possibility for ENABLE_UNIFIED_AND_FREEZABLE_CONFIG_RECORD, as the C++ compiler might be able to resolve the DLL bindings for offlineasm automatically (see this comment https://github.com/WebKit/WebKit/blob/b20a04aa7b405aaf02cebbda8fa73e51e32b1db8/Source/WTF/wtf/PlatformEnable.h#L856-L867)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2035249</commentid>
    <comment_count>1</comment_count>
    <who name="Ian Grunert">ian.grunert</who>
    <bug_when>2024-05-14 07:49:24 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/28538</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2035287</commentid>
    <comment_count>2</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2024-05-14 09:57:31 -0700</bug_when>
    <thetext>Committed 278767@main (45da61538271): &lt;https://commits.webkit.org/278767@main&gt;

Reviewed commits have been landed. Closing PR #28538 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2035288</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2024-05-14 09:58:17 -0700</bug_when>
    <thetext>&lt;rdar://problem/128071085&gt;</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>