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

Issue 17573003: Add phase zone to CompilationInfo and use it in GVN pass. (Closed)

Created:
7 years, 6 months ago by Benedikt Meurer
Modified:
7 years, 6 months ago
Reviewers:
danno
Base URL:
git@github.com:v8/v8.git@master
Visibility:
Public.

Description

Add phase zone to CompilationInfo and use it in GVN pass. The phase_zone of CompilationInfo is intended for local allocations that are freed at the end of the phase. R=danno@chromium.org BUG= Committed: 15294

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+66 lines, -33 lines) Patch
M src/compiler.h View 7 chunks +29 lines, -13 lines 0 comments Download
M src/compiler.cc View 2 chunks +20 lines, -9 lines 0 comments Download
M src/hydrogen.cc View 1 chunk +1 line, -1 line 0 comments Download
M src/hydrogen-gvn.h View 1 chunk +4 lines, -1 line 0 comments Download
M src/hydrogen-gvn.cc View 4 chunks +12 lines, -9 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Benedikt Meurer
7 years, 6 months ago (2013-06-23 14:38:39 UTC) #1
Benedikt Meurer (Google)
7 years, 6 months ago (2013-06-23 14:39:01 UTC) #2
Benedikt Meurer (Google)
7 years, 6 months ago (2013-06-24 06:48:01 UTC) #3
danno
lgtm
7 years, 6 months ago (2013-06-24 12:10:27 UTC) #4
Benedikt Meurer
7 years, 6 months ago (2013-06-24 13:37:53 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 manually as r15294 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698