LGTM with nits.

https://codereview.chromium.org/664123002/diff/60001/src/compiler/code-generator.h
File src/compiler/code-generator.h (right):

https://codereview.chromium.org/664123002/diff/60001/src/compiler/code-generator.h#newcode120
src/compiler/code-generator.h:120: Frame* frame_;
Nit: Frame* const frame_;

https://codereview.chromium.org/664123002/diff/60001/src/compiler/code-generator.h#newcode121
src/compiler/code-generator.h:121: Linkage* linkage_;
Nit: Linkage* const linkage_;

https://codereview.chromium.org/664123002/diff/60001/src/compiler/instruction-selector.h
File src/compiler/instruction-selector.h (right):

https://codereview.chromium.org/664123002/diff/60001/src/compiler/instruction-selector.h#newcode195
src/compiler/instruction-selector.h:195: Linkage* linkage_;
Nit: Linkage* const linkage_;

https://codereview.chromium.org/664123002/

--
--
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/d/optout.

Reply via email to