https://codereview.chromium.org/14895019/diff/2001/src/hydrogen-instructions.cc
File src/hydrogen-instructions.cc (right):
https://codereview.chromium.org/14895019/diff/2001/src/hydrogen-instructions.cc#newcode812
src/hydrogen-instructions.cc:812: HValue *value = OperandAt(i);
On 2013/05/16 13:18:57, Jakob wrote:
nit: "HValue* value"
...or just "OperandAt(i)->PrintNameTo(stream);"
Done.
https://codereview.chromium.org/14895019/diff/2001/src/hydrogen.cc
File src/hydrogen.cc (right):
https://codereview.chromium.org/14895019/diff/2001/src/hydrogen.cc#newcode5442
src/hydrogen.cc:5442: void HGraph::MarkLive(HValue *ref, HValue* instr,
On 2013/05/16 13:18:57, Jakob wrote:
nit: while you're here: "HValue* ref", and if all parameters in a
method/function declaration/definition don't fit on one line, they
each get
their own, as in:
void HGraph::MarkLive(HValue* ref,
HValue* instr,
ZoneList<HValue*>* worklist) {
Fits :)
https://codereview.chromium.org/14895019/diff/2001/src/hydrogen.cc#newcode5451
src/hydrogen.cc:5451: ALLOW_HANDLE_DEREF(isolate(), "debug mode
printing");
On 2013/05/16 13:18:57, Jakob wrote:
I don't see any handle being dereferenced?
Yeah, it can happen if you try to print an instruction which is a
HConstant of a map.
https://codereview.chromium.org/14895019/diff/2001/test/mjsunit/regress/regress-seqstrsetchar-ex1.js
File test/mjsunit/regress/regress-seqstrsetchar-ex1.js (right):
https://codereview.chromium.org/14895019/diff/2001/test/mjsunit/regress/regress-seqstrsetchar-ex1.js#newcode32
test/mjsunit/regress/regress-seqstrsetchar-ex1.js:32: return 311;
On 2013/05/16 13:18:57, Jakob wrote:
nit: indentation
Done.
https://codereview.chromium.org/14895019/
--
--
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.
