It doesn't look like you've ported to ARM or MIPS in this CL yet?
https://chromiumcodereview.appspot.com/10443114/diff/11001/src/hydrogen.cc File src/hydrogen.cc (right): https://chromiumcodereview.appspot.com/10443114/diff/11001/src/hydrogen.cc#newcode699 src/hydrogen.cc:699: LChunkBuilder builder(info, this, &allocator, info->isolate()->zone()); Please put the zone in the graph. It's a simple change that I think makes sense even now. https://chromiumcodereview.appspot.com/10443114/diff/11001/src/ia32/lithium-ia32.h File src/ia32/lithium-ia32.h (right): https://chromiumcodereview.appspot.com/10443114/diff/11001/src/ia32/lithium-ia32.h#newcode2377 src/ia32/lithium-ia32.h:2377: zone_(zone), get it directly from the zone https://chromiumcodereview.appspot.com/10443114/diff/11001/src/jsregexp.cc File src/jsregexp.cc (right): https://chromiumcodereview.appspot.com/10443114/diff/11001/src/jsregexp.cc#newcode647 src/jsregexp.cc:647: zone); nit: weird indentation https://chromiumcodereview.appspot.com/10443114/diff/11001/src/list.h File src/list.h (right): https://chromiumcodereview.appspot.com/10443114/diff/11001/src/list.h#newcode48 src/list.h:48: // template <typename T, class P = FreeStoreAllocationPolicy> class List; May want to change the comment to use AllocationPolicy https://chromiumcodereview.appspot.com/10443114/diff/11001/src/splay-tree.h File src/splay-tree.h (right): https://chromiumcodereview.appspot.com/10443114/diff/11001/src/splay-tree.h#newcode190 src/splay-tree.h:190: explicit NodeDeleter() { Why is this change needed? I don't think you should add explicit here. https://chromiumcodereview.appspot.com/10443114/diff/11001/src/v8globals.h File src/v8globals.h (right): https://chromiumcodereview.appspot.com/10443114/diff/11001/src/v8globals.h#newcode159 src/v8globals.h:159: template <typename Config, class AllocationPolicy = FreeStoreAllocationPolicy> No need to change this file for just this one-line change. https://chromiumcodereview.appspot.com/10443114/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
