LGTM if comment addressed.
https://chromiumcodereview.appspot.com/18887002/diff/5001/src/hydrogen.cc
File src/hydrogen.cc (right):
https://chromiumcodereview.appspot.com/18887002/diff/5001/src/hydrogen.cc#newcode4990
src/hydrogen.cc:4990: // Second chance: the property is on the prototype
and all maps have the
Maybe change the wording to "check whether the property is on the
prototype and [...]"
https://chromiumcodereview.appspot.com/18887002/diff/5001/src/hydrogen.cc#newcode5003
src/hydrogen.cc:5003: if (lookup.IsFound()) return false;
how can you return false here?
https://chromiumcodereview.appspot.com/18887002/
--
--
v8-dev mailing list
[email protected]
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 [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
