[GR-16046] Use a method just doing yield rather than 1.times in Binding PE tests.

PullRequest: truffleruby/861

Author: eregon

test/truffle/compiler/pe.sh

@@ -2,4 +2,4 @@
 
 source test/truffle/common.sh.inc
 
-jt ruby --vm.Dgraal.TruffleCompilationExceptionsAreThrown=true --vm.Dgraal.TruffleIterativePartialEscape=true --experimental-options --basic-ops-inline=false test/truffle/compiler/pe/pe.rb "$@"
+jt ruby --vm.Dgraal.TruffleCompilationExceptionsAreThrown=true --vm.Dgraal.TruffleIterativePartialEscape=true test/truffle/compiler/pe/pe.rb "$@"

test/truffle/compiler/pe/core/binding_pe.rb

@@ -6,6 +6,12 @@
 # GNU General Public License version 2, or
 # GNU Lesser General Public License version 2.1.
 
+module BindingFixtures
+  def self.yielder
+    yield
+  end
+end
+
 # Kernel#binding
 example "x = 14; binding.local_variable_get(:x)", 14
 
@@ -23,13 +29,13 @@
 example "b = binding.dup; b.local_variable_set(:x, 14); b.local_variable_get(:x)", 14
 
 # get (2 levels)
-example "x = 14; y = nil; 1.times { y = binding.local_variable_get(:x) }; y", 14
+example "x = 14; y = nil; BindingFixtures.yielder { y = binding.local_variable_get(:x) }; y", 14
 
 # set (2 levels)
-example "x = nil; 1.times { binding.local_variable_set(:x, 15) }; x", 15
+example "x = nil; BindingFixtures.yielder { binding.local_variable_set(:x, 15) }; x", 15
 
 # get + set (2 levels)
-example "x = 14; y = nil; 1.times { binding.local_variable_set(:x, 15); y = binding.local_variable_get(:x) }; y", 15
+example "x = 14; y = nil; BindingFixtures.yielder { binding.local_variable_set(:x, 15); y = binding.local_variable_get(:x) }; y", 15
 
 # defined
 

test/truffle/compiler/pe/core/fixnum_pe.rb renamed to test/truffle/compiler/pe/core/integer_pe.rb

@@ -24,3 +24,7 @@
 counter example "14 / rand"
 
 example "14 <=> 2", 1
+
+example "x = 0; 1.times { x = 14 }; x", 14
+
+example "sum = 0; 3.times { sum += 14 }; sum", 42

test/truffle/compiler/pe/core/string_pe.rb

@@ -48,7 +48,7 @@
 
 example "'A' == String.from_codepoint(65, Encoding::US_ASCII)", true
 example "'A' == 65.chr", true
-example "'A'.ord == 65", true
+tagged example "'A'.ord == 65", true
 
 example "'aba'[0] == 'aca'[-1]", true
 
@@ -66,8 +66,8 @@
 
 example "x = 'abc'; y = 'xyz'; x.replace(y) == y", true
 
-example "'abc'.getbyte(0) == 97", true
-example "'abc'.getbyte(-1) == 99", true
+tagged example "'abc'.getbyte(0) == 97", true
+tagged example "'abc'.getbyte(-1) == 99", true
 example "'abc'.getbyte(10_000) == nil", true
 
 example "14.to_s.length", 2

test/truffle/compiler/pe/pe.rb

@@ -70,7 +70,7 @@ def counter(example)
   require_relative 'language/defined_pe.rb'
   require_relative 'language/global_pe.rb'
   require_relative 'core/truefalse_pe.rb'
-  require_relative 'core/fixnum_pe.rb'
+  require_relative 'core/integer_pe.rb'
   require_relative 'core/float_pe.rb'
   require_relative 'core/symbol_pe.rb'
   require_relative 'core/method_pe.rb'

tool/jt.rb

@@ -2297,7 +2297,7 @@ def dockerfile(*args)
       end
 
       configs.each do |config|
-        lines.push "RUN " + setup_env["ruby #{config} --vm.Dgraal.TruffleCompilationExceptionsAreThrown=true --vm.Dgraal.TruffleIterativePartialEscape=true --experimental-options --basic-ops-inline=false pe/pe.rb"]
+        lines.push "RUN " + setup_env["ruby #{config} --vm.Dgraal.TruffleCompilationExceptionsAreThrown=true --vm.Dgraal.TruffleIterativePartialEscape=true pe/pe.rb"]
       end
     end