Changeset 288363 in webkit for trunk/Source/JavaScriptCore/jit/JITCode.cpp

Timestamp:

Jan 21, 2022, 8:11:23 AM (3 years ago)

Author:

[email protected]

Message:

Fix build when using Visual Studio 2022
​https://bugs.webkit.org/show_bug.cgi?id=235440

Reviewed by Antti Koivisto.

It doesn't like having a switch with a default but no cases.
This is cleaner with if statements anyways.
Also make members const because I can.

• jit/JITCode.cpp:

(JSC::JITCode::calleeSaveRegisters const):

• jit/JITCode.h:

File:

• trunk/Source/JavaScriptCore/jit/JITCode.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/jit/JITCode.cpp

@@ -100 +100 @@
 const RegisterAtOffsetList* JITCode::calleeSaveRegisters() const
 {
-    switch (jitType()) {
 #if ENABLE(FTL_JIT)
-    case JITType::FTLJIT:
+    if (m_jitType == JITType::FTLJIT)
         return static_cast<const FTL::JITCode*>(this)->calleeSaveRegisters();
 #endif
 #if ENABLE(DFG_JIT)
-    case JITType::DFGJIT:
+    if (m_jitType == JITType::DFGJIT)
         return &RegisterAtOffsetList::dfgCalleeSaveRegisters();
 #endif
-    default:
 #if !ENABLE(C_LOOP)
-        return &RegisterAtOffsetList::llintBaselineCalleeSaveRegisters();
+    return &RegisterAtOffsetList::llintBaselineCalleeSaveRegisters();
 #else
-        return nullptr;
-#endif
-    }
+    return nullptr;
+#endif
 }