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

Make head and tail calls more memory efficient #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Make head and tail calls more memory efficient #1

wants to merge 1 commit into from

Conversation

cgray
Copy link

@cgray cgray commented Mar 17, 2016

Used array_slice instead of array_pop/array_shift to keep a copy of the source array from being created

Used array_slice as opposed to using array_pop/array_shift to keep a copy of the source array from being created
shadowhand added a commit that referenced this pull request Mar 21, 2016
In order to verify changes to functions that might affect performance.

Refs #1
@shadowhand shadowhand mentioned this pull request Mar 21, 2016
shadowhand added a commit that referenced this pull request Mar 21, 2016
In order to verify changes to functions that might affect performance.

Also runs benchmarks during builds for easy comparison.

Refs #1
@shadowhand
Copy link
Contributor

@cgray can you rebase this so that we can compare before/after performance (see #2)?

@ameech
Copy link
Contributor

ameech commented Apr 1, 2016

@cgray any update on this?

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.

3 participants