Factor out the code for emitting the IA32 binary operations div and...

Factor out the code for emitting the IA32 binary operations div and
mod so they do not share code with the other binary operations.  They
now preallocate their fixed registers (eax and edx).  There is now no
frame effect between entries to the deferred call to the stub.


Committed: http://code.google.com/p/v8/source/detail?r=2087

[William Hesse, 2009-06-02 09:11:54]

LGTM.

http://codereview.chromium.org/118110/diff/1/2
File src/ia32/codegen-ia32.cc (right):

http://codereview.chromium.org/118110/diff/1/2#newcode5778
Line 5778: if (!(remainder.is_register() && remainder.reg().is(edx))) {
remainder is either invalid or edx at this point, so only the test is_register()
is needed.

[Kevin Millikin (Chromium), 2009-06-02 10:36:59]

http://codereview.chromium.org/118110/diff/1/2
File src/ia32/codegen-ia32.cc (right):

http://codereview.chromium.org/118110/diff/1/2#newcode5778
Line 5778: if (!(remainder.is_register() && remainder.reg().is(edx))) {
Thanks.  I've changed it to !remainder.is_valid() which reads much better.