Bug 125186 – [Win] Cannot build 64-bit with ENABLE_LLINT but without ENABLE_JIT

• Component: JavaScriptCore
• Status: RESOLVED FIXED

[Brent Fulgham]

Attempting to create a 64-bit build on Windows with ENABLE_JIT undefined results in the following build error:

7>C:\Projects\WebKit\OpenSource\WebKitBuild\Release\obj64\JavaScriptCore\DerivedSources\LLIntAssembly.h(5): error C2065: 'callToJavaScript' : undeclared identifier
7>C:\Projects\WebKit\OpenSource\WebKitBuild\Release\obj64\JavaScriptCore\DerivedSources\LLIntAssembly.h(5): error C2051: case expression not constant
7>C:\Projects\WebKit\OpenSource\WebKitBuild\Release\obj64\JavaScriptCore\DerivedSources\LLIntAssembly.h(17): error C2065: 'returnFromJavaScript' : undeclared identifier
7>C:\Projects\WebKit\OpenSource\WebKitBuild\Release\obj64\JavaScriptCore\DerivedSources\LLIntAssembly.h(17): error C2051: case expression not constant

The 64-bit build automatically activates ENABLE_LLINT, which may be the problem.

"callToJavaScript" and "returnFromJavaScript" are implemented in assembly code (see JITStubsMSVC64.asm).  However, I don't think the declarations are visible to LowLevelInterpreter.cpp when building without ENABLE_JIT.

If I add declarations for those methods, I still encounter the "case expression not constant" error, since it is attempting to use a method declaration as a label.

[Mark Lam]

Created attachment 218364 [details]
the patch.

[Mark Lam]

Comment on attachment 218364 [details]
the patch.

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

> Source/JavaScriptCore/ChangeLog:12
> +        - Win64 will build JITStubMSVC64.asm even when !WNABLE(JIT). This results

typo: WNABLE ==> ENABLE.  Will fix before landing.

[Michael Saboff]

Comment on attachment 218364 [details]
the patch.

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

> Source/JavaScriptCore/jit/JITOperationsMSVC64.cpp:35
> +// The following is a workaround that is only needed because JITStubsMSVC64.asm is built

Please add a FIXME:

[Mark Lam]

Thanks for the review.  Landed in r160062: <http://trac.webkit.org/r160062>.