WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
288181
[JSC] Move some inine functions from AirTmpWidth.h to AirTmpWidthInlines.h
https://bugs.webkit.org/show_bug.cgi?id=288181
Summary
[JSC] Move some inine functions from AirTmpWidth.h to AirTmpWidthInlines.h
Fujii Hironori
Reported
2025-02-20 22:21:15 PST
"perl .\Tools\Scripts\build-webkit --debug --no-unified-builds" reports an error. [8/33] Building CXX object Source\JavaScriptCore\CMakeFiles\JavaScriptCore.dir\b3\air\AirAllocateRegistersByGreedy.cpp.obj FAILED: Source/JavaScriptCore/CMakeFiles/JavaScriptCore.dir/b3/air/AirAllocateRegistersByGreedy.cpp.obj C:\PROGRA~1\LLVM\bin\clang-cl.exe [...] -c -- C:\webkit\wa\Source\JavaScriptCore\b3\air\AirAllocateRegistersByGreedy.cpp In file included from C:\webkit\wa\Source\JavaScriptCore\b3\air\AirAllocateRegistersByGreedy.cpp:41: C:\webkit\wa\Source\JavaScriptCore\b3\air\AirTmpWidth.h(109,20): error: inline function 'JSC::B3::Air::TmpWidth::widths' is not defined [-Werror,-Wundefined-inline] 109 | inline Widths& widths(Tmp); | ^ C:\webkit\wa\Source\JavaScriptCore\b3\air\AirTmpWidth.h(64,28): note: used here 64 | Widths tmpWidths = widths(tmp); | ^ 1 error generated.
Attachments
Add attachment
proposed patch, testcase, etc.
Fujii Hironori
Comment 1
2025-02-20 22:31:38 PST
Pull request:
https://github.com/WebKit/WebKit/pull/41044
EWS
Comment 2
2025-02-21 15:08:06 PST
Committed
290830@main
(9eb2c98a7cee): <
https://commits.webkit.org/290830@main
> Reviewed commits have been landed. Closing PR #41044 and removing active labels.
Radar WebKit Bug Importer
Comment 3
2025-02-21 15:09:17 PST
<
rdar://problem/145325897
>
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