[Mlir-commits] [mlir] [mlir][llvm] Use the roundtrip flag in test (PR #173782)

[llvmlistbot at llvm.org]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-mlir

@llvm/pr-subscribers-mlir-llvm

Author: Tobias Gysi (gysit)

<details>
<summary>Changes</summary>

This commit changes the debug info test to use the `--verify-roundtrip` flag. The byte code rountrip has been problematic before. Recent changes to the attribute parsing make it possible to use the flag.

This closes https://github.com/llvm/llvm-project/issues/163004

---
Full diff: https://github.com/llvm/llvm-project/pull/173782.diff


1 Files Affected:

- (modified) mlir/test/Dialect/LLVMIR/debuginfo.mlir (+1-2) 


``````````diff
diff --git a/mlir/test/Dialect/LLVMIR/debuginfo.mlir b/mlir/test/Dialect/LLVMIR/debuginfo.mlir
index d7bf99bfaed7f..545715f73f449 100644
--- a/mlir/test/Dialect/LLVMIR/debuginfo.mlir
+++ b/mlir/test/Dialect/LLVMIR/debuginfo.mlir
@@ -1,5 +1,4 @@
-// RUN: mlir-opt %s | mlir-opt | FileCheck %s
-// RUN: mlir-opt -emit-bytecode %s | mlir-opt | FileCheck %s
+// RUN: mlir-opt %s --verify-roundtrip | mlir-opt | FileCheck %s
 
 // CHECK-DAG: #[[FILE:.*]] = #llvm.di_file<"debuginfo.mlir" in "/test/">
 #file = #llvm.di_file<"debuginfo.mlir" in "/test/">

``````````

</details>


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