From b87962eb04f6cc25f97203662e04c3ca39bbb795 Mon Sep 17 00:00:00 2001 From: Alan Wu Date: Fri, 19 Feb 2021 15:44:53 -0500 Subject: [PATCH] Stop using UJIT.install_entry in btest CI runs on some platforms uJIT doesn't support. They don't have UJIT.install_entry. Maybe we want something like `install_entry_if_possible`. --- bootstraptest/test_ujit.rb | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/bootstraptest/test_ujit.rb b/bootstraptest/test_ujit.rb index 5d6695b34a..a5252d4570 100644 --- a/bootstraptest/test_ujit.rb +++ b/bootstraptest/test_ujit.rb @@ -168,7 +168,8 @@ assert_normal_exit %q{ end new.foo - UJIT.install_entry(RubyVM::InstructionSequence.of(instance_method(:foo))) + new.foo + new.foo new.foo end @@ -199,20 +200,20 @@ assert_equal "nil\n", %q{ } # Test that opt_aref checks the class of the receiver -assert_equal ":special\n", %q{ +assert_equal 'special', %q{ def foo(array) array[30] end - UJIT.install_entry(RubyVM::InstructionSequence.of(method(:foo))) + foo([]) + foo([]) special = [] def special.[](idx) - :special + 'special' end - p foo(special) - nil + foo(special) } # Test that object references in generated code get marked and moved