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

WIP: const wrapper for __ldg #18

Closed
wants to merge 3 commits into from
Closed

WIP: const wrapper for __ldg #18

wants to merge 3 commits into from

Conversation

vchuravy
Copy link
Collaborator

@vchuravy vchuravy commented Feb 5, 2018

based on tb/ldg for CUDAnative

@vchuravy
Copy link
Collaborator Author

vchuravy commented Feb 6, 2018

Doesn't pass test yet since the promotion of different BroadcastStyle is losing track of the shape...

@maleadt
Copy link
Collaborator

maleadt commented Feb 7, 2018

It'll probably be easier to have Const be a function returning a CuArray with some constness/cache type param set, instead of actually wrapping the array object.

@maleadt
Copy link
Collaborator

maleadt commented Feb 14, 2018

Most of this is now in JuliaGPU/CUDAnative.jl#158, with the remaining PoC changes in https://github.com/jrevels/MixedModeBroadcastAD.jl/compare/tb/cached_load.

@maleadt maleadt changed the title add const wrapper for __ldg WIP: const wrapper for __ldg Feb 14, 2018
@jrevels jrevels closed this Feb 14, 2018
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.

3 participants