Damien L-G
Damien L-G
> Is it possible to remove the conversion Yes it looks like an oversight > I'm happy to open a PR if such change is welcomed. Please do
This is false advertising. You are doing more than moving things around. You did changes to the class. Please provide a description.
Did you check whether it actually helps? First thing that comes to mind is `std::nextafterf()` is not portable and it the function is currently not provided in [``](https://github.com/kokkos/kokkos/blob/d8969090864e2dbc63e16e688db8bc8699910cac/core/src/Kokkos_MathematicalFunctions.hpp#L263-L354) but if...
It means that if it turns out it is useful we will need workarounds in Arborx until we require Kokkos 3.7
Just found out about [``](https://github.com/kokkos/kokkos/blob/develop/core/src/Kokkos_Profiling_ProfileSection.hpp) Obviously we will not introduce `` then but we should still consider using section profiling hooks instead of the timers we currently have.
And `Kokkos::Profiling::ProfilingSection` being copyable and assignable looks like a bug.
> > And `Kokkos::Profiling::ProfilingSection` being copyable and assignable looks like a bug. > > I agree. Ar you fixing it? Planning on it.
> This does not compile. Can you elaborate. It should compile now that we require Kokkos 3.4
This is true if the tool wait for Kokkos::finalize to print
Retest this please