Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(21)

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
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 Delta from patch set Stats (+58 lines, -1 line) Patch
M src/extensions/gc-extension.cc View 1 chunk +5 lines, -1 line 0 comments Download
M test/mjsunit/mjsunit.status View 1 2 chunks +4 lines, -0 lines 0 comments Download
A test/mjsunit/regress/regress-deopt-gc.js View 1 chunk +49 lines, -0 lines 0 comments Download

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
This is Rietveld 408576698