Skip to content

Remove at::StorageImpl::finalizer_#11022

Closed
cpuhrsch wants to merge 2 commits intopytorch:masterfrom
cpuhrsch:nofin
Closed

Remove at::StorageImpl::finalizer_#11022
cpuhrsch wants to merge 2 commits intopytorch:masterfrom
cpuhrsch:nofin

Conversation

@cpuhrsch
Copy link
Contributor

Unused member variable

Further align at::StorageImpl with caffe2::StorageImpl
Unused member variable
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.

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

zdevito pushed a commit to zdevito/ATen that referenced this pull request Aug 30, 2018
Summary:
Unused member variable
Pull Request resolved: pytorch/pytorch#11022

Reviewed By: ezyang

Differential Revision: D9562520

Pulled By: cpuhrsch

fbshipit-source-id: af190b3ba06d33d65fa0fabffb34a0df769f38d0
PenghuiCheng pushed a commit to PenghuiCheng/pytorch that referenced this pull request Sep 11, 2018
Summary:
Unused member variable
Pull Request resolved: pytorch#11022

Reviewed By: ezyang

Differential Revision: D9562520

Pulled By: cpuhrsch

fbshipit-source-id: af190b3ba06d33d65fa0fabffb34a0df769f38d0
@ezyang ezyang added the merged label Jun 26, 2019
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