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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 109303006: Revert "Remove flag track-allocation-sites." (Closed)

Created:
7 years ago by Hannes Payer (out of office)
Modified:
7 years ago
Reviewers:
Jakob Kummerow
CC:
v8-dev
Visibility:
Public.

Description

Revert "Remove flag track-allocation-sites." This reverts commit 6c430da40efe388035504d3603756aa8c46ed1dc. BUG= Committed: https://code.google.com/p/v8/source/detail?r=18386

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+114 lines, -155 lines) Patch
M src/code-stubs.h View 1 chunk +2 lines, -1 line 0 comments Download
M src/flag-definitions.h View 1 chunk +2 lines, -0 lines 0 comments Download
M src/ia32/code-stubs-ia32.cc View 1 chunk +6 lines, -1 line 0 comments Download
M src/ia32/full-codegen-ia32.cc View 1 chunk +2 lines, -1 line 0 comments Download
M src/objects.cc View 2 chunks +21 lines, -19 lines 0 comments Download
M src/objects-inl.h View 2 chunks +4 lines, -2 lines 0 comments Download
M test/mjsunit/allocation-site-info.js View 1 chunk +1 line, -1 line 0 comments Download
M test/mjsunit/array-constructor-feedback.js View 1 chunk +1 line, -1 line 0 comments Download
M test/mjsunit/array-feedback.js View 1 chunk +1 line, -1 line 0 comments Download
M test/mjsunit/array-literal-feedback.js View 1 chunk +1 line, -1 line 0 comments Download
M test/mjsunit/array-natives-elements.js View 1 chunk +1 line, -0 lines 0 comments Download
M test/mjsunit/elements-kind.js View 2 chunks +45 lines, -59 lines 0 comments Download
M test/mjsunit/elements-transition.js View 3 chunks +11 lines, -29 lines 0 comments Download
M test/mjsunit/elements-transition-and-store.js View 1 chunk +2 lines, -0 lines 0 comments Download
M test/mjsunit/elements-transition-hoisting.js View 3 chunks +5 lines, -10 lines 0 comments Download
M test/mjsunit/opt-elements-kind.js View 2 chunks +2 lines, -13 lines 0 comments Download
M test/mjsunit/osr-elements-kind.js View 2 chunks +2 lines, -12 lines 0 comments Download
M test/mjsunit/regress/regress-320532.js View 1 chunk +1 line, -1 line 0 comments Download
M test/mjsunit/regress/regress-convert-hole2.js View 1 chunk +1 line, -1 line 0 comments Download
M test/mjsunit/regress/regress-crbug-244461.js View 1 chunk +1 line, -0 lines 0 comments Download
M test/mjsunit/regress/regress-crbug-245480.js View 1 chunk +1 line, -1 line 0 comments Download
M test/mjsunit/transition-elements-kind.js View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
Hannes Payer (out of office)
TBR
7 years ago (2013-12-20 12:04:26 UTC) #1
Hannes Payer (out of office)
Committed patchset #1 manually as r18386.
7 years ago (2013-12-20 12:04:51 UTC) #2
Jakob Kummerow
7 years ago (2013-12-20 12:05:50 UTC) #3
Message was sent while issue was closed. 
rubberstamp LGTM except for the commit message: please mention the SVN revision
instead of the git commit hash of the CL you're reverting.

Powered by Google App Engine
This is Rietveld 408576698