object.c: use RCLASS_M_TBL_WRAPPER for equality checks

* class.c (include_modules_at): use RCLASS_M_TBL_WRAPPER for
  equality checks. this avoids an unnecessary deference inside a tight
  loop, fixing a performance regression from r43973.
* object.c (rb_obj_is_kind_of): ditto.
* object.c (rb_class_inherited_p): ditto.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@44156 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
tmm1 2013-12-12 23:19:02 +00:00
Родитель f27509fd1a
Коммит c2dcb947aa
3 изменённых файлов: 12 добавлений и 4 удалений

Просмотреть файл

@ -1,3 +1,11 @@
Fri Dec 13 08:15:31 2013 Aman Gupta <ruby@tmm1.net>
* class.c (include_modules_at): use RCLASS_M_TBL_WRAPPER for
equality checks. this avoids an unnecessary deference inside a tight
loop, fixing a performance regression from r43973.
* object.c (rb_obj_is_kind_of): ditto.
* object.c (rb_class_inherited_p): ditto.
Wed Dec 13 02:00:00 2013 Kenta Murata <mrkn@mrkn.jp> Wed Dec 13 02:00:00 2013 Kenta Murata <mrkn@mrkn.jp>
* ext/bigdecimal/bigdecimal.c (VpSetPTR): fix for limitation of the resulting * ext/bigdecimal/bigdecimal.c (VpSetPTR): fix for limitation of the resulting

Просмотреть файл

@ -862,7 +862,7 @@ include_modules_at(const VALUE klass, VALUE c, VALUE module)
for (p = RCLASS_SUPER(klass); p; p = RCLASS_SUPER(p)) { for (p = RCLASS_SUPER(klass); p; p = RCLASS_SUPER(p)) {
switch (BUILTIN_TYPE(p)) { switch (BUILTIN_TYPE(p)) {
case T_ICLASS: case T_ICLASS:
if (RCLASS_M_TBL(p) == RCLASS_M_TBL(module)) { if (RCLASS_M_TBL_WRAPPER(p) == RCLASS_M_TBL_WRAPPER(module)) {
if (!superclass_seen) { if (!superclass_seen) {
c = p; /* move insertion point */ c = p; /* move insertion point */
} }

Просмотреть файл

@ -646,7 +646,7 @@ rb_obj_is_kind_of(VALUE obj, VALUE c)
c = class_or_module_required(c); c = class_or_module_required(c);
c = RCLASS_ORIGIN(c); c = RCLASS_ORIGIN(c);
while (cl) { while (cl) {
if (cl == c || RCLASS_M_TBL(cl) == RCLASS_M_TBL(c)) if (cl == c || RCLASS_M_TBL_WRAPPER(cl) == RCLASS_M_TBL_WRAPPER(c))
return Qtrue; return Qtrue;
cl = RCLASS_SUPER(cl); cl = RCLASS_SUPER(cl);
} }
@ -1549,13 +1549,13 @@ rb_class_inherited_p(VALUE mod, VALUE arg)
} }
arg = RCLASS_ORIGIN(arg); arg = RCLASS_ORIGIN(arg);
while (mod) { while (mod) {
if (RCLASS_M_TBL(mod) == RCLASS_M_TBL(arg)) if (RCLASS_M_TBL_WRAPPER(mod) == RCLASS_M_TBL_WRAPPER(arg))
return Qtrue; return Qtrue;
mod = RCLASS_SUPER(mod); mod = RCLASS_SUPER(mod);
} }
/* not mod < arg; check if mod > arg */ /* not mod < arg; check if mod > arg */
while (arg) { while (arg) {
if (RCLASS_M_TBL(arg) == RCLASS_M_TBL(start)) if (RCLASS_M_TBL_WRAPPER(arg) == RCLASS_M_TBL_WRAPPER(start))
return Qfalse; return Qfalse;
arg = RCLASS_SUPER(arg); arg = RCLASS_SUPER(arg);
} }