LGTM with nits: Remove another print-related occurrence of the flag + improve the flag description in flag-definitions.h a bit, "hydrogen use/trace filter" is
a bit cryptic and doesn't even mention the "-" prefix.


https://chromiumcodereview.appspot.com/9691042/diff/5001/src/codegen.cc
File src/codegen.cc (right):

https://chromiumcodereview.appspot.com/9691042/diff/5001/src/codegen.cc#newcode127
src/codegen.cc:127: if (print_code) {
If we remove the flag here, it should probably be removed in
MakeCodePrologue, too.

https://chromiumcodereview.appspot.com/9691042/

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

Reply via email to