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

[PoC] select_k: Replace specialization by split header #1434

Closed

Conversation

ahendriksen
Copy link
Contributor

@ahendriksen ahendriksen commented Apr 19, 2023

This PR is a proof of concept of issue #1416 and is more approachable than PR #1415.

It splits the headers of matrix/select_k.cuh and removes the specializations.

This is necessary because the raft_runtime source files can sometimes be
in the same place as where the template instantiation files will be.
@ahendriksen ahendriksen added enhancement New feature or request non-breaking Non-breaking change Build Time Improvement 5 - DO NOT MERGE Hold off on merging; see PR for details labels Apr 19, 2023
@ahendriksen
Copy link
Contributor Author

@tfeher and @cjnolet for visibility. This PR should be easier to review. It performs splitting of headers only for select_k.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - DO NOT MERGE Hold off on merging; see PR for details Build Time Improvement CMake cpp enhancement New feature or request non-breaking Non-breaking change
Projects
Development

Successfully merging this pull request may close these issues.

1 participant