Skip to content

[TensorLayout] Mark MatMul and Pad as nodes that can operate on any layout #3832

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
wants to merge 1 commit into from

Conversation

shajrawi
Copy link
Contributor

@shajrawi shajrawi commented Dec 2, 2019

Summary:
While they are not data parallel, they can operate on any order of dimensions.

Fixes #3804

Test Plan:

model-compiler -backend=CPU -model=model1.onnx -emit-bundle=build
model-compiler -backend=CPU -model=model2.onnx -emit-bundle=build

…ayout

While they are not data parallel, they can operate on any order of dimensions.
Copy link
Contributor

@opti-mix opti-mix left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@shajrawi has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@mciprian13
Copy link
Contributor

If I'm not too tiresome :) can you take a look over all the operators currently supported by Glow and verify if there is one which might have slipped from being marked as 'any layout'? Thanks for fixing this!

vdantu pushed a commit to vdantu/glow that referenced this pull request Jul 12, 2020
…3832)

Summary:
While they are not data parallel, they can operate on any order of dimensions.

Fixes pytorch#3804
Pull Request resolved: pytorch#3832

Test Plan:
```
model-compiler -backend=CPU -model=model1.onnx -emit-bundle=build
model-compiler -backend=CPU -model=model2.onnx -emit-bundle=build
```

Differential Revision: D18765346

Pulled By: shajrawi

fbshipit-source-id: c44713036b00918ac14ef62f117982d0dfcfc062
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Layout] Layout verification failed
4 participants