Commit 9061eca2 authored by Tobias Tebbi's avatar Tobias Tebbi Committed by Commit Bot

[turbofan] [wasm] enable switch jump tables by default

With this CL, switch jump tables are enabled even with
--untrusted-code-mitigations

Bug: chromium:849098
Change-Id: I60545ed7bef2cd847710fd9660ef1007aac3b428
Reviewed-on: https://chromium-review.googlesource.com/1110817Reviewed-by: 's avatarJaroslav Sevcik <jarin@chromium.org>
Commit-Queue: Tobias Tebbi <tebbi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#53993}
parent f9868eaa
......@@ -28,7 +28,7 @@ OptimizedCompilationInfo::OptimizedCompilationInfo(
SetFlag(kCalledWithCodeStartRegister);
if (FLAG_function_context_specialization) MarkAsFunctionContextSpecializing();
if (FLAG_turbo_splitting) MarkAsSplittingEnabled();
if (!FLAG_untrusted_code_mitigations) SetFlag(kSwitchJumpTableEnabled);
SetFlag(kSwitchJumpTableEnabled);
if (FLAG_untrusted_code_mitigations) MarkAsPoisoningRegisterArguments();
// TODO(yangguo): Disable this in case of debugging for crbug.com/826613
......@@ -62,12 +62,10 @@ OptimizedCompilationInfo::OptimizedCompilationInfo(
#endif
SetTracingFlags(
PassesFilter(debug_name, CStrVector(FLAG_trace_turbo_filter)));
if (!FLAG_untrusted_code_mitigations) {
// Embedded builtins don't support embedded absolute code addresses, so we
// cannot use jump tables.
if (code_kind != Code::BUILTIN) {
SetFlag(kSwitchJumpTableEnabled);
}
// Embedded builtins don't support embedded absolute code addresses, so we
// cannot use jump tables.
if (code_kind != Code::BUILTIN) {
SetFlag(kSwitchJumpTableEnabled);
}
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment