Skip to content

some fixes and cleanups #712

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Jul 17, 2015
Merged

some fixes and cleanups #712

merged 6 commits into from
Jul 17, 2015

Conversation

kmod
Copy link
Collaborator

@kmod kmod commented Jul 17, 2015

and a couple new benchmarks. we still have an annoying amount of build-to-build performance variability (there aren't any performance improvements here):

       django_template.py             3.6s (2)             3.6s (2)  -0.4%
            pyxl_bench.py             3.5s (2)             3.4s (2)  -1.1%
sqlalchemy_imperative2.py             4.3s (2)             4.4s (2)  +0.3%
                  geomean                 3.8s                 3.8s  -0.4%

kmod added 6 commits July 16, 2015 23:48
@kmod
Add a simplified version of django_template
0130b77 
This only does the lexing portion of the process.

Further cut that down into a re.split ubench
@kmod
Makefile: make it easier to run custom cpythons
7ad32c2 
And some other small cleanups
@kmod
add cpython regex tests
069a701
@kmod
int() continues to be tricky
1505cc6 
int(str) and int(float) don't always return ints (cant return longs, doh).
If we call int() on a subclass of int, we should call its __int__ method in
case the subclass overrode it.
@kmod
fix: need to check tp_getattro as well
7c4c909
@kmod
cmake dependency tracking workaround
5385cf7
kmod added a commit that referenced this pull request Jul 17, 2015
@kmod
Merge pull request #712 from kmod/perf2
baad890 
some fixes and cleanups
@kmod kmod merged commit baad890 into pyston:master Jul 17, 2015
@kmod kmod deleted the perf2 branch July 17, 2015 01:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kmod