From aec73eade7af0e22c944714bec31570181bc1ad4 Mon Sep 17 00:00:00 2001 From: Aart Bik <39774503+aartbik@users.noreply.github.com> Date: Wed, 10 Jan 2024 13:36:20 -0800 Subject: [PATCH] [mlir][sparse] allow unknown ops in one-shot bufferization in mini-pipeline (#77688) Rationale: Since this mini-pipeline may be used in alternative pipelines (viz. different from the default "sparsifier" pipeline) where unknown ops are handled by alternative bufferization methods that are downstream of this mini-pipeline, we allow unknown ops by default (failure to bufferize is eventually apparent by failing to convert to LLVM IR). This is part of enabling e2e testing for TORCH-MLIR tests using a sparsifier backend --- .../Transforms/SparsificationAndBufferizationPass.cpp | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/mlir/lib/Dialect/SparseTensor/Transforms/SparsificationAndBufferizationPass.cpp b/mlir/lib/Dialect/SparseTensor/Transforms/SparsificationAndBufferizationPass.cpp index 6266c63064ff..f497be6e48eb 100644 --- a/mlir/lib/Dialect/SparseTensor/Transforms/SparsificationAndBufferizationPass.cpp +++ b/mlir/lib/Dialect/SparseTensor/Transforms/SparsificationAndBufferizationPass.cpp @@ -201,6 +201,12 @@ mlir::getBufferizationOptionsForSparsification(bool analysisOnly) { options.testAnalysisOnly = true; options.printConflicts = true; } + // Since this mini-pipeline may be used in alternative pipelines (viz. + // different from the default "sparsifier" pipeline) where unknown ops + // are handled by alternative bufferization methods that are downstream + // of this mini-pipeline, we allow unknown ops by default (failure to + // bufferize is eventually apparent by failing to convert to LLVM IR). + options.allowUnknownOps = true; return options; }