From fd99b6ce9793a11422b25563e857324121c6fa57 Mon Sep 17 00:00:00 2001 From: Geoffrey Martin-Noble Date: Wed, 19 Jun 2019 11:30:16 -0700 Subject: [PATCH] Remove unnecessary -verify-diagnostics These were likely added in error because of confusion about the flag when it was just called "-verify". The extra flag doesn't cause much harm, but it does make mlir-opt do more work and clutter the RUN line PiperOrigin-RevId: 254037016 --- mlir/test/Examples/Linalg/Linalg1.mlir | 4 ++-- mlir/test/Linalg/roundtrip.mlir | 2 +- mlir/test/Quantizer/remove-instrumentation.mlir | 2 +- mlir/test/Transforms/loop-fusion.mlir | 4 ++-- mlir/test/Transforms/loop-invariant-code-motion.mlir | 2 +- mlir/test/Transforms/memref-dataflow-opt.mlir | 2 +- 6 files changed, 8 insertions(+), 8 deletions(-) diff --git a/mlir/test/Examples/Linalg/Linalg1.mlir b/mlir/test/Examples/Linalg/Linalg1.mlir index 6858707dca9c..5be5cdf18a96 100644 --- a/mlir/test/Examples/Linalg/Linalg1.mlir +++ b/mlir/test/Examples/Linalg/Linalg1.mlir @@ -1,5 +1,5 @@ -// RUN: linalg1-opt %s -verify-diagnostics | FileCheck %s -// RUN: linalg1-opt %s -lower-linalg-to-llvm -verify-diagnostics | FileCheck %s -check-prefix=LLVM +// RUN: linalg1-opt %s | FileCheck %s +// RUN: linalg1-opt %s -lower-linalg-to-llvm | FileCheck %s -check-prefix=LLVM func @view_op(%arg0: memref, %arg1: memref, %arg2: memref) { %c3 = constant 3 : index diff --git a/mlir/test/Linalg/roundtrip.mlir b/mlir/test/Linalg/roundtrip.mlir index ce7b14c0d8cd..5f8658152a1d 100644 --- a/mlir/test/Linalg/roundtrip.mlir +++ b/mlir/test/Linalg/roundtrip.mlir @@ -1,4 +1,4 @@ -// RUN: mlir-opt %s -verify-diagnostics | mlir-opt -verify-diagnostics | FileCheck %s +// RUN: mlir-opt %s | mlir-opt | FileCheck %s // CHECK: #[[map0:.*]] = (d0, d1, d2) -> (d0, d2, d1) // CHECK: #[[map1:.*]] = (d0, d1, d2) -> (d2, d1, d0) diff --git a/mlir/test/Quantizer/remove-instrumentation.mlir b/mlir/test/Quantizer/remove-instrumentation.mlir index 16d5337bef2e..222ddf751bd8 100644 --- a/mlir/test/Quantizer/remove-instrumentation.mlir +++ b/mlir/test/Quantizer/remove-instrumentation.mlir @@ -1,4 +1,4 @@ -// RUN: mlir-opt %s -quantizer-remove-instrumentation -verify-diagnostics -split-input-file | FileCheck %s +// RUN: mlir-opt %s -quantizer-remove-instrumentation -split-input-file | FileCheck %s // ----- // CHECK-LABEL: remove_ops diff --git a/mlir/test/Transforms/loop-fusion.mlir b/mlir/test/Transforms/loop-fusion.mlir index ee3614d05e61..ea8640232034 100644 --- a/mlir/test/Transforms/loop-fusion.mlir +++ b/mlir/test/Transforms/loop-fusion.mlir @@ -1,5 +1,5 @@ -// RUN: mlir-opt %s -affine-loop-fusion -split-input-file -verify-diagnostics | FileCheck %s -// RUN: mlir-opt %s -affine-loop-fusion -fusion-maximal -split-input-file -verify-diagnostics | FileCheck %s --check-prefix=MAXIMAL +// RUN: mlir-opt %s -affine-loop-fusion -split-input-file | FileCheck %s +// RUN: mlir-opt %s -affine-loop-fusion -fusion-maximal -split-input-file | FileCheck %s --check-prefix=MAXIMAL // TODO(andydavis) Add more tests: // *) Add nested fusion test cases when non-constant loop bound support is diff --git a/mlir/test/Transforms/loop-invariant-code-motion.mlir b/mlir/test/Transforms/loop-invariant-code-motion.mlir index 85bbd7a95755..4173386d5855 100644 --- a/mlir/test/Transforms/loop-invariant-code-motion.mlir +++ b/mlir/test/Transforms/loop-invariant-code-motion.mlir @@ -1,4 +1,4 @@ -// RUN: mlir-opt %s -affine-loop-invariant-code-motion -split-input-file -verify-diagnostics | FileCheck %s +// RUN: mlir-opt %s -affine-loop-invariant-code-motion -split-input-file | FileCheck %s func @nested_loops_both_having_invariant_code() { %m = alloc() : memref<10xf32> diff --git a/mlir/test/Transforms/memref-dataflow-opt.mlir b/mlir/test/Transforms/memref-dataflow-opt.mlir index 8a176466c957..979f4867d9bd 100644 --- a/mlir/test/Transforms/memref-dataflow-opt.mlir +++ b/mlir/test/Transforms/memref-dataflow-opt.mlir @@ -1,4 +1,4 @@ -// RUN: mlir-opt %s -memref-dataflow-opt -verify-diagnostics | FileCheck %s +// RUN: mlir-opt %s -memref-dataflow-opt | FileCheck %s // CHECK-DAG: [[MAP0:#map[0-9]+]] = (d0, d1) -> (d1 + 1) // CHECK-DAG: [[MAP1:#map[0-9]+]] = (d0, d1) -> (d0)