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

Remove unneeded pyston workarounds + fix misc cpython test errors #944

Merged
merged 2 commits into from
Oct 15, 2015

Conversation

undingen
Copy link
Contributor

No description provided.

@undingen undingen added the wip label Sep 30, 2015
@undingen undingen force-pushed the workarounds branch 2 times, most recently from 1568ff5 to 5ef005a Compare October 1, 2015 19:48
@undingen
Copy link
Contributor Author

undingen commented Oct 1, 2015

this sadly slowdowns pyxl by about 4%. Quick measurements show that about half of this is coming from adding weakref support to old style classes and the other part is coming from the collections.py change..

@kmod
Copy link
Collaborator

kmod commented Oct 2, 2015

ah too bad about the slowdown. maybe we can punt on those particular issues for now and get the rest of it in?

add list self assignment and list.index with non int args
add thread._count
copy over test_urllib2 from the correct python version
@undingen
Copy link
Contributor Author

ooh I forgot about this PR...
I rebased it and removed the collections.py change (means I left the pyston change to the file in) because it looked like this caused most of the perf regression.
This PR still regresses the perf slightly (I think this is mostly caused by making old style classes weakref able) but it's less than 1% so I think we should just merge it in.

kmod added a commit that referenced this pull request Oct 15, 2015
Remove unneeded pyston workarounds + fix misc cpython test errors
@kmod kmod merged commit 4818b5a into pyston:master Oct 15, 2015
@kmod
Copy link
Collaborator

kmod commented Oct 15, 2015

nice :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants