Skip to content

MINOR: cleanup setting callcounts for JIT#4750

Merged
kares merged 1 commit intojruby:masterfrom
original-brownbear:cleanup-redundant-jit-code
Aug 26, 2017
Merged

MINOR: cleanup setting callcounts for JIT#4750
kares merged 1 commit intojruby:masterfrom
original-brownbear:cleanup-redundant-jit-code

Conversation

@original-brownbear
Copy link
Contributor

Just some trivial spots I found recently:

  • Some of those promoteToFullBuild (and analogously named) methods can be made private since they aren't used in child classes
  • Removed dead import import org.jruby.EvalType;
  • Fixed logger for MixedModeIRBlockBody (it was using the wrong class to bind to)
  • Removed two redundant sets of the callcount to -1 (callCount = -1; is not necessary since context.runtime.getJITCompiler().buildThresholdReached(context, this); will do this:
    public void buildThresholdReached(ThreadContext context, final Compilable method) {
        final RubyInstanceConfig config = context.runtime.getInstanceConfig();

        // Disable any other jit tasks from entering queue
        method.setCallCount(-1);

=> sets the -1 here either way.

@kares kares merged commit c60643a into jruby:master Aug 26, 2017
@original-brownbear original-brownbear deleted the cleanup-redundant-jit-code branch August 26, 2017 13:15
@kares kares added this to the JRuby 9.2.0.0 milestone Aug 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants