[compiler-rt] [scudo] Fix expectation in ScudoTimingTest.VerifyMax (PR #106062)

via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 26 04:52:45 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: Fabio D'Urso (fabio-d)

<details>
<summary>Changes</summary>



---
Full diff: https://github.com/llvm/llvm-project/pull/106062.diff


1 Files Affected:

- (modified) compiler-rt/lib/scudo/standalone/tests/timing_test.cpp (+1-1) 


``````````diff
diff --git a/compiler-rt/lib/scudo/standalone/tests/timing_test.cpp b/compiler-rt/lib/scudo/standalone/tests/timing_test.cpp
index 23f0a02ea4277b..a762aee48f7c63 100644
--- a/compiler-rt/lib/scudo/standalone/tests/timing_test.cpp
+++ b/compiler-rt/lib/scudo/standalone/tests/timing_test.cpp
@@ -154,7 +154,7 @@ TEST_F(ScudoTimingTest, VerifyMax) {
   unsigned long long MaxNs = std::strtoull(&end[6], &end, 10);
   ASSERT_TRUE(end != nullptr);
 
-  EXPECT_GT(MaxNs, AvgNs);
+  EXPECT_GE(MaxNs, AvgNs);
 }
 
 TEST_F(ScudoTimingTest, VerifyMultipleTimerCalls) {

``````````

</details>


https://github.com/llvm/llvm-project/pull/106062


More information about the llvm-commits mailing list