Damien L-G comments

Results 491 comments of


                                            Damien L-G

Detect memory leaks in the ci

Couldn't we write suppressions for those reports so we are able to detect if we accidentally reintroduce leaks on our end?

Allow building a C++20 module for Kokkos

Any reason not to close this PR?

Build warnings with missing braces

What compiler(s)?

Build warnings with missing braces

https://godbolt.org/z/fjodW7oh3

Build warnings with missing braces

I am a bit surprised, I wonder if it means it doesn't select the constructor from a ~~non-const~~ const ref to a array as we expected it would.

[4.7.02] bulk cherry-pick: PRs #8283 #8386 #8466 #8320 #8562 #8615

``` /builds/kokkos/kokkos/core/unit_test/TestCompilerMacros.cpp:64:5: error: loop not vectorized: the optimizer was unable to perform the requested transformation; the transformation might be disabled or specified as part of an unsupported transformation ordering [-Werror,-Wpass-failed=transform-warning]...

Moved test testing AtomicRef equivalence with atomic View into a new file due to better fit

@janciesko Please set the DCO to pass

CMake multliple exports when using FetchContent

Please provide a reproducer. I tried ```diff diff --git a/CMakeLists.txt b/CMakeLists.txt index d1826e4b..16aab214 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -9,8 +9,12 @@ endif() # use gnu standard install directories include(GNUInstallDirs)...

C++20 module for Kokkos std algorithms and basic setup

Retest this please

Make Impl::HostMirror:Space type more consistent

Please resolve conflicts