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

Add a SIMD-enabled version of SideOffsets2D specialized for i32 #31

Merged
merged 4 commits into from Apr 30, 2014

Conversation

@brendanzab
Copy link
Member

brendanzab commented Apr 30, 2014

This will allow for future optimizations in Servo.

This will allow for future optimizations in Servo.
@metajack
Copy link
Contributor

metajack commented Apr 30, 2014

Don't r+ yet since @bjz plans to fix is_zero

@brendanzab
Copy link
Member Author

brendanzab commented Apr 30, 2014

Updated

The SIMD version is approximately comparable in speed to the naive version, but should reduce code size.
@brendanzab
Copy link
Member Author

brendanzab commented Apr 30, 2014

@pcwalton de-nitted

pcwalton added a commit that referenced this pull request Apr 30, 2014
Add a SIMD-enabled version of SideOffsets2D specialized for i32
@pcwalton pcwalton merged commit 38e0f82 into servo:master Apr 30, 2014
@brendanzab brendanzab deleted the brendanzab:simd branch Apr 30, 2014
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

3 participants
You can’t perform that action at this time.