Changes

SYCL: Cleanup device selection

SYCL CI: Avoid setvars.sh

Cleanup KOKKOS_CONFIGURE_CORE

Update GitHub actions ot use Node 20

[4.2.01]: changelog update  (#6656)

* [ci skip] [4.2.01]: changelog update

* [ci skip] [4.2.01]: changelog update

* Update CHANGELOG.md

Co-authored-by: Damien L-G <dalg24+github@gmail.com>

* Update CHANGELOG.md

Co-authored-by: Damien L-G <dalg24+github@gmail.com>

* Update CHANGELOG.md

Co-authored-by: Damien L-G <dalg24+github@gmail.com>

* Update CHANGELOG.md

Address review comments
- Add missing item for MSVC CUDA build fixes
- Drop item unnecessary for changelog

* Update changelog

* Update changelog

* [ci skip] Update patch version for 4.2.01

* Apply suggestions from code review

---------

Co-authored-by: Nathan Ellingwood <ndellin@sandia.gov>
Co-authored-by: Damien L-G <dalg24+github@gmail.com>

Clean up shift_{right, left}_team_impl (#6821)

* Clean up shift_right_team_impl

* Update shift_left_team_impl for consistency

Avoid detecting unwanted fences in the parallel_scan_no_fence test

Add test for current CTAD support with RangePolicy (#6803)

* Add test for current CTAD support with RangePolicy

Co-authored-by: Nevin Liber <nliber@anl.gov>

* Rework CTAD test to avoid "memeber <bla> was declared but never referenced" warnings with icpc 19

* Attempt to fix CI

* Attempt to eliminate maybe unused warning in icpc

* Disable CTAD tests for nvcc < 11.2
as compiler bugs prevent CTAD expressions inside decltype

---------

Co-authored-by: Nevin Liber <nliber@anl.gov>
Co-authored-by: Nevin ":-)" Liber <nliber+github@gmail.com>

multi-GPU support: Add test for all policies (#6782)

* Cuda multi-GPU support: Test with managed and unmanaged Views

* Move check for cuda_device

* Also test copying between devices

* Refactor using StreamsAndDevices

* Don't use shared_ptr

Implement sort_by_key (#6801)

* Implement sort_by_key

* Address review comments

* Make passed in view const ref

* Fix _via_sort for OpenMPTarget

Co-authored-by: Daniel Arndt <arndtd@ornl.gov>

* Rip out ROCThrust for now

* Few changes to address the comments

* Fix SYCL

* Fix SYCL implementation

---------

Co-authored-by: Daniel Arndt <arndtd@ornl.gov>

Fix scorecard workflow (#6831)

* Add Scorecrad

* another one

* Don't run on pull requests

* Add comments back in

Remove redundant RangePolicy constructor (#6841)

* Remove redundant RangePolicy constructor

* Disable test case for Cuda <11.2 completely

Bump Google Benchmark version v1.{6.2 -> 7.1} in CMake FetchContent

Remove variadic range policy constructor (#6845)

* Removed variadic ctor from RangePolicy, as well as extra
set(...) methods.

* In RangePolicy:
Deprecated set(ChunkSize)
In the ctor that takes a ChunkSize, set the chunk size via
set_chunk_size instead of via member initializers (to be more consistent
with other code)

* Removed superfluous "inline" from two of the RangePolicy constructors

Avoid unused variable warning in TestRangePolicyCTAD.cpp

Make Host backends be included before device backends

There was an inclusion order issue when trying to enable
unified memory, which was due to incomplete types.
Effectively the Host backends must be defined before
the device memory spaces for them to be marked accessible.

We may have the same issue if we try to mark HostSpace accessible
by device memory spaces - but I am not sure.

Add Grace CPU architecture

Add option to enable CUDA unified memory architectures

This is in support of Grace Hopper making, CudaSpace host accessible.
I also added an emulation mode to run on other CUDA architectures,
by making the cudaMalloc wrapper call cudaMallocManaged.

Kokkos_ENABLE_IMPL_CUDA_EMULATE_UNIFIED_MEMORY is the option

A new macro KOKKOS_ENABLE_IMPL_CUDA_UNIFIED_MEMORY will be defined
if both Grace and Hopper are enabled.

Fix tests for Unified Memory option being on

Apply suggestions from code review

Co-authored-by: Damien L-G <dalg24+github@gmail.com>
Co-authored-by: Daniel Arndt <arndtd@ornl.gov>

Print configuration for UNIFIED_MEMORY

SYCL: Cleanup device selection

SYCL CI: Avoid setvars.sh

Cleanup KOKKOS_CONFIGURE_CORE

Update GitHub actions ot use Node 20

[4.2.01]: changelog update  (#6656)

* [ci skip] [4.2.01]: changelog update

* [ci skip] [4.2.01]: changelog update

* Update CHANGELOG.md

Co-authored-by: Damien L-G <dalg24+github@gmail.com>

* Update CHANGELOG.md

Co-authored-by: Damien L-G <dalg24+github@gmail.com>

* Update CHANGELOG.md

Co-authored-by: Damien L-G <dalg24+github@gmail.com>

* Update CHANGELOG.md

Address review comments
- Add missing item for MSVC CUDA build fixes
- Drop item unnecessary for changelog

* Update changelog

* Update changelog

* [ci skip] Update patch version for 4.2.01

* Apply suggestions from code review

---------

Co-authored-by: Nathan Ellingwood <ndellin@sandia.gov>
Co-authored-by: Damien L-G <dalg24+github@gmail.com>

Clean up shift_{right, left}_team_impl (#6821)

* Clean up shift_right_team_impl

* Update shift_left_team_impl for consistency

Avoid detecting unwanted fences in the parallel_scan_no_fence test

Add test for current CTAD support with RangePolicy (#6803)

* Add test for current CTAD support with RangePolicy

Co-authored-by: Nevin Liber <nliber@anl.gov>

* Rework CTAD test to avoid "memeber <bla> was declared but never referenced" warnings with icpc 19

* Attempt to fix CI

* Attempt to eliminate maybe unused warning in icpc

* Disable CTAD tests for nvcc < 11.2
as compiler bugs prevent CTAD expressions inside decltype

---------

Co-authored-by: Nevin Liber <nliber@anl.gov>
Co-authored-by: Nevin ":-)" Liber <nliber+github@gmail.com>

multi-GPU support: Add test for all policies (#6782)

* Cuda multi-GPU support: Test with managed and unmanaged Views

* Move check for cuda_device

* Also test copying between devices

* Refactor using StreamsAndDevices

* Don't use shared_ptr

Implement sort_by_key (#6801)

* Implement sort_by_key

* Address review comments

* Make passed in view const ref

* Fix _via_sort for OpenMPTarget

Co-authored-by: Daniel Arndt <arndtd@ornl.gov>

* Rip out ROCThrust for now

* Few changes to address the comments

* Fix SYCL

* Fix SYCL implementation

---------

Co-authored-by: Daniel Arndt <arndtd@ornl.gov>

Fix scorecard workflow (#6831)

* Add Scorecrad

* another one

* Don't run on pull requests

* Add comments back in

Remove redundant RangePolicy constructor (#6841)

* Remove redundant RangePolicy constructor

* Disable test case for Cuda <11.2 completely

Bump Google Benchmark version v1.{6.2 -> 7.1} in CMake FetchContent

Remove variadic range policy constructor (#6845)

* Removed variadic ctor from RangePolicy, as well as extra
set(...) methods.

* In RangePolicy:
Deprecated set(ChunkSize)
In the ctor that takes a ChunkSize, set the chunk size via
set_chunk_size instead of via member initializers (to be more consistent
with other code)

* Removed superfluous "inline" from two of the RangePolicy constructors

Avoid unused variable warning in TestRangePolicyCTAD.cpp

Make Host backends be included before device backends

There was an inclusion order issue when trying to enable
unified memory, which was due to incomplete types.
Effectively the Host backends must be defined before
the device memory spaces for them to be marked accessible.

We may have the same issue if we try to mark HostSpace accessible
by device memory spaces - but I am not sure.

Add Grace CPU architecture

Add option to enable CUDA unified memory architectures

This is in support of Grace Hopper making, CudaSpace host accessible.
I also added an emulation mode to run on other CUDA architectures,
by making the cudaMalloc wrapper call cudaMallocManaged.

Kokkos_ENABLE_IMPL_CUDA_EMULATE_UNIFIED_MEMORY is the option

A new macro KOKKOS_ENABLE_IMPL_CUDA_UNIFIED_MEMORY will be defined
if both Grace and Hopper are enabled.

Fix tests for Unified Memory option being on

Apply suggestions from code review

Co-authored-by: Damien L-G <dalg24+github@gmail.com>
Co-authored-by: Daniel Arndt <arndtd@ornl.gov>

Print configuration for UNIFIED_MEMORY