LGTM (with a nit).

https://chromiumcodereview.appspot.com/9702026/diff/1/src/objects-inl.h
File src/objects-inl.h (right):

https://chromiumcodereview.appspot.com/9702026/diff/1/src/objects-inl.h#newcode97
src/objects-inl.h:97: // Getter that returns a raw Smi and setter that
writes a raw Smi.
I would refer to them as "tagged Smi" instead of "raw Smi".

https://chromiumcodereview.appspot.com/9702026/

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to