[llvm] 30c5c48 - [CAS][Tests] Fix unit tests that hangs on two cores (#154151)

via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 19 08:21:37 PDT 2025


Author: Steven Wu
Date: 2025-08-19T08:21:34-07:00
New Revision: 30c5c48d871df99844d30e2bf4c31cd81150cc9f

URL: https://github.com/llvm/llvm-project/commit/30c5c48d871df99844d30e2bf4c31cd81150cc9f
DIFF: https://github.com/llvm/llvm-project/commit/30c5c48d871df99844d30e2bf4c31cd81150cc9f.diff

LOG: [CAS][Tests] Fix unit tests that hangs on two cores (#154151)

Added: 
    

Modified: 
    llvm/unittests/CAS/ObjectStoreTest.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/unittests/CAS/ObjectStoreTest.cpp b/llvm/unittests/CAS/ObjectStoreTest.cpp
index e84e30374c9aa..54083fdb408f6 100644
--- a/llvm/unittests/CAS/ObjectStoreTest.cpp
+++ b/llvm/unittests/CAS/ObjectStoreTest.cpp
@@ -269,8 +269,7 @@ TEST_P(CASTest, NodesBig) {
     ASSERT_THAT_ERROR(CAS->validate(CAS->getID(ID)), Succeeded());
 }
 
-// FIXME: Re-enable the test.
-#if 0
+#if LLVM_ENABLE_THREADS
 /// Common test functionality for creating blobs in parallel. You can vary which
 /// cas instances are the same or 
diff erent, and the size of the created blobs.
 static void testBlobsParallel(ObjectStore &Read1, ObjectStore &Read2,
@@ -318,10 +317,10 @@ static void testBlobsParallel(ObjectStore &Read1, ObjectStore &Read2,
 
   DefaultThreadPool Threads;
   for (unsigned I = 0; I < BlobCount; ++I) {
-    Threads.async(Consumer, I, &Read1);
-    Threads.async(Consumer, I, &Read2);
     Threads.async(Producer, I, &Write1);
     Threads.async(Producer, I, &Write2);
+    Threads.async(Consumer, I, &Read1);
+    Threads.async(Consumer, I, &Read2);
   }
 
   Threads.wait();


        


More information about the llvm-commits mailing list