Revert "[turbofan] Stage --turbo-inlining behind --turbo flag." and "[turbofan] Skip more tests that fail with --turbo-inlining."
This reverts commit019f9408dc,6ed05f44afande34c343d2a. TBR=mstarzinger@chromium.org BUG=v8:4493 LOG=n Review URL: https://codereview.chromium.org/1413183004 . Cr-Commit-Position: refs/heads/master@{#31393}
This commit is contained in:
Benedikt Meurer
parent
8be20eee3b
commit
063e2316d7
@ -404,16 +404,15 @@ DEFINE_BOOL(omit_map_checks_for_leaf_maps, true,
|
||||
|
||||
// Flags for TurboFan.
|
||||
DEFINE_BOOL(turbo, false, "enable TurboFan compiler")
|
||||
DEFINE_IMPLICATION(turbo, turbo_asm_deoptimization)
|
||||
DEFINE_IMPLICATION(turbo, turbo_inlining)
|
||||
DEFINE_BOOL(turbo_shipping, true, "enable TurboFan compiler on subset")
|
||||
DEFINE_BOOL(turbo_greedy_regalloc, false, "use the greedy register allocator")
|
||||
DEFINE_BOOL(turbo_preprocess_ranges, true,
|
||||
"run pre-register allocation heuristics")
|
||||
DEFINE_BOOL(turbo_loop_stackcheck, true, "enable stack checks in loops")
|
||||
|
||||
DEFINE_IMPLICATION(turbo, turbo_asm_deoptimization)
|
||||
DEFINE_STRING(turbo_filter, "~~", "optimization filter for TurboFan compiler")
|
||||
DEFINE_BOOL(trace_turbo, false, "trace generated TurboFan IR")
|
||||
DEFINE_IMPLICATION(trace_turbo, turbo_source_positions)
|
||||
DEFINE_BOOL(trace_turbo_graph, false, "trace generated TurboFan graphs")
|
||||
DEFINE_IMPLICATION(trace_turbo_graph, trace_turbo)
|
||||
DEFINE_STRING(trace_turbo_cfg_file, NULL,
|
||||
@ -433,6 +432,7 @@ DEFINE_BOOL(turbo_types, true, "use typed lowering in TurboFan")
|
||||
DEFINE_BOOL(turbo_type_feedback, false, "use type feedback in TurboFan")
|
||||
DEFINE_BOOL(turbo_source_positions, false,
|
||||
"track source code positions when building TurboFan IR")
|
||||
DEFINE_IMPLICATION(trace_turbo, turbo_source_positions)
|
||||
DEFINE_BOOL(function_context_specialization, false,
|
||||
"enable function context specialization in TurboFan")
|
||||
DEFINE_BOOL(native_context_specialization, true,
|
||||
|
||||
@ -120,13 +120,6 @@
|
||||
'test-debug/ScriptBreakPointByIdThroughJavaScript': [PASS, NO_VARIANTS],
|
||||
'test-debug/ScriptBreakPointByNameThroughJavaScript': [PASS, NO_VARIANTS],
|
||||
|
||||
# Issue 4493: Open issues before we can ship general purpose inlining.
|
||||
'test-decls/CrossScriptGlobal': [PASS, NO_VARIANTS],
|
||||
'test-decls/CrossScriptReferencesHarmony': [PASS, NO_VARIANTS],
|
||||
'test-decls/CrossScriptLoadICs': [PASS, NO_VARIANTS],
|
||||
'test-decls/CrossScriptStoreICs': [PASS, NO_VARIANTS],
|
||||
'test-decls/Regress3941': [PASS, NO_VARIANTS],
|
||||
|
||||
############################################################################
|
||||
# Slow tests.
|
||||
'test-api/Threading1': [PASS, ['mode == debug', SLOW]],
|
||||
|
||||
@ -152,26 +152,6 @@
|
||||
# TODO(titzer): too slow in --turbo mode due to O(n^2) graph verification.
|
||||
'regress/regress-1122': [PASS, NO_VARIANTS],
|
||||
|
||||
# Issue 4493: Open issues before we can ship general purpose inlining.
|
||||
'array-reduce': [PASS, NO_VARIANTS],
|
||||
'compiler/lazy-deopt-in-literal': [PASS, NO_VARIANTS],
|
||||
'compiler/osr-infinite': [PASS, NO_VARIANTS],
|
||||
'es6/array-find': [PASS, NO_VARIANTS],
|
||||
'es6/array-findindex': [PASS, NO_VARIANTS],
|
||||
'es6/generators-runtime': [PASS, NO_VARIANTS], # only in GC stress.
|
||||
'es6/microtask-delivery': [PASS, NO_VARIANTS],
|
||||
'es6/promises': [PASS, NO_VARIANTS], # only on 64-bit.
|
||||
'es6/templates': [PASS, NO_VARIANTS],
|
||||
'es6/typedarray-find': [PASS, NO_VARIANTS],
|
||||
'es6/typedarray-findindex': [PASS, NO_VARIANTS],
|
||||
'harmony/destructuring': [PASS, NO_VARIANTS],
|
||||
'harmony/proxies': [PASS, NO_VARIANTS],
|
||||
'harmony/proxies-example-membrane': [PASS, NO_VARIANTS], # only on 64-bit.
|
||||
'harmony/reflect': [PASS, NO_VARIANTS], # only in arm64 sim debug.
|
||||
'harmony/spread-array': [PASS, NO_VARIANTS],
|
||||
'regress/cross-script-vars': [PASS, NO_VARIANTS],
|
||||
'tools/profile': [PASS, NO_VARIANTS],
|
||||
|
||||
# issue 4078:
|
||||
'allocation-site-info': [PASS, NO_VARIANTS],
|
||||
|
||||
|
||||
@ -73,13 +73,6 @@
|
||||
# TODO(mstarzinger): Investigate failure with lookup slot declaration.
|
||||
'js1_5/Regress/regress-343713': [PASS, NO_VARIANTS],
|
||||
|
||||
# Issue 4493: Open issues before we can ship general purpose inlining.
|
||||
'js1_4/Eval/eval-002': [PASS, NO_VARIANTS],
|
||||
'js1_5/Scope/scope-003': [PASS, NO_VARIANTS],
|
||||
'js1_5/Scope/regress-77578-001': [PASS, NO_VARIANTS],
|
||||
'ecma_3/ExecutionContexts/10.1.3': [PASS, NO_VARIANTS],
|
||||
'ecma_3/ExecutionContexts/regress-23346': [PASS, NO_VARIANTS],
|
||||
|
||||
############################ INVALID TESTS #############################
|
||||
|
||||
# Function length properties are configurable in ES6
|
||||
|
||||
@ -23,9 +23,6 @@
|
||||
# Slow tests.
|
||||
'benchmarks/kernel-template': [PASS, SLOW],
|
||||
'benchmarks/memset': [PASS, SLOW],
|
||||
|
||||
# Issue 4493: Open issues before we can ship general purpose inlining.
|
||||
'shell_test_runner': [PASS, NO_VARIANTS],
|
||||
}],
|
||||
|
||||
######################################################################
|
||||
|
||||
@ -60,26 +60,6 @@
|
||||
# Unicode canonicalization is not available with i18n turned off.
|
||||
'built-ins/String/prototype/localeCompare/15.5.4.9_CE': [['no_i18n', SKIP]],
|
||||
|
||||
# Issue 4493: Open issues before we can ship general purpose inlining.
|
||||
'built-ins/Math/acos/S15.8.2.2_A5': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/acos/S15.8.2.2_A6': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/asin/S15.8.2.3_A6': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/atan/S15.8.2.4_A6': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/atan2/S15.8.2.5_A7': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/atan2/S15.8.2.5_A2': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/atan2/S15.8.2.5_A3': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/atan2/S15.8.2.5_A11': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/atan2/S15.8.2.5_A12': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/atan2/S15.8.2.5_A13': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/atan2/S15.8.2.5_A15': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/atan2/S15.8.2.5_A17': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/atan2/S15.8.2.5_A18': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/atan2/S15.8.2.5_A19': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/atan2/S15.8.2.5_A24': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/log/S15.8.2.10_A6': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/pow/S15.8.2.13_A24': [PASS, NO_VARIANTS],
|
||||
'built-ins/Math/sqrt/S15.8.2.17_A6': [PASS, NO_VARIANTS],
|
||||
|
||||
###################### NEEDS INVESTIGATION #######################
|
||||
|
||||
# Possibly same cause as S8.5_A2.1, below: floating-point tests.
|
||||
|
||||
@ -37,8 +37,6 @@
|
||||
'fast/js/deep-recursion-test': [PASS, NO_VARIANTS],
|
||||
# TODO(bmeurer,svenpanne): Investigate test failure.
|
||||
'fast/js/toString-number': [SKIP],
|
||||
# Issue 4493: Open issues before we can ship general purpose inlining.
|
||||
'call-apply-crash': [PASS, NO_VARIANTS],
|
||||
}], # ALWAYS
|
||||
['mode == debug', {
|
||||
# Too slow in debug mode.
|
||||
|
||||
Loading…
Reference in New Issue
Block a user