RESOLVED FIXED 274154
[Win] Switch from MASM to clang-cl for assembling offlineasm
https://bugs.webkit.org/show_bug.cgi?id=274154
Summary [Win] Switch from MASM to clang-cl for assembling offlineasm
Ian Grunert
Reported 2024-05-14 07:29:31 PDT
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&T syntax. I also wonder if it'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)
Attachments
Ian Grunert
Comment 1 2024-05-14 07:49:24 PDT
EWS
Comment 2 2024-05-14 09:57:31 PDT
Committed 278767@main (45da61538271): <https://commits.webkit.org/278767@main> Reviewed commits have been landed. Closing PR #28538 and removing active labels.
Radar WebKit Bug Importer
Comment 3 2024-05-14 09:58:17 PDT
Note You need to log in before you can comment on or make changes to this bug.