tree 6be3da93baf2ef1390c972ead8dfd6bd544a08d8
parent f0f7e1089b23b5d9b2f9f648602743d8472fa1cd
author Dean Michael Berris <dberris@google.com> 1541396602 +0000
committer Copybara-Service <copybara-worker@google.com> 1598763034 -0700

[XRay] Update TSC math to handle wraparound

Summary:
Prior to this change, we can run into situations where the TSC we're
getting when exiting a function is less than the TSC we got when
entering it. This would sometimes cause the counter for cumulative call
times overflow, which was erroneously also being stored as a signed
64-bit integer.

This change addresses both these issues while adding provisions for
tracking CPU migrations. We do this because moving from one CPU to
another doesn't guarantee that the timestamp counter for some
architectures aren't guaranteed to be synchronised. For the moment, we
leave the provisions there until we can update the data format to
include the counting of CPU migrations we can catch.

We update the necessary tests as well, ensuring that our expectations
for the cycle accounting to be met in case of counter wraparound.

Reviewers: mboerger

Subscribers: llvm-commits

Differential Revision: https://reviews.llvm.org/D54088

llvm-svn: 346116
GitOrigin-RevId: 1e255e7a7b3752ef7f786506e695f0f7b58e2339