[SYCL] queue use perfect forwarding

[lslusarczyk]

Use perfect forwarding in queue inline implementations. In the same place use event move if possible. For sake of performance.

As the change is small one. Most benchmarks show no performance improvement, but some have improved, e.g. (see difference between green lines above and two last points of red line marked in a circle)

Not using perfect forwarding in the API taking universal references is a wrong pattern. I want it to be fixed, in order to

• do not to wonder in the future whether if we are slow because of this or because of something else (quickly jump to "something else" branch)
• promote correct API for future calls, as people usually look around how things are written and write code in a similar way
• improve perf a bit, as it indeed improves it

[vinser52]

You are changing a public API. Is it really necessary?

@gmlueck Could you please comment if spec gives us freedom to accept Properties by universal reference?

[vinser52]

Why haven't you updated the else path? And what about the subsequent calls down to the stack? If you forward here, but all next calls in the stack still accept by value, then this optimization makes little sense.

[vinser52]

Same question here as above

[vinser52]

The right fix here should be to use sycl::span in the queue_impl::memset to accept DepEvents to avoid temporary std::vector creation.