-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Faster timers based on rdtsc instead of chrono #972
Comments
The code is essentially here Note, this was developed thanks to a lot of documentation by many people, including these and many others |
(Note: the overhead of std chrono clocks is in any case much less than we had in the past due to O/S issues #116) |
See this commit in PR #970: c863d69
|
One thing that may still be interesting is subtracting the overhead of the counters from the counter measurements. (See for instance https://www.strchr.com/performance_measurements_with_rdtsc#:~:text=Subtracting%20overhead,%2C%20100%20000%20clock%20cycles) In first approximation it should be enough to run say 1000 start/stop and time how long that takes, for instance. One could then report the estimated timer overhead (and/or subtract it from all individual and the overall timers) |
… branch prof0 (new timers madgraph5#972) and add copyright
… branch prof0 (new timers madgraph5#972) and add copyright
Faster timers based on rdtsc instead of chrono
I file this explcitly as an issue to make this clearer. This was quite a bit of work in the last few days.
I have completed the whole task
The changes are used in many PRs, but I would say that they can be merged from PR #962
The text was updated successfully, but these errors were encountered: