[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