Michael Schellenberger Costa
Michael Schellenberger Costa
I am super excited about this thanks a lot for working on this 😻
> I agree with @miscco that I don't think this is functionality we can advertise until we have a way to test it. I believe properly testing this requires us...
> > use NV_IF_TARGET to test this within the current framework > > Sure, you'd just need to rewrite all of the current `atomic` tests that use `pre-sm-70` to instead...
/ok to test
@Artem-B could you verify that this fixes your issues?
The obvious wrinkle here is that those types wont find `cuda::std::begin` but AFAIK there is nothing we can do there and it is better than always breaking compilation
I also added a workaround for `swap` which suffers from the same issue
@Artem-B I have pushed backported fixes here https://github.com/miscco/cccl/tree/backport_adl_fixes
I love how fixing ambiguities with begin and friend brings up errors like: >thrust/system/cuda/detail/internal/copy_cross_system.h(94): error #694: reinterpret_cast cannot cast away const or other type qualifiers