-
Notifications
You must be signed in to change notification settings - Fork 221
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
Mark cuarray as noalias #2395
Mark cuarray as noalias #2395
Conversation
I have no idea what |
Partially, so I'm hoping to make something which generates an llvm noalias on the result of a call (which permits optimizations -- which are particularly helpul for broadcasting). Specifically, here I'd like to provide the semantics a fresh allocation of a CuArray doesn't alias any other memory in the system (but of course a view taken of that variable may alias the original variable). |
And yeah I'm looking to add it (EnzymeAD/Enzyme.jl#1467) as a companion to this PR to be able to propagate this information. In contrast jl_alloc_array_1d we are able to add LLVM-level noalias this ourselves, but to add noalias for codes out of Julia base requires an interface function was my thought. I'd love to alternatively read this in thorugh Julia effects, but I don't know if that has something helpful here. |
@vchuravy with this passing, mind approving/merging? |
It did not:
Changing Can you please verify the Enzyme tests pass locally before marking a PR ready for review? We don't have the capacity for CI-driven development. Also please include |
This reverts commit beff592. [skip ci]
Oh rip sorry I saw the green on CI and assumed it was fine, I guess it just didn't run |
@vchuravy @maleadt I don't suppose you know a better way to get this info from julia effects a priori.
Has corresponding EnzymeCore update.
Note this is part of an ongoing effort for Enzyme to successfully differentiate (in fwd mode to start with) Cuda broadcasts/etc