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

Issue 8372085: MIPS: Remove the forward-bailout stack from the non-optimizing compiler. (Closed)

Created:
9 years, 1 month ago by kisg
Modified:
9 years, 1 month ago
CC:
v8-dev
Visibility:
Public.

Description

MIPS: Remove the forward-bailout stack from the non-optimizing compiler. Port r9863 (0996dc77) Original commit message: This was pretty heavyweight. It was kept in just for a few corner cases that assumed it was there. We can work around them by making sure that the expression in a reified test context is always really the expression that was visited in that context; and by inspecting the context manually and consing up a pair of extra AST IDs for the unusual case of unary not in a value AST context. BUG= TEST= Committed: http://code.google.com/p/v8/source/detail?r=9865

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+133 lines, -90 lines) Patch
M src/mips/full-codegen-mips.cc View 62 chunks +133 lines, -90 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
kisg
Please land together with the following to avoid build and test errors: http://codereview.chromium.org/8387044/ Thanks, Gergely
9 years, 1 month ago (2011-11-02 22:49:30 UTC) #1
Kevin Millikin (Chromium)
9 years, 1 month ago (2011-11-03 08:56:25 UTC) #2
Both changes LGTM, I will land them.

Powered by Google App Engine
This is Rietveld 408576698