Skip to content

Conversation

@LJC00118
Copy link
Collaborator

@LJC00118 LJC00118 commented Dec 31, 2025

Summary by CodeRabbit

  • Chores
    • Updated underlying compilation infrastructure to latest version for improved stability and performance.

✏️ Tip: You can customize this high-level summary in your review settings.

@github-actions
Copy link

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

Please remember to run pre-commit run --all-files 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! 🚀

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 31, 2025

📝 Walkthrough

Walkthrough

The 3rdparty/tvm submodule reference is updated to a new commit, moving from 23bce012ffd255a24289eea6ceab74a40b94a096 to dcfe86ebb33380597e894acce08e818fcdb27b0a, incorporating upstream changes.

Changes

Cohort / File(s) Summary
Submodule Update
3rdparty/tvm
Submodule commit reference updated to latest upstream version

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A hop, a skip, a commit so new,
TVM's updated through and through!
No changes here, just versions aligned,
Upstream progress, perfectly timed! 🚀

Pre-merge checks

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title describes adding PrimExpr substitution support for AttrStmt nodes, but the actual change only updates a submodule reference in 3rdparty/tvm with no direct code changes visible. Verify the PR title matches the actual changes made. If this is a submodule update, use a title like 'Update 3rdparty/tvm submodule' or clarify what the submodule update accomplishes.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 15c457f and d1bd90b.

📒 Files selected for processing (1)
  • 3rdparty/tvm
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Test for Python 3.12 with Metal (on macos-latest)
🔇 Additional comments (1)
3rdparty/tvm (1)

1-1: Submodule update is verified to contain the intended PrimExpr substitution support for AttrStmt nodes.

The commit dcfe86ebb33380597e894acce08e818fcdb27b0a adds PrimExpr substitution handling in src/tir/ir/stmt_functor.cc (5 added lines), extending the IRSubstitute function to properly handle PrimExpr nodes in AttrStmt objects. This is a minimal, focused change that follows existing substitution patterns.

Ensure integration tests pass with the updated submodule reference before merging.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@LeiWang1999 LeiWang1999 merged commit e1f76d1 into tile-ai:main Jan 1, 2026
7 checks passed
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.

2 participants