Good catch, thanks for the fix! LGTM with comment:
The ARM version can be turned into a one-liner too, see patch set 2. (The
diff
view between patch sets is screwed up due to rebasing, sorry about that.)
I'll land this for you.
https://codereview.chromium.org/22964004/diff/1/test/mjsunit/shift-for-integer-div.js
File test/mjsunit/shift-for-integer-div.js (right):
https://codereview.chromium.org/22964004/diff/1/test/mjsunit/shift-for-integer-div.js#newcode1
test/mjsunit/shift-for-integer-div.js:1: // Copyright 2013 the V8
project authors. All rights reserved.
nit: we don't update copyright years anymore.
https://codereview.chromium.org/22964004/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
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 v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.
