LGTM if you move most of the method bodies to utils.cc.
http://codereview.chromium.org/3181036/diff/1/9 File src/utils.h (right): http://codereview.chromium.org/3181036/diff/1/9#newcode541 src/utils.h:541: Vector<T> ToVector() { You should move some of these methods to utils.cc and only keep the ones you actually want inlined in the header file. http://codereview.chromium.org/3181036/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
