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

Atomix Extension #2549

Closed
wants to merge 5 commits into from
Closed

Atomix Extension #2549

wants to merge 5 commits into from

Conversation

christiangnrd
Copy link
Contributor

@christiangnrd christiangnrd commented Nov 7, 2024

Proof of concept for having the Atomix.jl backend as a CUDA.jl extension instead of a standalone package that lives in Atomix.jl.

Removes the need to add and import AtomixCUDA which was created before package extensions were a thing.

Added the tests to the main pipeline since Atomix is already a dependency and there are 3 tests so extra runtime should be minimal.

See JuliaConcurrent/Atomix.jl#39 (comment)

@maleadt
Copy link
Member

maleadt commented Nov 8, 2024

Thanks. I'm not convinced we want these extensions in the back-end repositories. I'd hope that the whole Atomix stack is a temporary thing (cc @vchuravy, hopefully we support plain LLVM atomics from Base at some point), so maybe it's better to keep these in the Atomix.jl repository?

@christiangnrd
Copy link
Contributor Author

Closing in favour of JuliaConcurrent/Atomix.jl#42.

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

Successfully merging this pull request may close these issues.

None yet

2 participants