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

[FEA] Support linking multiple versions of cuco in the same project #189

Open
sleeepyjack opened this issue Jul 19, 2022 · 1 comment
Open
Labels
P1: Should have Necessary but not critical type: feature request New feature request

Comments

@sleeepyjack
Copy link
Collaborator

CUB and Thrust both allow for custom namespaces via e.g. CUB_NS_PREFIX and CUB_NS_POSTFIX, such that multiple shared libraries can each utilize their own copy of it (and thus different versions can safely coexist). (see NVIDIA/thrust#1401 )

cuco should also implement this feature.

@sleeepyjack sleeepyjack added the type: feature request New feature request label Jul 19, 2022
@PointKernel PointKernel added the P1: Should have Necessary but not critical label Jul 19, 2022
@sleeepyjack
Copy link
Collaborator Author

@allisonvacanti suggested to utilize versioned inline namespaces to solve this problem:

namespace cuco {
inline namespace CUCO_VERSION {
  void foo();
} /* inline namespace CUCO_VERSION */
} /* namespace cuco */

This option requires #190 to be implemented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1: Should have Necessary but not critical type: feature request New feature request
Projects
None yet
Development

No branches or pull requests

2 participants