[Mlir-commits] [mlir] [MLIR] Remove uses of %T from lit tests (PR #150725)
llvmlistbot at llvm.org
llvmlistbot at llvm.org
Fri Jul 25 16:58:51 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-mlir
Author: Aiden Grossman (boomanaiden154)
<details>
<summary>Changes</summary>
%T has been deprecated for about seven years and is to be avoided given it does not use a unique dir per test. Remove it from lit tests in MLIR with the eventual goal to be removing support within llvm-lit.
This patch just reuses %t.o given it is deleted in between sections of the test.
---
Full diff: https://github.com/llvm/llvm-project/pull/150725.diff
1 Files Affected:
- (modified) mlir/test/mlir-runner/simple.mlir (+3-3)
``````````diff
diff --git a/mlir/test/mlir-runner/simple.mlir b/mlir/test/mlir-runner/simple.mlir
index 1a03b9901174f..21dabdd42825e 100644
--- a/mlir/test/mlir-runner/simple.mlir
+++ b/mlir/test/mlir-runner/simple.mlir
@@ -15,10 +15,10 @@
// RUN: ls %t.o
// RUN: rm %t.o
-// RUN: mlir-runner %s -dump-object-file -object-filename=%T/test.o \
+// RUN: mlir-runner %s -dump-object-file -object-filename=%t.o \
// RUN: %if target={{s390x-.*}} %{ -argext-abi-check=false %} | FileCheck %s
-// RUN: ls %T/test.o
-// RUN: rm %T/test.o
+// RUN: ls %t.o
+// RUN: rm %t.o
// Declarations of C library functions.
llvm.func @logbf(f32) -> f32
``````````
</details>
https://github.com/llvm/llvm-project/pull/150725
More information about the Mlir-commits
mailing list