Damien L-G comments

Results 492 comments of


                                            Damien L-G

Simplify brute force example

I think we should keep an example that shows how both indexes can be used.

Add support for CUDA unified memory architectures i.e. Grace Hopper

Please rebase or create a new PR to tackle the "unified memory" part

Only copy data in UnorderedMap deep_copy

Deferring to Christian and Conrad.

Add `likely` and `unlikely` attribute from C++20 to ref counting in views

@rgayatri23 Make sure you add this to the change log

SYCL: Improve team_reduce implementation

Retest this please

SYCL: Improve team_reduce implementation

(SYCL build passed)

Avoid gcc-13 warning about out-of-range access for subviews

> As far as I can tell these warnings are spurious. This pull request proposes two different solutions to suppress this warning: > > 1. Guard the offending code with...

Avoid gcc-13 warning about out-of-range access for subviews

> I just enabled compiler warnings and tests and the compiler reports (using Homebrew on MacOS X) Did you try to reproduce on the compiler explorer with Kokkos 4.1?

Avoid gcc-13 warning about out-of-range access for subviews

Also do we understand why we can't reproduce it on the compiler explorer nor in the CI build? (not saying we are not interested in fixing, just trying to understand...

Avoid gcc-13 warning about out-of-range access for subviews

> Did you consider changing the arguments to `std::integral`? I think it would be more readable It does not really work because we need to do +1