libcudacxx icon indicating copy to clipboard operation
libcudacxx copied to clipboard

Make `cuda::aligned_size_t` available in a more appropriate header than `<cuda/barrier>`

Open jrhemstad opened this issue 1 year ago • 3 comments

The cuda::aligned_size_t<N> type is currently defined in <cuda/std/barrier>.

This requires me to include <cuda/std/barrier> any time I wish to use cuda::aligned_size_t. This is especially problematic as merely including cuda/barrier prevents compiling with <sm_70.

cuda::aligned_size_t is useful independent of its usage with barrier.

I want to be able to use cuda::aligned_size_t without including <cuda(/std)/barrier>.

I'm thinking we could add it to <cuda/cstddef>?

jrhemstad avatar Jul 17 '22 18:07 jrhemstad

Could this potentially land in v1.9.0?

sleeepyjack avatar Aug 11 '22 21:08 sleeepyjack

You'd have to convince @wmaxey.

It is a relatively simple change, so I'd be fine with sneaking it in if we can, but Wesley calls the shots 'round here.

jrhemstad avatar Aug 12 '22 00:08 jrhemstad

@wmaxey could this be a candidate for v1.9.0?

Also related: #163

sleeepyjack avatar Aug 29 '22 17:08 sleeepyjack