Skip to content

Commit

Permalink
* array.c (ary_memcpy): try to enable optimization.
Browse files Browse the repository at this point in the history
  At least on my environments, I don't see any errors
  with many trials. Please tell us if you find any GC bugs.



git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@42192 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
  • Loading branch information
ko1 committed Jul 26, 2013
1 parent 4844371 commit d320b2d
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
6 changes: 6 additions & 0 deletions ChangeLog
Original file line number Diff line number Diff line change
@@ -1,3 +1,9 @@
Fri Jul 26 18:21:58 2013 Koichi Sasada <ko1@atdot.net>

* array.c (ary_memcpy): try to enable optimization.
At least on my environments, I don't see any errors
with many trials. Please tell us if you find any GC bugs.

Fri Jul 26 17:49:26 2013 Nobuyoshi Nakada <nobu@ruby-lang.org>

* win32/file.c (fix_string_encoding): fix target encoding. the
Expand Down
3 changes: 1 addition & 2 deletions array.c
Original file line number Diff line number Diff line change
Expand Up @@ -69,8 +69,7 @@ ary_memcpy(VALUE ary, long beg, long argc, const VALUE *argv)
{
#if 1
if (OBJ_PROMOTED(ary)) {
if (0 /* disalbe now, because it cause mysterious error. */
&& argc > 32 /* 32 is magic number */) {
if (argc > (128/sizeof(VALUE)) /* is magic number (cache line size) */) {
rb_gc_writebarrier_remember_promoted(ary);
RARRAY_PTR_USE(ary, ptr, {
MEMCPY(ptr+beg, argv, VALUE, argc);
Expand Down

0 comments on commit d320b2d

Please sign in to comment.