Skip to content

Deduplicate THTensor and THCTensor.#9495

Closed
ezyang wants to merge 1 commit intopytorch:masterfrom
ezyang:pr/merge-thtensor-thctensor
Closed

Deduplicate THTensor and THCTensor.#9495
ezyang wants to merge 1 commit intopytorch:masterfrom
ezyang:pr/merge-thtensor-thctensor

Conversation

@ezyang
Copy link
Contributor

@ezyang ezyang commented Jul 17, 2018

Stacked on #9494

commit fc24fae89b66a4d21dc3d0c78e29379d9db2e571 (HEAD -> pytorch-tmp, ezyang/pr/merge-thtensor-thctensor)
Author: Edward Z. Yang <ezyang@fb.com>
Date:   Tue Jul 17 08:17:35 2018 -0700

    Deduplicate THTensor and THCTensor.
    
    This is enabled by the allocator patch; previously we could not
    deduplicate THStorage_free/THCStorage_free; now we can.
    
    Signed-off-by: Edward Z. Yang <ezyang@fb.com>

@ezyang ezyang requested a review from gchanan July 17, 2018 15:20
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.

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

This is enabled by the allocator patch; previously we could not
deduplicate THStorage_free/THCStorage_free; now we can.

Signed-off-by: Edward Z. Yang <ezyang@fb.com>
@ezyang ezyang force-pushed the pr/merge-thtensor-thctensor branch from fc24fae to c4d25ce Compare July 17, 2018 20:33
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.

@ezyang 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 Jul 18, 2018
Summary:
This is enabled by the allocator patch; previously we could not
deduplicate THStorage_free/THCStorage_free; now we can.

Pull Request resolved: pytorch/pytorch#9495

Reviewed By: SsnL

Differential Revision: D8875497

Pulled By: ezyang

fbshipit-source-id: 387198dff446eb9f84d2d6187066fae1d595dea7
facebook-github-bot pushed a commit that referenced this pull request Jul 18, 2018
Summary:
Stacked on #9495
Pull Request resolved: #9496

Differential Revision: D8875528

Pulled By: ezyang

fbshipit-source-id: 6419d2ffb07aaf49c1462e7b64737019abbb7f61
jramseyer pushed a commit to jramseyer/pytorch that referenced this pull request Jul 30, 2018
Summary:
This is enabled by the allocator patch; previously we could not
deduplicate THStorage_free/THCStorage_free; now we can.

Pull Request resolved: pytorch#9495

Reviewed By: SsnL

Differential Revision: D8875497

Pulled By: ezyang

fbshipit-source-id: 387198dff446eb9f84d2d6187066fae1d595dea7
jramseyer pushed a commit to jramseyer/pytorch that referenced this pull request Jul 30, 2018
Summary:
Stacked on pytorch#9495
Pull Request resolved: pytorch#9496

Differential Revision: D8875528

Pulled By: ezyang

fbshipit-source-id: 6419d2ffb07aaf49c1462e7b64737019abbb7f61
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary:
This is enabled by the allocator patch; previously we could not
deduplicate THStorage_free/THCStorage_free; now we can.

Pull Request resolved: pytorch#9495

Reviewed By: SsnL

Differential Revision: D8875497

Pulled By: ezyang

fbshipit-source-id: 387198dff446eb9f84d2d6187066fae1d595dea7
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary:
Stacked on pytorch#9495
Pull Request resolved: pytorch#9496

Differential Revision: D8875528

Pulled By: ezyang

fbshipit-source-id: 6419d2ffb07aaf49c1462e7b64737019abbb7f61
@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