ruby-changes:55331
From: k0kubun <ko1@a...>
Date: Sun, 14 Apr 2019 14:52:21 +0900 (JST)
Subject: [ruby-changes:55331] k0kubun:r67538 (trunk): Revert "Try to set false explicitly"
k0kubun 2019-04-14 14:52:14 +0900 (Sun, 14 Apr 2019) New Revision: 67538 https://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=67538 Log: Revert "Try to set false explicitly" This reverts commit a1b5d20068a9b1859b383b249e510cd33d196e4a. Revert "Revert "Skip recompiling tests on i686 Linux"" This reverts commit 7b88a9207b97b94bc3c3be35084c742296f4aff2. Revert "Simplify matrix for debugging" This reverts commit e85d6c5c5e62dde37c6f6ffdb7125b9008b9ebfa. Sorry, these 3 commits were under debugging in https://github.com/ruby/ruby/pull/2129 but accidentally merged by using `git svn dcommit` instead of `git push` to the topic branch :bow: Modified files: trunk/.travis.yml trunk/mjit.c trunk/test/ruby/test_jit.rb Index: mjit.c =================================================================== --- mjit.c (revision 67537) +++ mjit.c (revision 67538) @@ -307,12 +307,8 @@ mjit_add_iseq_to_process(const rb_iseq_t https://github.com/ruby/ruby/blob/trunk/mjit.c#L307 iseq->body->jit_func = (mjit_func_t)NOT_READY_JIT_ISEQ_FUNC; create_unit(iseq); - if (compile_info != NULL) { + if (compile_info != NULL) iseq->body->jit_unit->compile_info = *compile_info; - } else { - iseq->body->jit_unit->compile_info.disable_ivar_cache = false; - iseq->body->jit_unit->compile_info.disable_send_cache = false; - } if (iseq->body->jit_unit == NULL) // Failure in creating the unit. return; Index: test/ruby/test_jit.rb =================================================================== --- test/ruby/test_jit.rb (revision 67537) +++ test/ruby/test_jit.rb (revision 67538) @@ -529,6 +529,9 @@ class TestJIT < Test::Unit::TestCase https://github.com/ruby/ruby/blob/trunk/test/ruby/test_jit.rb#L529 print block.call(obj) end; + if RUBY_PLATFORM =~ /i686/ + skip 'recompilation is not happening on i686' + end # send call -> optimized call (send JIT) -> optimized call assert_eval_with_jit("#{<<~"begin;"}\n#{<<~"end;"}", stdout: '122', success_count: 2, min_calls: 2) begin; @@ -705,6 +708,9 @@ class TestJIT < Test::Unit::TestCase https://github.com/ruby/ruby/blob/trunk/test/ruby/test_jit.rb#L708 end def test_inlined_undefined_ivar + if RUBY_PLATFORM =~ /i686/ + skip 'recompilation is not happening on i686' + end assert_eval_with_jit("#{<<~"begin;"}\n#{<<~"end;"}", stdout: "bbb", success_count: 3, min_calls: 3) begin; class Foo Index: .travis.yml =================================================================== --- .travis.yml (revision 67537) +++ .travis.yml (revision 67538) @@ -291,7 +291,25 @@ env: https://github.com/ruby/ruby/blob/trunk/.travis.yml#L291 matrix: include: # to reduce time for finishing all jobs, run the slowest osx build first. + - <<: *x86_64-darwin17 + - <<: *x86_64-linux - <<: *i686-linux + - <<: *jemalloc + - <<: *pedanticism + - <<: *ASAN + - <<: *MSAN + - <<: *UBSAN + - <<: *VM_CHECK_MODE + - <<: *FIBER_USE_sjlj + - <<: *TOKEN_THREADED_CODE + - <<: *CALL_THREADED_CODE + - <<: *NO_THREADED_CODE + - <<: *rubyspec + allow_failures: + - name: uinversal.x86_64h-darwin17 + - name: -fsanitize=address + - name: -fsanitize=memory + - name: -fsanitize=undefined fast_finish: true before_script: @@ -359,7 +377,9 @@ before_script: https://github.com/ruby/ruby/blob/trunk/.travis.yml#L377 $RUBY_PREFIX/bin/gem install --no-document $GEMS_FOR_TEST script: - - $SETARCH make -s test-all -o exts TESTOPTS="-n 'TestJIT#test_compile_insn_opt_aref' -n 'TestJIT#test_inlined_undefined_ivar'" + - $SETARCH make -s test TESTOPTS="${TESTOPTS=$JOBS -q --tty=no}" + - travis_wait 40 $SETARCH make -s test-all -o exts TESTOPTS="${TEST_ALL_OPTS=$TESTOPTS}" RUBYOPT="-w" + - $SETARCH make -s test-spec MSPECOPT=-ff # not using `-j` because sometimes `mspec -j` silently dies # Branch matrix. Not all branches are Travis-ready so we limit branches here. branches: -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/