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

Issue 446963002: NodeAuxData needs a Zone, not a Graph. (Closed)

Created:
6 years, 4 months ago by sigurds
Modified:
6 years, 4 months ago
Reviewers:
titzer
Project:
v8
Visibility:
Public.

Description

NodeAuxData needs a Zone, not a Graph. BUG= R=titzer@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=22965

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M src/compiler/node-aux-data.h View 1 chunk +1 line, -1 line 0 comments Download
M src/compiler/node-aux-data-inl.h View 1 chunk +2 lines, -2 lines 0 comments Download
M src/compiler/source-position.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
sigurds
6 years, 4 months ago (2014-08-06 18:11:22 UTC) #1
sigurds
On 2014/08/06 18:11:22, sigurds wrote: Maybe I should clarify: Some phases will need to set ...
6 years, 4 months ago (2014-08-06 18:23:26 UTC) #2
titzer
lgtm
6 years, 4 months ago (2014-08-07 07:02:40 UTC) #3
sigurds
6 years, 4 months ago (2014-08-07 09:11:51 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 manually as 22965 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698