[Mlir-commits] [mlir] [mlir][Python] remove stray nb::cast (PR #176299)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Thu Jan 15 18:28:31 PST 2026


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir

Author: Maksim Levental (makslevental)

<details>
<summary>Changes</summary>

In https://github.com/llvm/llvm-project/pull/155114 we removed `liveOperations` but forget this line which was being used to invalidate operations under a transform root, which currently isn't being used for anything. So remove. 

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


1 Files Affected:

- (modified) mlir/lib/Bindings/Python/TransformInterpreter.cpp (-6) 


``````````diff
diff --git a/mlir/lib/Bindings/Python/TransformInterpreter.cpp b/mlir/lib/Bindings/Python/TransformInterpreter.cpp
index a9f204ff9d0a5..b263e65ff8cf8 100644
--- a/mlir/lib/Bindings/Python/TransformInterpreter.cpp
+++ b/mlir/lib/Bindings/Python/TransformInterpreter.cpp
@@ -71,12 +71,6 @@ static void populateTransformInterpreterSubmodule(nb::module_ &m) {
          PyOperationBase &transformModule, const PyTransformOptions &options) {
         mlir::python::CollectDiagnosticsToStringScope scope(
             mlirOperationGetContext(transformRoot.getOperation()));
-
-        // Calling back into Python to invalidate everything under the payload
-        // root. This is awkward, but we don't have access to PyMlirContext
-        // object here otherwise.
-        nb::object obj = nb::cast(payloadRoot);
-
         MlirLogicalResult result = mlirTransformApplyNamedSequence(
             payloadRoot.getOperation(), transformRoot.getOperation(),
             transformModule.getOperation(), options.options);

``````````

</details>


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


More information about the Mlir-commits mailing list