1
0
Fork 0
mirror of https://github.com/ruby/ruby.git synced 2022-11-09 12:17:21 -05:00

merge revision(s) 57283,57284: [Backport #13113]

benchmarks for [Bug #13113] [ci skip]
	variable.c: resolve permanent name only

	* variable.c (rb_const_set): resolve and cache class name
	  immediately only if the outer class/module has the name,
	  otherwise just set the ID.  [ruby-core:79007] [Bug #13113]

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_4@57850 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
naruse 2017-03-11 15:03:50 +00:00
parent 00d2e36831
commit d4f4ba6f03
4 changed files with 24 additions and 5 deletions

View file

@ -0,0 +1,5 @@
i = 0
while i<6_000_000 # benchmark loop 2
i += 1
Module.new.const_set(:X, Module.new)
end

View file

@ -0,0 +1,8 @@
i = 0
module M
end
$VERBOSE = nil
while i<6_000_000 # benchmark loop 2
i += 1
M.const_set(:X, Module.new)
end

View file

@ -2608,7 +2608,12 @@ rb_const_set(VALUE klass, ID id, VALUE val)
* and avoid order-dependency on const_tbl
*/
if (rb_cObject && (RB_TYPE_P(val, T_MODULE) || RB_TYPE_P(val, T_CLASS))) {
rb_class_name(val);
if (!NIL_P(rb_class_path_cached(val))) {
rb_name_class(val, id);
if (rb_class_path_cached(klass)) {
rb_class_name(val);
}
}
}
}
@ -2656,7 +2661,8 @@ const_tbl_update(struct autoload_const_set_args *args)
}
rb_clear_constant_cache();
setup_const_entry(ce, klass, val, visibility);
} else {
}
else {
rb_clear_constant_cache();
ce = ZALLOC(rb_const_entry_t);

View file

@ -1,10 +1,10 @@
#define RUBY_VERSION "2.4.0"
#define RUBY_RELEASE_DATE "2017-03-11"
#define RUBY_PATCHLEVEL 17
#define RUBY_RELEASE_DATE "2017-03-12"
#define RUBY_PATCHLEVEL 18
#define RUBY_RELEASE_YEAR 2017
#define RUBY_RELEASE_MONTH 3
#define RUBY_RELEASE_DAY 11
#define RUBY_RELEASE_DAY 12
#include "ruby/version.h"