Skip to content

Conversation

@zoecarver
Copy link
Contributor

No description provided.

@zoecarver zoecarver requested a review from brnorris03 January 6, 2026 19:44
Copy link
Contributor

@brnorris03 brnorris03 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would confirm first that the exact same steps work on qb or any tt hw machine, I suspect the tt-mlir configure options in cmake/modules/ExternTTMLIR.cmake would need updating since we don't require ttrt and use ttnn jit.

@zoecarver
Copy link
Contributor Author

I want to land this to validate that the added worker is usable, then I can look into build config issues.

@brnorris03
Copy link
Contributor

I want to land this to validate that the added worker is usable, then I can look into build config issues.

You can bypass requirements then.

@zoecarver
Copy link
Contributor Author

I would feel better about an approval :)

Is there a different way that you'd like to proceed with this?

@zoecarver
Copy link
Contributor Author

@zoecarver zoecarver closed this Jan 12, 2026
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.

3 participants