WebPlatformForEmbedded / WPEWebKit

WPE WebKit port (downstream)
210 stars 135 forks source link

[wpe-2.42] JavaScriptCore build is broken after a66301835162306f74856880da13412f5fbe0381 #1289

Closed enocknt closed 4 months ago

enocknt commented 4 months ago

After the latest merge commit applied to wpe-2.42 branch, the JavaScriptCore build was broken.

 .../webkit/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp:339:21:
    error: use of undeclared identifier 't6'

        UNUSED_VARIABLE(t6);
                        ^
 .../webkit/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp:340:21:
    error: use of undeclared identifier 't7'

        UNUSED_VARIABLE(t7);
                        ^
    2 errors generated.

The issue can be seen in LowLevelInterpreter.cpp on lines 338 and 339.

https://github.com/WebPlatformForEmbedded/WPEWebKit/blob/a66301835162306f74856880da13412f5fbe0381/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp#L338-L339

This occurs due to the variables t6 and t7 not being defined on line 326.

https://github.com/WebPlatformForEmbedded/WPEWebKit/blob/a66301835162306f74856880da13412f5fbe0381/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp#L326

This issue is resolved at https://github.com/WebKit/WebKit/commit/3d5373575695b293b8559155431d0079a6153aff.

magomez commented 4 months ago

I've just pushed the fix for this to the wpe-2.42 branch. Thanks the detailed report!