Fix tick processor to correctly handle the 'parallel-compiler' tick. (issue 11347043)

5 views
Skip to first unread message

rkri...@codeaurora.org

unread,
Oct 30, 2012, 6:11:27 PM10/30/12
to da...@chromium.org, v8-...@googlegroups.com
Reviewers: danno,

Message:
Hi Danno,
I would appreciate it if you could review my change.
Thanks!

Description:
Fix tick processor to correctly handle the 'parallel-compiler' tick.
BUG=none
TEST=none

Please review this at https://codereview.chromium.org/11347043/

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
M tools/tickprocessor.js


Index: tools/tickprocessor.js
===================================================================
--- tools/tickprocessor.js (revision 12764)
+++ tools/tickprocessor.js (working copy)
@@ -231,8 +231,9 @@
JS: 0,
GC: 1,
COMPILER: 2,
- OTHER: 3,
- EXTERNAL: 4
+ PARALLEL_COMPILER_PROLOGUE: 3,
+ OTHER: 4,
+ EXTERNAL: 5
};




yan...@chromium.org

unread,
Oct 31, 2012, 7:41:20 AM10/31/12
to rkri...@codeaurora.org, da...@chromium.org, v8-...@googlegroups.com
On 2012/10/30 22:11:27, rkrithiv wrote:
> Hi Danno,
> I would appreciate it if you could review my change.
> Thanks!

Thanks for the patch! LGTM. I'll land it.

https://codereview.chromium.org/11347043/
Reply all
Reply to author
Forward
0 new messages