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

Issue 643001: Refactor assignments in the ia32 code generator. (Closed)

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

Description

Refactor assignments in the ia32 code generator. Change compilation of assignment expressions in the ia32 code generator to no longer use the Reference helper class. This gives us more explicit control over which subexpressions are evaluated and when. Committed: http://code.google.com/p/v8/source/detail?r=3899

Patch Set 1 #

Patch Set 2 : Replace inadvertent CHECK with ASSERT. #

Patch Set 3 : Merged with bleeding_edge@HEAD. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+433 lines, -321 lines) Patch
M src/ia32/codegen-ia32.h View 1 1 chunk +15 lines, -4 lines 0 comments Download
M src/ia32/codegen-ia32.cc View 1 2 13 chunks +418 lines, -317 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Kevin Millikin (Chromium)
10 years, 10 months ago (2010-02-18 11:26:47 UTC) #1
Mads Ager (chromium)
10 years, 10 months ago (2010-02-18 12:02:29 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698