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

Issue 310773003: When flag --nouse-osr is set, don't allow osr from hidden runtime calls. (Closed)

Created:
6 years, 6 months ago by mvstanton
Modified:
6 years, 6 months ago
Reviewers:
Yang
CC:
v8-dev
Visibility:
Public.

Description

When flag --nouse-osr is set, don't allow osr from hidden runtime calls. BUG=379770 R=yangguo@chromium.org LOG=N Committed: https://code.google.com/p/v8/source/detail?r=21622

Patch Set 1 #

Total comments: 1

Patch Set 2 : Code comments. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+33 lines, -3 lines) Patch
M src/runtime.cc View 1 2 chunks +7 lines, -3 lines 0 comments Download
A test/mjsunit/regress/regress-379770.js View 1 1 chunk +26 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
mvstanton
Hi Yang, Here is a small OSR issue. PTAL, thx! --Michael
6 years, 6 months ago (2014-06-03 05:51:18 UTC) #1
Yang
LGTM, but the test case can be a *lot* simpler. Something like Flags: --nouse-osr function ...
6 years, 6 months ago (2014-06-03 07:10:13 UTC) #2
mvstanton
Thanks, I simplified the test and added the runtime assert. Much appreciated! --Michael
6 years, 6 months ago (2014-06-03 07:43:51 UTC) #3
mvstanton
6 years, 6 months ago (2014-06-03 07:45:47 UTC) #4
Message was sent while issue was closed.
Committed patchset #2 manually as r21622 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698