Skip to content
This repository has been archived by the owner on Oct 15, 2020. It is now read-only.

Commit

Permalink
[Merge chakra-core/ChakraCore@f092881abe] [MERGE #3506 @Cellule] Defa…
Browse files Browse the repository at this point in the history
…ult case in InliningDecider::GetBuiltInInfoCommon

Merge pull request #3506 from Cellule:cleanup

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
  • Loading branch information
chakrabot authored and kfarnung committed Aug 10, 2017
1 parent 4d1aff1 commit 78d5a93
Showing 1 changed file with 1 addition and 19 deletions.
20 changes: 1 addition & 19 deletions deps/chakrashim/core/lib/Backend/InliningDecider.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Expand Down

0 comments on commit 78d5a93

Please sign in to comment.