[compiler-rt] [NFC] Fix flakiness in test if run unsharded (PR #112439)

via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 15 14:38:06 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

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

Author: Vitaly Buka (vitalybuka)

<details>
<summary>Changes</summary>

If we run all test in a single process, there is high
probability that `99` is already claimed.


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


1 Files Affected:

- (modified) compiler-rt/lib/sanitizer_common/tests/sanitizer_chained_origin_depot_test.cpp (+1-1) 


``````````diff
diff --git a/compiler-rt/lib/sanitizer_common/tests/sanitizer_chained_origin_depot_test.cpp b/compiler-rt/lib/sanitizer_common/tests/sanitizer_chained_origin_depot_test.cpp
index a557c4645ba0c5..61171019a5706d 100644
--- a/compiler-rt/lib/sanitizer_common/tests/sanitizer_chained_origin_depot_test.cpp
+++ b/compiler-rt/lib/sanitizer_common/tests/sanitizer_chained_origin_depot_test.cpp
@@ -31,7 +31,7 @@ TEST(SanitizerCommon, ChainedOriginDepotBasic) {
 
 TEST(SanitizerCommon, ChainedOriginDepotAbsent) {
   u32 prev_id;
-  EXPECT_EQ(0U, chainedOriginDepot.Get(99, &prev_id));
+  EXPECT_EQ(0U, chainedOriginDepot.Get(123456, &prev_id));
   EXPECT_EQ(0U, prev_id);
 }
 

``````````

</details>


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


More information about the llvm-commits mailing list