[Bugfix] revert incorrect fast path for parallel layout inference

[LeiWang1999]

as title.

Summary by CodeRabbit

• Refactor
  • Simplified buffer layout handling in parallel operations by removing conditional validation logic. Source buffer layouts are now consistently reused for common access patterns, streamlining the code path.

_{✏️ 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]

📝 Walkthrough

Walkthrough

The ComputeLoopLayoutFromBuffer function in src/op/parallel.cc has been simplified to unconditionally reuse the source buffer layout for common access buffers, removing the prior conditional validation logic that checked for matching loop variable counts and loop ranges before allowing layout reuse.

Changes

Cohort / File(s)	Summary
Buffer Layout Simplification src/op/parallel.cc	Removed 20 lines of conditional logic from ComputeLoopLayoutFromBuffer that validated can_use_src_layout_directly flag before reusing source layout; now directly assigns result = src_layout for common buffers without prior compatibility checks

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• [BugFix] Corrected when proving loop layout contains a fragment buffer layout #1708: Also modifies loop-layout/fragment containment logic in src/op/parallel.cc, specifically altering ProveFragmentContains calls during candidate validation.
• [Bugfix] Recover code for flexible parallel #1032: Modifies the same ComputeLoopLayoutFromBuffer function in src/op/parallel.cc and changes its control flow for buffer source layout reuse.

Poem

🐰 Conditions trimmed with careful snip,
Layout reuse need not skip,
Simpler logic, cleaner code,
Down the faster, bolder road!

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: reverting an incorrect fast path for parallel layout inference in ComputeLoopLayoutFromBuffer function.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

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.}