Skip to content

Use an LLVM intrinsic instead of libc_::memmove. #4413

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
Closed
wting opened this issue Jan 10, 2013 · 2 comments
Closed

Use an LLVM intrinsic instead of libc_::memmove. #4413

wting opened this issue Jan 10, 2013 · 2 comments
Labels
A-codegen Area: Code generation I-slow Issue: Problems and improvements with respect to performance of generated code.
Milestone
0.7

Comments

@wting
Copy link
Contributor

wting commented Jan 10, 2013

As @brson mentioned in IRC, ptr::memcpy uses libc_::memmove which can be optimized with an LLVM intrinsic.
@brson
Copy link
Contributor

brson commented Jan 11, 2013

I did a quick test and this does make nearly a 4x difference when doing small memcpy's in a tight loop.

@brson brson mentioned this issue Jan 14, 2013
Closed
@pnkfelix pnkfelix mentioned this issue Mar 22, 2013
Closed
@graydon
Copy link
Contributor

graydon commented Apr 29, 2013

fixed in 0c1d9be

@graydon graydon closed this as completed Apr 29, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-codegen Area: Code generation I-slow Issue: Problems and improvements with respect to performance of generated code.
Projects
None yet
Milestone
0.7
Development

No branches or pull requests
3 participants
@graydon @brson @wting