-
-
Notifications
You must be signed in to change notification settings - Fork 925
Permalink
Choose a base ref
{{ refName }}
default
Choose a head ref
{{ refName }}
default
Comparing changes
Choose two branches to see what’s changed or to start a new pull request.
If you need to, you can also or
learn more about diff comparisons.
Open a pull request
Create a new pull request by comparing changes across two branches. If you need to, you can also .
Learn more about diff comparisons here.
base repository: jruby/jruby
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: ff2143c7ec90
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
...
head repository: jruby/jruby
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: a3a8783b5bb9
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
- 9 commits
- 10 files changed
- 1 contributor
Commits on Dec 17, 2015
-
Configuration menu - View commit details
-
Copy full SHA for 9d4d585 - Browse repository at this point
Copy the full SHA 9d4d585View commit details -
Configuration menu - View commit details
-
Copy full SHA for 1a8b8d0 - Browse repository at this point
Copy the full SHA 1a8b8d0View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5b15e64 - Browse repository at this point
Copy the full SHA 5b15e64View commit details -
Configuration menu - View commit details
-
Copy full SHA for 79b0cb8 - Browse repository at this point
Copy the full SHA 79b0cb8View commit details -
Configuration menu - View commit details
-
Copy full SHA for a71479a - Browse repository at this point
Copy the full SHA a71479aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 3200624 - Browse repository at this point
Copy the full SHA 3200624View commit details -
Configuration menu - View commit details
-
Copy full SHA for cc74969 - Browse repository at this point
Copy the full SHA cc74969View commit details -
Change "has protocol" to "can call direct" and fix all paths.
IR protocol is not the only requirement for a direct call, and the updating of both "has ir protocol" flag and the directly- callable path was racy in both interpreter and JIT. In addition, full builds and JIT that happen at threshold=0 in sync with execution were sometimes continuing down the non-direct path using the direct logic. This resulted in some call protocols running twice, potentially messing up state for the direct logic. The new flag is "canCallDirect", indicating that a transition to the next tier has happened. Paths through normal non-direct logic that might trigger a tier upgrade now re-check to see if that has happened, re-branching back to the direct path.
Configuration menu - View commit details
-
Copy full SHA for 2039454 - Browse repository at this point
Copy the full SHA 2039454View commit details -
CompiledIRBlockBody should never have non-direct calls.
Leaving this logic in place, commented, for the moment.
Configuration menu - View commit details
-
Copy full SHA for a3a8783 - Browse repository at this point
Copy the full SHA a3a8783View commit details
Loading
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff ff2143c7ec90...a3a8783b5bb9