http://codereview.chromium.org/8857001/diff/19008/src/hydrogen.cc File src/hydrogen.cc (right):
http://codereview.chromium.org/8857001/diff/19008/src/hydrogen.cc#newcode3853 src/hydrogen.cc:3853: UNREACHABLE(); Maybe a short comment that this is checked statically and can't occur therefore. http://codereview.chromium.org/8857001/diff/19008/src/hydrogen.cc#newcode4044 src/hydrogen.cc:4044: UNREACHABLE(); Also a short comment for the case here that this is already checked. http://codereview.chromium.org/8857001/diff/19008/src/x64/lithium-codegen-x64.cc File src/x64/lithium-codegen-x64.cc (right): http://codereview.chromium.org/8857001/diff/19008/src/x64/lithium-codegen-x64.cc#newcode2077 src/x64/lithium-codegen-x64.cc:2077: Remove extra blank line. http://codereview.chromium.org/8857001/diff/19008/src/x64/lithium-codegen-x64.cc#newcode2079 src/x64/lithium-codegen-x64.cc:2079: Also no need for blank line here. http://codereview.chromium.org/8857001/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
