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

Issue 184373004: Fail early when re-entering code that has been patched for lazy deopt (Closed)

Created:
6 years, 9 months ago by Jakob Kummerow
Modified:
6 years, 9 months ago
Reviewers:
ulan
CC:
v8-dev
Visibility:
Public.

Description

Fail early when re-entering code that has been patched for lazy deopt R=ulan@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=19607

Patch Set 1 #

Patch Set 2 : cover OSR entry too #

Unified diffs Side-by-side diffs Delta from patch set Stats (+85 lines, -0 lines) Patch
M src/a64/deoptimizer-a64.cc View 1 chunk +3 lines, -0 lines 0 comments Download
M src/arm/deoptimizer-arm.cc View 1 1 chunk +20 lines, -0 lines 0 comments Download
M src/ia32/deoptimizer-ia32.cc View 1 1 chunk +21 lines, -0 lines 0 comments Download
M src/mips/deoptimizer-mips.cc View 1 1 chunk +20 lines, -0 lines 0 comments Download
M src/objects.h View 1 chunk +1 line, -0 lines 0 comments Download
M src/x64/deoptimizer-x64.cc View 1 1 chunk +20 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Jakob Kummerow
PTAL.
6 years, 9 months ago (2014-02-28 10:35:06 UTC) #1
ulan
lgtm
6 years, 9 months ago (2014-02-28 10:41:30 UTC) #2
Jakob Kummerow
New patch set patches OSR entry points too.
6 years, 9 months ago (2014-02-28 12:28:25 UTC) #3
ulan
lgtm
6 years, 9 months ago (2014-02-28 12:34:27 UTC) #4
Jakob Kummerow
6 years, 9 months ago (2014-02-28 12:41:34 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 manually as r19607 (tree was closed).

Powered by Google App Engine
This is Rietveld 408576698