WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Add attachment
proposed patch, testcase, etc.
Ian Grunert
Comment 1
2024-05-14 07:49:24 PDT
Pull request:
https://github.com/WebKit/WebKit/pull/28538
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
<
rdar://problem/128071085
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug