-
Notifications
You must be signed in to change notification settings - Fork 355
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
aten::empty_like #2654
aten::empty_like #2654
Conversation
Had a doubt on this one. Does this require a test. In the following test:
Is this expected? Is it something to do with no splits happening for the above graph? |
I'm not sure what the |
I do not think that the graph would be empty since it would reduce to the lowering operations of |
I verified the above test case with three cases-
Without decomposition of empty_like
b. After AOT trace
c. After lowering passes
This is the graph for partition With the decomposition of empty_like
b. After AOT trace
c. After lowering passes
The above graph partitioning errors out at
Like the above case during compilation, if the empty_like is included in the decomposition, the shape of x is extracted statically before runtime and the graph subgraphs is not created.
With the decomposition of empty_like
b. After AOT trace
c. After lowering passes
In the above case since there are additional add nodes with the frozen_param nodes, so the subgraph is created. Studying the above cases, it seems that the aten lowering is happening during AOT trace. As discussed ideally a test case should not be required. I do not believe |
Thanks for the analysis @apbose - this is very helpful. It looks like the Regarding |
Ok I will go ahead and make a separate PR for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
No description provided.