Skip to content
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

[AutoBump] Merge with fixes of 00efec0b (May 10) (35) #268

Merged
merged 2 commits into from
Sep 3, 2024

Conversation

mgehre-amd
Copy link
Collaborator

No description provided.

stellaraccident and others added 2 commits May 10, 2024 13:45
* Enables assume_strict_symbolic_shapes on fx_importer imported
programs, indicating strict shape semantics.
* Reworks the view->reshape lowering to take advantage of strict mode
and do one of:
  * Collapse to 0D
  * Flatten/Unflatten when there is an inferred dim.
  * Fallback to tensor.reshape
* Splits some test cases up and adds an attribute to control the old
pattern (so new corners can be tested in strict mode in isolation).
* Dynamic inferred mode needs upstream work to generalize expand_shape
(so that case is suppressed here).
* Deletes the assert from the existing tensor.reshape lowering if strict
shape mode is enabled (since the condition it is dynamically asserting
cannot happen).
@mgehre-amd mgehre-amd requested a review from cferry-AMD August 27, 2024 13:10
@cferry-AMD
Copy link

One note on TOSA bumps but not just those: I'm blindly approving whatever comes from bumps, trusting the test suite to make sure nothing breaks. If it turns out non-trivial changes are needed to reconcile our fork and upstream, please send a heads up asking for a detailed review :)

Base automatically changed from bump_to_adafd518 to feature/backport_ea1_ops September 3, 2024 20:09
An error occurred while trying to automatically change base from bump_to_adafd518 to feature/backport_ea1_ops September 3, 2024 20:09
@mgehre-amd mgehre-amd merged commit 01c9f23 into feature/backport_ea1_ops Sep 3, 2024
4 checks passed
@mgehre-amd mgehre-amd deleted the bump_to_00efec0b branch September 3, 2024 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants