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

Issue 8429001: Use placement-new for zone-allocation in the Lithium classes. (Closed)

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

Description

Use placement-new for zone-allocation in the Lithium classes. This avoid calling Isolate::Current on each zone-allocation of a Lithium object. Committed: http://code.google.com/p/v8/source/detail?r=9856

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+202 lines, -170 lines) Patch
M src/ia32/lithium-ia32.h View 3 chunks +3 lines, -0 lines 0 comments Download
M src/ia32/lithium-ia32.cc View 1 87 chunks +199 lines, -170 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
fschneider
9 years, 1 month ago (2011-10-31 15:03:34 UTC) #1
fschneider
A version without macro for ia32 only.
9 years, 1 month ago (2011-11-01 08:34:46 UTC) #2
Vyacheslav Egorov (Chromium)
9 years, 1 month ago (2011-11-01 11:07:48 UTC) #3
lgtm

Powered by Google App Engine
This is Rietveld 408576698