Skip to content

[test]#946

Open
WCY2000 wants to merge 4 commits intotile-ai:npuirfrom
WCY2000:npuir_test
Open

[test]#946
WCY2000 wants to merge 4 commits intotile-ai:npuirfrom
WCY2000:npuir_test

Conversation

@WCY2000
Copy link
Copy Markdown

@WCY2000 WCY2000 commented Apr 29, 2026

No description provided.

@github-actions
Copy link
Copy Markdown

👋 Hi! Thank you for contributing to the TileLang project.

Please remember to run bash format.sh in the root directory of the project to ensure your changes are properly linted and formatted. This will help ensure your contribution passes the format check.

We appreciate you taking this step! Our team will review your contribution, and we look forward to your awesome work!

🚀

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a debug print statement in the compile function when the target is set to 'npuir'. The feedback correctly identifies this as a debug artifact that should be removed or replaced with a proper logger.

Comment thread tilelang/jit/__init__.py
Additional keyword arguments to pass to the Compiler PassContext.
"""
if target == "npuir":
print("<<<< line 70, target == npuir")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

This print statement appears to be a debug artifact and should be removed. If logging is required, consider using the module-level logger defined in this file.

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.

1 participant