http://codereview.chromium.org/7114004/diff/6002/src/hydrogen-instructions.h
File src/hydrogen-instructions.h (right):

http://codereview.chromium.org/7114004/diff/6002/src/hydrogen-instructions.h#newcode802
src/hydrogen-instructions.h:802: bool HasNext() { return next_ <
instr_->SuccessorCount(); }
On 2011/06/10 11:39:15, Kevin Millikin wrote:
Most of the existing hydrogen/lithium iterators have changed to have
the
interface

bool Done();
T* Current();
void Advance();

Could you change this one?

Done.

http://codereview.chromium.org/7114004/diff/6002/src/hydrogen-instructions.h#newcode854
src/hydrogen-instructions.h:854: class HDeoptimize : public
HControlInstruction {
On 2011/06/10 11:39:15, Kevin Millikin wrote:
The rest of this file, with only a coupld of exceptions, doesn't have
a space
before ':'

Done.

http://codereview.chromium.org/7114004/

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

Reply via email to