diff options
author | Maxime Chevalier-Boisvert <maximechevalierb@gmail.com> | 2022-08-16 12:00:35 -0400 |
---|---|---|
committer | Takashi Kokubun <takashikkbn@gmail.com> | 2022-08-29 08:47:11 -0700 |
commit | 95dce1ccacb5e893bbd2bfb100c0778c5be83d47 (patch) | |
tree | bc9a18eb55d99cd1c27d1e23d937b7ae1fa1bdfc | |
parent | 09c12111d42573a19e7077bd8fa7e7cb709179de (diff) |
Temporarily disable rb_str_concat, add CI tests (https://github.com/Shopify/ruby/pull/407)
Make sure we can load the test-all runner and run test_yjit.rb
-rw-r--r-- | .cirrus.yml | 11 | ||||
-rw-r--r-- | yjit/src/codegen.rs | 2 |
2 files changed, 10 insertions, 3 deletions
diff --git a/.cirrus.yml b/.cirrus.yml index 3425ebd175..35f908df04 100644 --- a/.cirrus.yml +++ b/.cirrus.yml @@ -163,6 +163,13 @@ yjit_task: bootstraptest/test_yjit_rust_port.rb \ bootstraptest/test_yjit.rb - # FIXME: not currently working on CI, missing cargo # Check that we can do a full ruby build - #full_build_script: make -j + full_build_script: source $HOME/.cargo/env && make -j + + # Check that we can build rdoc successfully + make_rdoc_script: source $HOME/.cargo/env && make -j rdoc + + # Run John's YJIT instruction tests, and make sure we can load the test-all runner + test_yjit_script: source $HOME/.cargo/env && make test-all TESTS='test/ruby/test_yjit.rb' RUN_OPTS="--yjit-call-threshold=1" + + # TODO: check that we can we run all of test-all successfully diff --git a/yjit/src/codegen.rs b/yjit/src/codegen.rs index fd4a3e6b50..ed11f7cf0f 100644 --- a/yjit/src/codegen.rs +++ b/yjit/src/codegen.rs @@ -6134,7 +6134,7 @@ impl CodegenGlobals { //self.yjit_reg_method(rb_cString, "to_s", jit_rb_str_to_s); //self.yjit_reg_method(rb_cString, "to_str", jit_rb_str_to_s); self.yjit_reg_method(rb_cString, "bytesize", jit_rb_str_bytesize); - self.yjit_reg_method(rb_cString, "<<", jit_rb_str_concat); + //self.yjit_reg_method(rb_cString, "<<", jit_rb_str_concat); //self.yjit_reg_method(rb_cString, "+@", jit_rb_str_uplus); // Thread.current |