[v8-dev] Re: Greedy allocator: perf work (issue 1157663007 by mtro...@chromium.org)

2015-06-08 Thread jvoung
Some very-surface-level comments. https://codereview.chromium.org/1157663007/diff/40001/src/compiler/register-allocator.cc File src/compiler/register-allocator.cc (right): https://codereview.chromium.org/1157663007/diff/40001/src/compiler/register-allocator.cc#newcode2101

[v8-dev] Re: Introducing the LLVM greedy register allocator. (issue 1061923005 by mtro...@chromium.org)

2015-04-21 Thread jvoung
Throwing out some random nits from the CC list. https://codereview.chromium.org/1061923005/diff/290001/src/compiler/register-allocator.cc File src/compiler/register-allocator.cc (right): https://codereview.chromium.org/1061923005/diff/290001/src/compiler/register-allocator.cc#newcode2642