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

interpreter: fix pref regression from #29795 #29873

Merged
merged 1 commit into from
Nov 2, 2018
Merged

interpreter: fix pref regression from #29795 #29873

merged 1 commit into from
Nov 2, 2018

Conversation

vtjnash
Copy link
Member

@vtjnash vtjnash commented Oct 31, 2018

fix #29872

Silly me used the wrong variable / copied the code from the wrong place, and accidentally disabled inference of the wrong subgraph.

@vtjnash
Copy link
Member Author

vtjnash commented Oct 31, 2018

@nanosoldier runbenchmarks(ALL, vs=":jn/29872-compare") # current head of master with PR 29795 dropped, for comparison

@nanosoldier
Copy link
Collaborator

Your benchmark job has completed - possible performance regressions were detected. A full report can be found here. cc @ararslan

@vtjnash
Copy link
Member Author

vtjnash commented Nov 1, 2018

Not quite the right comparison (since it used the merge commit on master), but does conform there are no regressions with this fixed.

@vtjnash vtjnash merged commit e1fbd0a into master Nov 2, 2018
@vtjnash vtjnash deleted the jn/29872 branch November 2, 2018 15:04
tkf pushed a commit to tkf/julia that referenced this pull request Nov 21, 2018
KristofferC pushed a commit that referenced this pull request Nov 28, 2018
KristofferC pushed a commit that referenced this pull request Dec 12, 2018
KristofferC pushed a commit that referenced this pull request Feb 11, 2019
KristofferC pushed a commit that referenced this pull request Feb 20, 2020
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.

Performance regression on master in views
2 participants