Clarify result returned by remainder intrinsics.

Clarify result returned by remainder intrinsics.

R=zra@google.com

Committed: https://code.google.com/p/dart/source/detail?r=43576

[regis, 2015-02-06 22:22:43]

regis@google.com changed reviewers:
+ zra@google.com

[regis, 2015-02-06 22:22:44]

Comments only.
It took me a while to understand why the result was sometimes tagged, sometimes
untagged.

[zra, 2015-02-06 22:27:34]

lgtm

https://codereview.chromium.org/905653004/diff/1/runtime/vm/intrinsifier_mips.cc
File runtime/vm/intrinsifier_mips.cc (right):

https://codereview.chromium.org/905653004/diff/1/runtime/vm/intrinsifier_mips...
runtime/vm/intrinsifier_mips.cc:369: //   V0: Tagged return result (remainder).
V1 is available here if it would help to be consistent with the other
architectures.

[regis, 2015-02-06 22:42:41]

Thanks!

https://codereview.chromium.org/905653004/diff/1/runtime/vm/intrinsifier_mips.cc
File runtime/vm/intrinsifier_mips.cc (right):

https://codereview.chromium.org/905653004/diff/1/runtime/vm/intrinsifier_mips...
runtime/vm/intrinsifier_mips.cc:369: //   V0: Tagged return result (remainder).
On 2015/02/06 22:27:34, zra wrote:
> V1 is available here if it would help to be consistent with the other
> architectures.

It does not have to be. X64 uses RAX in both cases as well.

[regis, 2015-02-06 22:42:58]

Committed patchset #1 (id:1) manually as r43576 (presubmit successful).