[Mlir-commits] [mlir] 4b9d28b - Partial rollback: Disable MLIR verifier parallelism.

Stella Laurenzo llvmlistbot at llvm.org
Fri Jun 18 14:58:17 PDT 2021


Author: Stella Laurenzo
Date: 2021-06-18T21:58:03Z
New Revision: 4b9d28bd530fd227a5db4da094d5a60577e1d1ac

URL: https://github.com/llvm/llvm-project/commit/4b9d28bd530fd227a5db4da094d5a60577e1d1ac
DIFF: https://github.com/llvm/llvm-project/commit/4b9d28bd530fd227a5db4da094d5a60577e1d1ac.diff

LOG: Partial rollback: Disable MLIR verifier parallelism.

Deadlocks have been found in several downstream projects as noted on the original patch: https://reviews.llvm.org/D104207

Disabling pending full root cause analysis.

Differential Revision: https://reviews.llvm.org/D104570

Added: 
    

Modified: 
    mlir/lib/IR/Verifier.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/lib/IR/Verifier.cpp b/mlir/lib/IR/Verifier.cpp
index 5f10d97a6bfb..d1d9ffb5e61a 100644
--- a/mlir/lib/IR/Verifier.cpp
+++ b/mlir/lib/IR/Verifier.cpp
@@ -44,7 +44,9 @@ namespace {
 class OperationVerifier {
 public:
   explicit OperationVerifier(MLIRContext *context)
-      : parallelismEnabled(context->isMultithreadingEnabled()) {}
+      // TODO: Re-enable parallelism once deadlocks found in D104207 are
+      // resolved.
+      : parallelismEnabled(false) {}
 
   /// Verify the given operation.
   LogicalResult verifyOpAndDominance(Operation &op);


        


More information about the Mlir-commits mailing list