Commit 9ec70896 authored by Mythri A's avatar Mythri A Committed by V8 LUCI CQ

lazy-feedback-allocation conflicts with stress-concurrent-inlining

stress-concurrent-inlining has a negative implication for
lazy-feedback-allocation. So add lazy-feedback-allocation as
incompatible flag with stress-concurrent-inlining.

Bug: v8:12088, v8:11947
Change-Id: Ia8ff66c595f6c6288b44f7a066729ace0d7ad9d8
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3113630
Commit-Queue: Mythri Alle <mythria@chromium.org>
Reviewed-by: 's avatarLeszek Swirski <leszeks@chromium.org>
Cr-Commit-Position: refs/heads/main@{#76483}
parent 8ab11efb
...@@ -1063,8 +1063,6 @@ ...@@ -1063,8 +1063,6 @@
'baseline/test-osr': [SKIP], 'baseline/test-osr': [SKIP],
# BUG(v8:11826) Skipped until we remove flakes on NumFuzz. # BUG(v8:11826) Skipped until we remove flakes on NumFuzz.
'baseline/flush-only-baseline-code': [SKIP],
'baseline/flush-baseline-code': [SKIP],
'regress/wasm/regress-1231950': [SKIP], 'regress/wasm/regress-1231950': [SKIP],
# BUG(v8:12013) Skipped until we remove flakes on NumFuzz. # BUG(v8:12013) Skipped until we remove flakes on NumFuzz.
......
...@@ -57,7 +57,8 @@ INCOMPATIBLE_FLAGS_PER_VARIANT = { ...@@ -57,7 +57,8 @@ INCOMPATIBLE_FLAGS_PER_VARIANT = {
"nooptimization": ["--always-opt"], "nooptimization": ["--always-opt"],
"slow_path": ["--no-force-slow-path"], "slow_path": ["--no-force-slow-path"],
"stress_concurrent_allocation": ["--single-threaded-gc", "--predictable"], "stress_concurrent_allocation": ["--single-threaded-gc", "--predictable"],
"stress_concurrent_inlining": ["--single-threaded", "--predictable", "--turboprop"], "stress_concurrent_inlining": ["--single-threaded", "--predictable",
"--turboprop", "--lazy-feedback-allocation"],
"turboprop": ["--stress_concurrent_inlining"], "turboprop": ["--stress_concurrent_inlining"],
# The fast API tests initialize an embedder object that never needs to be # The fast API tests initialize an embedder object that never needs to be
# serialized to the snapshot, so we don't have a # serialized to the snapshot, so we don't have a
......
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