[Lldb-commits] [lldb] 9842726 - this test is flakey on arm in linux because I got too clever (#89267)

[via lldb-commits]

Author: Fred Grim
Date: 2024-04-18T10:29:39-07:00
New Revision: 98427264cab558d285c7222f1f25b904124c6b93

URL: https://github.com/llvm/llvm-project/commit/98427264cab558d285c7222f1f25b904124c6b93
DIFF: https://github.com/llvm/llvm-project/commit/98427264cab558d285c7222f1f25b904124c6b93.diff

LOG: this test is flakey on arm in linux because I got too clever (#89267)

the assembly jazz doesn't work on arm. oops

Added: 
    

Modified: 
    lldb/unittests/Host/linux/HostTest.cpp

Removed: 
    


################################################################################
diff  --git a/lldb/unittests/Host/linux/HostTest.cpp b/lldb/unittests/Host/linux/HostTest.cpp
index 046736dce18d7a..733909902474d7 100644
--- a/lldb/unittests/Host/linux/HostTest.cpp
+++ b/lldb/unittests/Host/linux/HostTest.cpp
@@ -69,13 +69,17 @@ TEST_F(HostTest, GetProcessInfo) {
   EXPECT_EQ(HostInfo::GetArchitecture(HostInfo::eArchKindDefault),
             Info.GetArchitecture());
   // Test timings
+  /*
+   * This is flaky in the buildbots on all archs
   ASSERT_TRUE(Host::GetProcessInfo(getpid(), Info));
   ProcessInstanceInfo::timespec user_time = Info.GetUserTime();
+  static volatile unsigned u = 0;
   for (unsigned i = 0; i < 10'000'000; i++) {
-    __asm__ __volatile__("" : "+g"(i) : :);
+    u = i;
   }
   ASSERT_TRUE(Host::GetProcessInfo(getpid(), Info));
   ProcessInstanceInfo::timespec next_user_time = Info.GetUserTime();
   ASSERT_TRUE(user_time.tv_sec < next_user_time.tv_sec ||
               user_time.tv_usec < next_user_time.tv_usec);
+  */
 }