Changeset 160062 in webkit for trunk/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp

Timestamp:

Dec 3, 2013, 5:18:19 PM (11 years ago)

Author:

[email protected]

Message:

Fix LLINT_C_LOOP build for Win64.
​https://bugs.webkit.org/show_bug.cgi?id=125186.

Reviewed by Michael Saboff.

• JavaScriptCore.vcxproj/JavaScriptCore.vcxproj:
• JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters:
• jit/JITOperationsMSVC64.cpp: Added.

(JSC::getHostCallReturnValueWithExecState):

• Win64 will build JITStubMSVC64.asm even when !ENABLE(JIT). This results in a linkage error due to a missing getHostCallReturnValueWithExecState(). So, we add a stub getHostCallReturnValueWithExecState() here to satisfy that linkage. This function will never be called. The alternative to providing such a stub is to make the MSVC project recognize if the JIT is enabled or not, and exclude JITStubMSVC64.asm if it's not enabled. We don't currently set ENABLE(JIT) via the MSVC project and the work to do that is too much trouble for what we're trying to achieve here. So, we're opting for this simpler workaround instead.

• llint/LowLevelInterpreter.asm:
• llint/LowLevelInterpreter.cpp:

(JSC::CLoop::execute):

• Don't build callToJavaScript if we're building the C loop. Otherwise, the C loop won't build if !ENABLE(COMPUTE_GOTO_OPCODES).

File:

• trunk/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp

@@ -310 +310 @@
     // 3. 64 bit result values will be in t0.
 
-    CLoopRegister t0, t1, t2, t3, a1;
+    CLoopRegister t0, t1, t2, t3;
 #if USE(JSVALUE64)
     CLoopRegister rBasePC, tagTypeNumber, tagMask;