Chromium Code Reviews

Issue 6312118: Add regression test for the deoptimizer immediately followed by gc bug.... (Closed)

Created:
9 years, 10 months ago by Rico
Modified:
9 years, 6 months ago
Reviewers:
Kevin Millikin (Chromium)
CC:
v8-dev
Visibility:
Public.

Description

Add regression test for the deoptimizer immediately followed by gc bug. In addition to the regression test I changed the gc-extension to take a boolean flag specifying if compaction should be used (default is false, existing tests will not change behaviour) The regression test is disabled on arm and x64 with crankshaft enabled. I made a bug to track this: http://code.google.com/p/v8/issues/detail?id=1094 Committed: http://code.google.com/p/v8/source/detail?r=6619

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Stats (+58 lines, -1 line)
M src/extensions/gc-extension.cc View 1 chunk +5 lines, -1 line 0 comments
M test/mjsunit/mjsunit.status View 2 chunks +4 lines, -0 lines 0 comments
A test/mjsunit/regress/regress-deopt-gc.js View 1 chunk +49 lines, -0 lines 0 comments

Messages

Total messages: 2 (0 generated)
Rico
9 years, 10 months ago (2011-02-03 13:13:47 UTC) #1
Kevin Millikin (Chromium)
9 years, 10 months ago (2011-02-03 13:45:12 UTC) #2
Cute.  LGTM.

Powered by Google App Engine