Skip to content

[BugFix] fix conditions to apply tma#174480

Closed
BoyuanFeng wants to merge 1 commit intopytorch:mainfrom
BoyuanFeng:export-D92533457
Closed

[BugFix] fix conditions to apply tma#174480
BoyuanFeng wants to merge 1 commit intopytorch:mainfrom
BoyuanFeng:export-D92533457

Conversation

@BoyuanFeng
Copy link
Copy Markdown
Contributor

@BoyuanFeng BoyuanFeng commented Feb 6, 2026 •
edited
Loading

Summary: This diff fixes a wrong condition introduced in D92015529 for applying TMA. This should fix S618103.

Test Plan: Test in mast job.

Differential Revision: D92533457

cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @ipiszy @kadeng @muchulee8 @amjames @chauhang @aakhundov @coconutruben @jataylo @jianyizh

@BoyuanFeng @facebook-github-bot
[BugFix] fix conditions to apply tma
c263f8a 
Summary: This diff fixes a wrong condition introduced in D92015529 for applying TMA. This should fix S618103.

Test Plan: Test in mast job.

Differential Revision: D92533457
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented Feb 6, 2026 •
edited
Loading

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/174480

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit c263f8a with merge base bedb3e9 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented Feb 6, 2026

This PR needs a release notes: label

If your changes are user facing and intended to be a part of release notes, please use a label starting with release notes:.

If not, please add the topic: not user facing label.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "topic: not user facing"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Feb 6, 2026

@BoyuanFeng has exported this pull request. If you are a Meta employee, you can view the originating Diff in D92533457.

@BoyuanFeng BoyuanFeng added the topic: not user facing topic category label Feb 6, 2026
@BoyuanFeng BoyuanFeng requested a review from eellison February 6, 2026 19:33
@pytorch-bot pytorch-bot Bot added the ciflow/trunk Trigger trunk jobs on your pull request label Feb 6, 2026
@facebook-github-bot
Copy link
Copy Markdown
Contributor

facebook-github-bot commented Feb 7, 2026

@pytorchbot merge

(Initiating merge automatically since Phabricator Diff has merged)

@pytorchmergebot
Copy link
Copy Markdown
Collaborator

pytorchmergebot commented Feb 7, 2026

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

radeksm pushed a commit to radeksm/pytorch that referenced this pull request Feb 20, 2026
@BoyuanFeng @radeksm
[BugFix] fix conditions to apply tma (pytorch#174480)
90a0a35 
Summary: This diff fixes a wrong condition introduced in D92015529 for applying TMA. This should fix S618103.

Test Plan: Test in mast job.

Differential Revision: D92533457

Pull Request resolved: pytorch#174480
Approved by: https://github.com/Microve
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Microve Microve Microve approved these changes

@eellison eellison Awaiting requested review from eellison

Assignees

No one assigned

Labels

ciflow/inductor ciflow/trunk Trigger trunk jobs on your pull request fb-exported Merged meta-exported module: inductor topic: not user facing topic category

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@BoyuanFeng @facebook-github-bot @pytorchmergebot @Microve