Split Ruby code to match other languages

[eregon]

This is needed to integrate with polybench, cc @boris-spas.

Notably this makes it possible to run a benchmark without needing to run harness.rb, like:

require_relative 'run'

RUN = Run.new('Bounce')
RUN.num_iterations = 1
RUN.inner_iterations = 1

5.times do
  RUN.run_benchmark
end

[eregon]

@smarr In CD, RuboCop is suggesting to change:

    # Y is guaranteed to be non-null at this point.
    if y.left
      x = y.left
    else
      x = y.right
    end

to

    # Y is guaranteed to be non-null at this point.
    x = y.left || y.right

I do that, or I add an ignore? (it is semantically equivalent, it avoids calling y.left twice)

[smarr]

    # Y is guaranteed to be non-null at this point.
    x = y.left || y.right

I do that, or I add an ignore? (it is semantically equivalent, it avoids calling y.left twice)

Hm, my feeling is that we would want a compiler to figure this out. So, I'd tent to disabling the relevant rubocop rule.

[smarr]

Just for confirmation, this is just moving code around right?

@boris-spas mentioned in Potsdam that he indeed wanted to do that to replicate the same code structure as in the other languages.

So, yeah, that's great.

Thanks

[eregon]

Just for confirmation, this is just moving code around right?

Mostly, yes. If you review commit-by-commit it's clearer what happens and what are the actual changes.

[eregon]

@boris-spas will double-check it works for polybench and then it should be ready to merged (that's why it's a draft PR currently).

[boris-spas]

LGTM! Polybench happily works with

require(PATH_TO_AWFY_BENCHMARKS_RUBY + "/run")

RUN = Run.new('Bounce')
RUN.num_iterations = 1
RUN.inner_iterations = 1

def run()
    RUN.run_benchmark
end

[eregon]

Ready to be merged then

[smarr]

Thank you!