[clang] Deflake the TimeProfile support tests (#138613)
These tests have been flaky since they were merged into the
AllClangUnitTests binary, but the flakiness is inherent to the nature of
timer-based tests.
diff --git a/clang/unittests/Support/TimeProfilerTest.cpp b/clang/unittests/Support/TimeProfilerTest.cpp
index f8053f2d..7698742 100644
--- a/clang/unittests/Support/TimeProfilerTest.cpp
+++ b/clang/unittests/Support/TimeProfilerTest.cpp
@@ -155,6 +155,16 @@
bool InsideCurrentEvent =
Event.TimestampBegin >= EventStack.top()->TimestampBegin &&
Event.TimestampEnd <= EventStack.top()->TimestampEnd;
+
+ // Presumably due to timer rounding, PerformPendingInstantiations often
+ // appear to be within the timer interval of the immediately previous
+ // event group. We always know these events occur at level 1, not level 2,
+ // in our tests, so pop an event in that case.
+ if (InsideCurrentEvent && Event.Name == "PerformPendingInstantiations" &&
+ EventStack.size() == 2) {
+ InsideCurrentEvent = false;
+ }
+
if (!InsideCurrentEvent)
EventStack.pop();
else
