Do not overwrite loaded tools in cluster benchmarks when verbose

[aprokop]

The output looks like this:

        *** Warning: Kokkos profiling tools are already active, ignoring the the --verbose argument. ***

[dalg24]

If we go that route, then technically the ArborXBenchmark::{push,pop}_region symbols do not need to be exposed (we could remove them from our headers and define them in an anonymous namespace).
Thoughts on making it a try_set_timer_hooks() -> bool and repeat the warning (we might want to use raw string literals)

[aprokop]

If we go that route, then technically the ArborXBenchmark::{push,pop}_region symbols do not need to be exposed (we could remove them from our headers and define them in an anonymous namespace).

One reason I did not eliminate those as I didn't want to populate the global namespace. I suppose I can put them in the anonymous namespace instead.

Thoughts on making it a try_set_timer_hooks() -> bool and repeat the warning (we might want to use raw string literals)

We can. try would not be the right name, as the current behavior always overwrites the existing hooks. Maybe set_timer_hooks() -> bool?

[aprokop]

@dalg24 I updated the code to follow your suggestions.