Christian Trott

Results 519 comments of Christian Trott

otherwise looks good.

This was not a bug or mistake: it was a design choice to provide comprehensible error messages. The issue is that the trait `is_constructible_v` is misleading since it only takes...

I have thought more about this: and I am not sure I want to "fix" this, this way. The basic issue is that `is_constructible_v` does not take into account *mandates*...

Because the code should have used `a_sub.layout()` corrected.

Also accessing stride(2) on a rank-2 view is kinda UB lol

Clang CUDA doesn't work: ``` /var/jenkins/workspace/Kokkos_PR-7013/core/src/Cuda/Kokkos_Cuda_abort.hpp:31:12: error: an attribute list cannot appear here [clang-diagnostic-error] __device__ [[noreturn]] void __assertfail(const void *message, const void *file, ```

They removed -cuda from their documented command line options. It used to be there now it's not.

Basically we are in somewhat of a bind here. In order to move across memory-space boundaries we may have no option other than to men-cpy but trivially-copyable is too harsh...

What is the actual GPU you are trying to use? I am running on an MI50 and it seems all to work with your command.

(Note I was using Kokkos 4.3.01 and ROCM 6.0 not 6.1 which I didn't have on my machine)