oneDPL icon indicating copy to clipboard operation
oneDPL copied to clipboard

Published 20 hours ago • verified publisher icon oneapi-src

Redefine `timing_t` as `std::chrono::milliseconds`

Open SergeyKopienko opened this issue 9 months ago • 1 comments

In this PR we redefine timing_t as std::chrono::milliseconds and remove count() function call at all, so we always operates by value with physical sense.

Of course it's not mandatory improvement, we are able to apply it later. @AnuyaWelling2801, up to you.

SergeyKopienko avatar Apr 30 '24 15:04 SergeyKopienko

As far as I understand, this change might affect the public API of dynamic selection, namely, the reporting requirements of the auto-tune policy. I suggest to take it for discussion, but not apply right now.

akukanov avatar Apr 30 '24 16:04 akukanov

@AnuyaWelling2801, what do you think, will you apply this PR or I may close it?

SergeyKopienko avatar Jun 10 '24 14:06 SergeyKopienko