From 26167cae45154d326aba88c8e8ca72e2bc42ea30 Mon Sep 17 00:00:00 2001 From: Mehdi Amini Date: Fri, 21 Jan 2022 05:15:46 +0000 Subject: [PATCH] Print the `// ----` separator between modules when using -split-input-file with mlir-opt This allows to pipe sequences of `mlir-opt -split-input-file | mlir-opt -split-input-file`. Depends On D117750 Reviewed By: rriddle Differential Revision: https://reviews.llvm.org/D117756 --- mlir/lib/Support/MlirOptMain.cpp | 10 ++++++---- mlir/test/Dialect/OpenACC/ops.mlir | 4 ++-- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/mlir/lib/Support/MlirOptMain.cpp b/mlir/lib/Support/MlirOptMain.cpp index 5c3f7a773240..af4681f50ccb 100644 --- a/mlir/lib/Support/MlirOptMain.cpp +++ b/mlir/lib/Support/MlirOptMain.cpp @@ -158,10 +158,12 @@ LogicalResult mlir::MlirOptMain(raw_ostream &outputStream, return splitAndProcessBuffer( std::move(buffer), [&](std::unique_ptr chunkBuffer, raw_ostream &os) { - return processBuffer(os, std::move(chunkBuffer), verifyDiagnostics, - verifyPasses, allowUnregisteredDialects, - preloadDialectsInContext, passManagerSetupFn, - registry, threadPool); + LogicalResult result = processBuffer( + os, std::move(chunkBuffer), verifyDiagnostics, verifyPasses, + allowUnregisteredDialects, preloadDialectsInContext, + passManagerSetupFn, registry, threadPool); + os << "// -----\n"; + return result; }, outputStream); diff --git a/mlir/test/Dialect/OpenACC/ops.mlir b/mlir/test/Dialect/OpenACC/ops.mlir index 9ced56acfe74..05e8026b554c 100644 --- a/mlir/test/Dialect/OpenACC/ops.mlir +++ b/mlir/test/Dialect/OpenACC/ops.mlir @@ -1,8 +1,8 @@ // RUN: mlir-opt -split-input-file %s | FileCheck %s // Verify the printed output can be parsed. -// RUN: mlir-opt -split-input-file %s | mlir-opt -allow-unregistered-dialect | FileCheck %s +// RUN: mlir-opt -split-input-file %s | mlir-opt -split-input-file | FileCheck %s // Verify the generic form can be parsed. -// RUN: mlir-opt -split-input-file -mlir-print-op-generic %s | mlir-opt -allow-unregistered-dialect | FileCheck %s +// RUN: mlir-opt -split-input-file -mlir-print-op-generic %s | mlir-opt -split-input-file | FileCheck %s func @compute1(%A: memref<10x10xf32>, %B: memref<10x10xf32>, %C: memref<10x10xf32>) -> memref<10x10xf32> { %c0 = arith.constant 0 : index