diff options
author | naruse <naruse@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2016-03-21 13:17:45 +0000 |
---|---|---|
committer | naruse <naruse@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2016-03-21 13:17:45 +0000 |
commit | f5715289ac9a3a0dd0dafabb7ff27482e5f07189 (patch) | |
tree | 7a3915a772d30e84e4aac440e208308fd228cadc /time.c | |
parent | d17852b9d553e0a188a3146e4e66ea6645acdf08 (diff) | |
download | ruby-f5715289ac9a3a0dd0dafabb7ff27482e5f07189.tar.gz |
* time.c (mod): Add Fixnum case.
* time.c (quo): c can be Fixnum except a == FIXNUM_MIN && b == -1.
Such case can be optimized out because quo()'s argument is constant.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@54213 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'time.c')
-rw-r--r-- | time.c | 13 |
1 files changed, 8 insertions, 5 deletions
@@ -107,10 +107,12 @@ mul(VALUE x, VALUE y) static VALUE mod(VALUE x, VALUE y) { - switch (TYPE(x)) { - case T_BIGNUM: return rb_big_modulo(x, y); - default: return rb_funcall(x, '%', 1, y); + if (FIXNUM_P(y)) { + if (FIX2LONG(y) == 0) rb_num_zerodiv(); + if (FIXNUM_P(x)) return LONG2FIX(rb_mod(FIX2LONG(x), FIX2LONG(y))); } + if (RB_TYPE_P(x, T_BIGNUM)) return rb_big_modulo(x, y); + return rb_funcall(x, '%', 1, y); } #define neg(x) (sub(INT2FIX(0), (x))) @@ -124,9 +126,10 @@ quo(VALUE x, VALUE y) a = FIX2LONG(x); b = FIX2LONG(y); if (b == 0) rb_num_zerodiv(); + if (a == FIXNUM_MIN && b == -1) LONG2NUM(-a); c = a / b; - if (c * b == a) { - return LONG2NUM(c); + if (a % b == 0) { + return LONG2FIX(c); } } ret = rb_funcall(x, id_quo, 1, y); |