Reviewers: Benedikt Meurer,

Message:
On 2014/11/13 09:22:05, Benedikt Meurer wrote:

https://codereview.chromium.org/700753003/diff/240001/test/unittests/compiler/register-allocator-unittest.cc
File test/unittests/compiler/register-allocator-unittest.cc (right):


https://codereview.chromium.org/700753003/diff/240001/test/unittests/compiler/register-allocator-unittest.cc#newcode362
test/unittests/compiler/register-allocator-unittest.cc:362: auto* unallocated
=
new (zone()) UnallocatedOperand(policy);
auto* looks weird, how about auto instead? :-)

auto* is supposed to be used for ranged loops, i'm not sure about elsewhere

Description:
[turbofan] add some registerallocator unittests

BUG=

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

Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files (+389, -164 lines):
  M test/unittests/compiler/register-allocator-unittest.cc


--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to