Enable argument type population of torch models #1073
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ticket
#1041, #856
Problem description
What's changed
torch.ops.tt.mark_argument_attributes
which applies astablehlo.custom_call
to a tensor :GraphModule
torch.compile
backend or by the utility function we provide:mark_model_user_inputs
.propagateRoleAttribute
to simply populate thettcore.argument_type
arg attribute in the function rather thantt.input_role
. This way we can bypass PopulateArgumentTypes in tt-mlir entirely as the graph is already populated.Checklist
I didn't add a test because we will not really be able to take advantage of this until the tt_torch backend is merged: #954, I will add a test after both of these are merged.
I did add an example though, using the "openxla" backend.