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

Assembly implementation of EncodeInt for amd64 #7

Merged
merged 14 commits into from May 21, 2018
Merged

Conversation

@mmcloughlin
Copy link
Owner

@mmcloughlin mmcloughlin commented May 20, 2018

Provides an assembly implementation of EncodeInt for amd64 architecture.

benchmark                old ns/op     new ns/op     delta
BenchmarkEncode-4        54.4          43.3          -20.40%
BenchmarkEncodeInt-4     10.8          2.50          -76.85%
@coveralls
Copy link

@coveralls coveralls commented May 20, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling 783a341 on mm-encodeint-asm into 538d003 on master.

@mmcloughlin mmcloughlin merged commit ed07850 into master May 21, 2018
7 checks passed
7 checks passed
ci/circleci Your tests passed on CircleCI!
Details
codebeat no reportable quality changes
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 100.0%
Details
@mmcloughlin mmcloughlin deleted the mm-encodeint-asm branch Oct 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants