Skip to content

Commit 57f57db

Browse files
rmacnak-googleCommit Queue
authored and
Commit Queue
committed
[vm] Don't assert assuming FLAG_max_polymorphic_checks is the same between AppJIT training and execution.
Compare FLAG_optimization_counter_threshold, which can likewise change; optimized compilation might happen sooner or later than with a consistent flag value, but should still be correct. TEST=dartfuzz Bug: #55625 Change-Id: Iced616163ecf8b3d0a811c67e0c01a9be3d8b18b Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/365904 Reviewed-by: Alexander Markov <[email protected]> Commit-Queue: Ryan Macnak <[email protected]>
1 parent 1658200 commit 57f57db

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

runtime/vm/object.cc

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17501,9 +17501,6 @@ ICDataPtr ICData::Clone(const ICData& from) {
1750117501
result.SetDeoptReasons(from.DeoptReasons());
1750217502
result.set_is_megamorphic(is_megamorphic);
1750317503

17504-
RELEASE_ASSERT(!is_megamorphic ||
17505-
result.NumberOfChecks() >= FLAG_max_polymorphic_checks);
17506-
1750717504
DEBUG_ONLY(result.AssertInvariantsAreSatisfied());
1750817505

1750917506
return result.ptr();

0 commit comments

Comments
 (0)