Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[fix] jit.excludes for block using class (and method) name #5893

Merged
merged 2 commits into from Sep 27, 2019

Conversation

@kares
Copy link
Member

kares commented Sep 26, 2019

wasn't working since the (internal) closure id was returned
since block tasks will know the impl module - return that
and use the common logic for extracting owner name

wasn't working since the (internal) closure id was returned
since block tasks will know the impl module - return that
and use the common logic for extracting owner name from it
@kares kares added the jit label Sep 26, 2019
@kares kares added this to the JRuby 9.2.9.0 milestone Sep 26, 2019
no longer exclude using internal id - limited usefulness
@kares kares changed the title [fix] jit.excludes for block using class name [fix] jit.excludes for block using class (and method) name Sep 27, 2019
@kares kares merged commit 53252c3 into master Sep 27, 2019
4 of 6 checks passed
4 of 6 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
jruby.jruby Build #20190927.1 succeeded
Details
jruby.jruby (Job linux) Job linux succeeded
Details
jruby.jruby (Job mac) Job mac succeeded
Details
jruby.jruby (Job windows) Job windows succeeded
Details
@kares kares deleted the jit-exclude-block-working branch Sep 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.