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

Issue 353823004: Remove unused Debug::AfterCompileFlags flags (Closed)

Created:
6 years, 5 months ago by kozyatinskiy1
Modified:
6 years, 5 months ago
Reviewers:
vsevik, Yang
CC:
v8-dev
Base URL:
git://github.com/v8/v8.git@master
Project:
v8
Visibility:
Public.

Description

Remove unused Debug::AfterCompileFlags flags. R=yangguo@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=22042

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -16 lines) Patch
M src/compiler.cc View 1 chunk +1 line, -1 line 0 comments Download
M src/debug.h View 1 chunk +1 line, -7 lines 0 comments Download
M src/debug.cc View 3 chunks +1 line, -7 lines 0 comments Download
M src/liveedit.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
kozyatinskiy1
Flags was added with live edit. Now, it isn't used. There is one function - ...
6 years, 5 months ago (2014-06-25 15:23:15 UTC) #1
kozyatinskiy1
Yang, please take a look.
6 years, 5 months ago (2014-06-25 17:43:45 UTC) #2
Yang
On 2014/06/25 17:43:45, kozyatinskiy wrote: > Yang, please take a look. lgtm.
6 years, 5 months ago (2014-06-26 06:32:28 UTC) #3
yurys
6 years, 5 months ago (2014-06-26 15:12:14 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 manually as r22042 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698