Skip to content

Conversation

xmfan
Copy link
Member

@xmfan xmfan commented Sep 19, 2025

[ghstack-poisoned]
Copy link

pytorch-bot bot commented Sep 19, 2025

🔗 Helpful Links

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

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

✅ No Failures

As of commit 60cf911 with merge base 23af32a (image):
💚 Looks good so far! There are no failures yet. 💚

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

xmfan added a commit that referenced this pull request Sep 19, 2025
@xmfan xmfan changed the title [inductor] as_strided lowering throws away .view(dtype) [inductor] as_strided lowering throws away .view(dtype/device) Sep 19, 2025
@eellison eellison self-requested a review September 19, 2025 16:25
[ghstack-poisoned]
xmfan added a commit that referenced this pull request Sep 19, 2025
@xmfan xmfan changed the title [inductor] as_strided lowering throws away .view(dtype/device) [inductor] fix dtype and device propagation through as_strided lowering for views Sep 19, 2025
@xmfan xmfan changed the title [inductor] fix dtype and device propagation through as_strided lowering for views [inductor] fix as_strided lowering propagation through dtype/device views Sep 19, 2025
@xmfan xmfan marked this pull request as ready for review September 19, 2025 20:08
[ghstack-poisoned]
xmfan added a commit that referenced this pull request Sep 19, 2025
@xmfan xmfan changed the title [inductor] fix as_strided lowering propagation through dtype/device views [inductor] fix as_strided lowering with dtype/device view inputs Sep 19, 2025
Comment on lines 1211 to 1215
# its metadata. Except for device type and dtype, which we need
# to propagate.

# Technically device is not needed because it is handled as a copy
# instead of a view today.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: Except for device type and dtype, which we need # to propagate.

rewrite this ? it's not possible to take cross-device views.

@xmfan xmfan added the topic: not user facing topic category label Sep 22, 2025
…inputs"


FIXES #163286

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

[ghstack-poisoned]
xmfan added a commit that referenced this pull request Sep 22, 2025
@xmfan xmfan changed the title [inductor] fix as_strided lowering with dtype/device view inputs [inductor] fix as_strided lowering with .view(dtype) inputs Sep 22, 2025
@xmfan
Copy link
Member Author

xmfan commented Sep 23, 2025

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Sep 23, 2025
@pytorchmergebot
Copy link
Collaborator

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

@pytorchmergebot
Copy link
Collaborator

The merge job was canceled or timed out. This most often happen if two merge requests were issued for the same PR, or if merge job was waiting for more than 6 hours for tests to finish. In later case, please do not hesitate to reissue the merge command
For more information see pytorch-bot wiki.

@xmfan
Copy link
Member Author

xmfan commented Sep 23, 2025

@pytorchbot merge

1 similar comment
@xmfan
Copy link
Member Author

xmfan commented Sep 23, 2025

@pytorchbot merge

@pytorchmergebot
Copy link
Collaborator

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

dsashidh pushed a commit to dsashidh/pytorch that referenced this pull request Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants