Bug 225501 - [WinCairo] Remove linker warning on ANGLE
Summary: [WinCairo] Remove linker warning on ANGLE
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Basuke Suzuki
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-05-06 18:32 PDT by Basuke Suzuki
Modified: 2021-05-09 22:14 PDT (History)
9 users (show)

See Also:


Attachments
PATCH (1.54 KB, patch)
2021-05-07 15:15 PDT, Basuke Suzuki
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Basuke Suzuki 2021-05-06 18:32:07 PDT
There're two warnings displayed when creating library lib64\libGLESv2.lib and object lib64\libGLESv2.exp:

LINK : warning LNK4217: symbol 'ANGLEGetDisplayPlatform' defined in 'ANGLE.lib(Platform.obj)' is imported by 'proc_table_egl_autogen.obj' in function '"void __cdecl egl::`dynamic initializer for 'g_procTable''(void)" (??__Eg_procTable@egl@@YAXXZ)' [W:\WebKit\WebKitBuild\Release\Source\ThirdParty\ANGLE\GLESv2.vcxproj]
LINK : warning LNK4217: symbol 'ANGLEResetDisplayPlatform' defined in 'ANGLE.lib(Platform.obj)' is imported by 'proc_table_egl_autogen.obj' in function '"void __cdecl egl::`dynamic initializer for 'g_procTable''(void)" (??__Eg_procTable@egl@@YAXXZ)' [W:\WebKit\WebKitBuild\Release\Source\ThirdParty\ANGLE\GLESv2.vcxproj]
Comment 1 Basuke Suzuki 2021-05-07 14:53:05 PDT
I'm still beginner of Windows build env, but my understanding of this warning is:
- ANGLE is compiled as static library
- That is linked to libGLESv2.dll
- __declspec(dllimport) is not required in this case

Is my understanding correct?
Comment 2 Basuke Suzuki 2021-05-07 15:15:24 PDT
Created attachment 428039 [details]
PATCH
Comment 3 EWS Watchlist 2021-05-07 15:16:21 PDT
Note that there are important steps to take when updating ANGLE. See https://trac.webkit.org/wiki/UpdatingANGLE
Comment 4 Darin Adler 2021-05-08 21:07:30 PDT
Comment on attachment 428039 [details]
PATCH

View in context: https://bugs.webkit.org/attachment.cgi?id=428039&action=review

> Source/ThirdParty/ANGLE/ChangeLog:8
> +        libANGLE is statically linked to libGLESv2.dll so that __declspec(dllimport) is not required here.

is *now* statically linked; we changed that recently
Comment 5 EWS 2021-05-08 21:31:50 PDT
Committed r277241 (237510@main): <https://commits.webkit.org/237510@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 428039 [details].
Comment 6 Radar WebKit Bug Importer 2021-05-08 21:32:14 PDT
<rdar://problem/77704188>
Comment 7 Basuke Suzuki 2021-05-09 22:14:11 PDT
> is *now* statically linked; we changed that recently


Make sense. Thanks!