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

Rewrite search8.cc and search16.cc using intrinsics for portability #44

Closed
torognes opened this issue Jan 1, 2015 · 7 comments
Closed
Assignees
Milestone

Comments

@torognes
Copy link
Owner

torognes commented Jan 1, 2015

Registers xmm8-15 and r8-15 are unavailble on 32-bit platforms. Rewrite with intrinsics to avoid problem.

@torognes torognes self-assigned this Jan 1, 2015
@torognes
Copy link
Owner Author

@torognes
Copy link
Owner Author

torognes commented Jan 14, 2019

I do think we should spend time on support for 32-bit systems, they are too old.

@frederic-mahe
Copy link
Collaborator

Yes, I was checking https://store.steampowered.com/hwsurvey/directx/ and less than 2% of steam users are on a non-64bit system. If we assume swarm users follow the same trend, then very few people will actually use a 32-bit version of swarm. Besides, datasets larger than 4 GB (the product of one Illumina MiSeq run) are very frequent now.

@torognes
Copy link
Owner Author

Rewrite using intrinsics should perhaps be done anyway. Especially in order to port to other systems as described in issue #107. Could be based on the vsearch code.

@frederic-mahe frederic-mahe changed the title Rewrite search8.cc and search16.cc using intrinsics to allow compilation on 32-bit platforms Rewrite search8.cc and search16.cc using intrinsics to facilitate porting Jan 14, 2019
@frederic-mahe
Copy link
Collaborator

Let's reopen the issue under a new title.

@frederic-mahe frederic-mahe reopened this Jan 14, 2019
@frederic-mahe frederic-mahe added this to the swarm 3.0 milestone Jan 17, 2019
@frederic-mahe frederic-mahe changed the title Rewrite search8.cc and search16.cc using intrinsics to facilitate porting Rewrite search8.cc and search16.cc using intrinsics for portability Jan 21, 2019
@torognes
Copy link
Owner Author

In the zobrist branch, the SIMD assembly code in search8.cc and search16.cc is now replaced by intrinsics for x86_64. Ready for porting to other architectures (#107).

@frederic-mahe
Copy link
Collaborator

Great! Let's close the issue then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants