Re-enable FakeTensor caching for SymInts #152662

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

aorenste wants to merge 24 commits into gh/aorenste/226/base from gh/aorenste/226/head

None on "Re-enable FakeTensor caching for SymInts"

Summary:

This backs out D60320595 which itself turned off FakeTensor caching when a SymInt was present.

Tests seem to pass so I'm assuming some dynamic shape work fixed what was breaking previously.

Test Plan: Reran the tests listed in T196779132 and they seem to pass.

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

[ghstack-poisoned]

Loading branch information

aorenste committed May 12, 2025

commit b103659e283eeb3bff3eb1b7b833c61c9793e072

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Re-enable FakeTensor caching for SymInts #152662

Uh oh!

Diff view

Diff view

Uh oh!

There are no files selected for viewing

This merge commit was added into this branch cleanly.