WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
280719
Stop using class-level JS_EXPORT_PRIVATE for JavaScriptCore/inspector
https://bugs.webkit.org/show_bug.cgi?id=280719
Summary
Stop using class-level JS_EXPORT_PRIVATE for JavaScriptCore/inspector
Fujii Hironori
Reported
2024-10-01 17:04:45 PDT
Stop using class-level JS_EXPORT_PRIVATE for JavaScriptCore/inspector
https://webkit.slack.com/archives/CU64U6FDW/p1727386191584539?thread_ts=1727381875.279419&cid=CU64U6FDW
> I think method level export is better due to various reasons, > 1. that becomes boundary and LTO does not work if it is used (e.g. WTF export) > 2. Most of JSC's class JS_EXPORT_PRIVATE was coming from B3, which requires very fine-grained control and testing from testb3 etc., and I think that's the reason.
Attachments
Add attachment
proposed patch, testcase, etc.
Fujii Hironori
Comment 1
2024-10-01 17:06:32 PDT
Pull request:
https://github.com/WebKit/WebKit/pull/34539
EWS
Comment 2
2024-10-06 14:34:15 PDT
Committed
284744@main
(28b90713af7c): <
https://commits.webkit.org/284744@main
> Reviewed commits have been landed. Closing PR #34539 and removing active labels.
Radar WebKit Bug Importer
Comment 3
2024-10-06 14:35:14 PDT
<
rdar://problem/137390144
>
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