LGTM with one note for your consideration:
http://codereview.chromium.org/4070003/diff/1/10 File src/frames.h (right): http://codereview.chromium.org/4070003/diff/1/10#newcode196 src/frames.h:196: Code* LookupCode(Isolate* isolate) const { "Lookup" made me think there is some hash table or other structure in which we'll be looking up... Maybe just "GetCode"? http://codereview.chromium.org/4070003/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
