Skip to content

Conversation

@admud
Copy link
Contributor

@admud admud commented Jul 18, 2020

Fixes #41700.

@gchanan gchanan requested a review from SplitInfinity July 20, 2020 23:43
@gchanan gchanan added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Jul 20, 2020
grad_output_contig_t.resize_(grad_output_contig_t.sizes(), layout);
TensorArg grad_output_contig{ grad_output_contig_t, "grad_output", 1 };

Choose a reason for hiding this comment

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

Can you undo this change? It does not look to be related to the rest.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

which one do you want me to undo? or are we good for now?

Copy link
Contributor

@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.

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

Copy link
Contributor

@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.

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

@facebook-github-bot
Copy link
Contributor

@SplitInfinity merged this pull request in cb9c204.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged open source triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace blacklist/whitelist in aten/src/ATen/native/cudnn/Conv.cpp

9 participants