[Mlir-commits] [mlir] [mlir][tosa] Converted TosaAvailability pass to ModuleOp from FuncOp, fixes crash in threaded pass manager (PR #183063)

[Arjun Bhamra]

================
@@ -19,7 +19,7 @@ using namespace mlir;
 namespace {
 /// A pass for testing Tosa op availability.
 struct PrintOpAvailability
-    : public PassWrapper<PrintOpAvailability, OperationPass<func::FuncOp>> {
+    : public PassWrapper<PrintOpAvailability, OperationPass<ModuleOp>> {
----------------
abhamra wrote:

The original issue was triggering non-deterministically because the pass manager parallelizes by default: https://github.com/llvm/llvm-project/issues/182255. Is there a good way to make a test case for this situation? I wasn't sure how to go about it, as it seemed like a straightforward conversion otherwise.

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