[Bugfix] Avoid considering local.var buffer as local

[LeiWang1999]

This pull request includes minor improvements to logging and buffer scope checks in the parallel operation utilities.

Buffer scope handling:

• Updated IsLocalBuffer in src/op/utils.h to only consider buffers with scope "local" as local buffers, removing support for "local.var" scope.

Logging cleanup:

• Removed an informational log statement from the ExpandLetBindings method in src/op/parallel.cc to reduce unnecessary output.

Summary by CodeRabbit

• Bug Fixes

  • Improved buffer scope detection accuracy.

• Chores

  • Removed verbose logging for buffer operations.

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

[github-actions]

👋 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]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Two utility files modified to refine buffer classification logic: removed verbose INFO logging from ExpandLetBindings and tightened IsLocalBuffer helper to exclusively recognize buffers with "local" scope, excluding "local.var" scope from local-buffer classification.

Changes

Cohort / File(s)	Summary
Logging Reduction src/op/parallel.cc	Removed verbose INFO log statement in ExpandLetBindings when encountering BufferLoad referencing fragment buffers not yet in indice_map_; logic unchanged
Buffer Scope Classification src/op/utils.h	Tightened IsLocalBuffer helper to only classify buffers with "local" scope as local; "local.var" scope buffers now excluded from local-buffer detection

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• [Bugfix] Disable Memory Info Analysis for local.var #851: Both PRs modify buffer-scope classification logic, affecting how "local.var" and ".var" buffers are categorized in memory handling.
• [Enhancement] Introduce a workaround for layout inference for local buffer store #1055: Related through shared local-buffer handling changes; both PRs directly impact buffer scope checks that influence parallel-loop layout inference logic.

Poem

🐰 A fragment's log hops away,
Local scopes now strictly say:
Only "local" buffers stay,
.var variants must play,
Cleaner logic rules the day! ✨

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9ff7c52 and f127902.

📒 Files selected for processing (2)

• src/op/parallel.cc
• src/op/utils.h

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[LeiWang1999]

It's likely this pr introduced some bugs