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

perf regression in @pure changes #13735

Closed
JeffBezanson opened this issue Oct 23, 2015 · 1 comment
Closed

perf regression in @pure changes #13735

JeffBezanson opened this issue Oct 23, 2015 · 1 comment
Labels
performance Must go faster regression Regression in behavior compared to a previous version

Comments

@JeffBezanson
Copy link
Member

Running @timholy 's test case from #13350:

commit 7dcc01b:

Without parentheses:
  0.567878 seconds (16.01 M allocations: 488.552 MB, 5.03% gc time)
With parentheses:
  0.421889 seconds (11 allocations: 480 bytes)

commit 8ce7a02

Without parentheses:
  2.149215 seconds (70.01 M allocations: 1.654 GB, 5.22% gc time)
With parentheses:
  2.251477 seconds (74.00 M allocations: 1.773 GB, 5.45% gc time)

cc @vtjnash

@vtjnash
Copy link
Member

vtjnash commented Feb 27, 2016

duplicate of #14294

@vtjnash vtjnash closed this as completed Feb 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Must go faster regression Regression in behavior compared to a previous version
Projects
None yet
Development

No branches or pull requests

2 participants