Skip to content

[BE] Use simdgroup_size constexpr#157751

Closed
malfet wants to merge 1 commit intogh/malfet/431/basefrom
gh/malfet/431/head
Closed

[BE] Use simdgroup_size constexpr#157751
malfet wants to merge 1 commit intogh/malfet/431/basefrom
gh/malfet/431/head

Conversation

@malfet
Copy link
Contributor

@malfet malfet commented Jul 8, 2025

Stack from ghstack (oldest at bottom):

Instead of every shader defining it separately, move it to c10/metal/common.h

[ghstack-poisoned]
@pytorch-bot
Copy link

pytorch-bot bot commented Jul 8, 2025

🔗 Helpful Links

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

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

✅ You can merge normally! (6 Unrelated Failures)

As of commit 01808fa with merge base 510c398 (image):

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

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

@pytorch-bot pytorch-bot bot added ciflow/mps Run MPS tests (subset of trunk) release notes: mps Release notes category labels Jul 8, 2025
@malfet malfet added the topic: not user facing topic category label Jul 8, 2025
@malfet malfet requested review from dcci and manuelcandales July 8, 2025 00:14
@malfet
Copy link
Contributor Author

malfet commented Jul 8, 2025

@pytorchbot merge -f "Those are just BE changes"

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

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

@github-actions github-actions bot deleted the gh/malfet/431/head branch August 8, 2025 02:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/mps Run MPS tests (subset of trunk) Merged release notes: mps Release notes category topic: not user facing topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants