LGTM I'll move the documentation to the header file and land.
http://codereview.chromium.org/3036008/diff/1/2 File src/arm/assembler-arm.cc (right): http://codereview.chromium.org/3036008/diff/1/2#newcode1198 src/arm/assembler-arm.cc:1198: // Saturate an optionally shifted signed value to an unsigned range. Could you move the documentation to the header file? I would be looking for it there. :) http://codereview.chromium.org/3036008/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
