Parcourir la source

When turned off `ENABLE_SIMDJS` it changed the behavior of the switch case in `InliningDecider::GetBuiltInInfoCommon`
Turns out we only use the return value for -off:<some inlining feature>, so this is just a clean up change

Michael Ferris il y a 8 ans
Parent
commit
51be3e2a29
1 fichiers modifiés avec 1 ajouts et 19 suppressions
  1. 1 19
      lib/Backend/InliningDecider.cpp

+ 1 - 19
lib/Backend/InliningDecider.cpp

@@ -624,26 +624,8 @@ bool InliningDecider::GetBuiltInInfoCommon(
         break;
 #endif
 
-#ifdef ENABLE_SIMDJS
-    // SIMD_JS
-    // we only inline, and hence type-spec on IA
-#if defined(_M_X64) || defined(_M_IX86)
     default:
-    {
-#if 0 // TODO OOP JIT, inline SIMD
-        // inline only if simdjs and simd128 type-spec is enabled.
-        if (scriptContext->GetConfig()->IsSimdjsEnabled() && SIMD128_TYPE_SPEC_FLAG)
-        {
-            *inlineCandidateOpCode = scriptContext->GetThreadContext()->GetSimdOpcodeFromFuncInfo(funcInfo);
-        }
-        else
-#endif
-        {
-            return false;
-        }
-    }
-#endif
-#endif // ENABLE_SIMDJS
+        return false;
     }
     return true;
 }