[前][次][番号順一覧][スレッド一覧]

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/

[前][次][番号順一覧][スレッド一覧]