Skip to content

Conversation

@mehrdad2m
Copy link
Contributor

Context:
As #2169 have added auto detection of xdsl passes, we don't need to specify the pass plugin argument in the tests

Description of the Change:
Remove the reference to xdsl pass plugin within the test suit.

Benefits:

Possible Drawbacks:

Related GitHub Issues:
[sc-101160]

@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2025

Hello. You may have forgotten to update the changelog!
Please edit doc/releases/changelog-dev.md on your branch with:

  • A one-to-two sentence description of the change. You may include a small working example for new features.
  • A link back to this PR.
  • Your name (or GitHub username) in the contributors section.

@mudit2812
Copy link
Contributor

As a follow up, we should update the callback functions (draw, mlir-graph, specs, graph viz) to stop using the plugin as well. This would require us to update how we're implementing the functionality entirely, so no need to do it right now.

@mehrdad2m mehrdad2m merged commit 2ba6725 into main Dec 9, 2025
39 checks passed
@mehrdad2m mehrdad2m deleted the remove-xdsl-from-tests branch December 9, 2025 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants