Skip to content

Move IdWrapper to ATen/core#10152

Closed
ezyang wants to merge 2 commits intopytorch:masterfrom
ezyang:export-D9123652
Closed

Move IdWrapper to ATen/core#10152
ezyang wants to merge 2 commits intopytorch:masterfrom
ezyang:export-D9123652

Conversation

@ezyang
Copy link
Contributor

@ezyang ezyang commented Aug 2, 2018

Summary:

  • Moved from namespace c10::guts to at
  • I fixed the use sites, since there were only three of them
  • Macro renamed from C10_ to AT_

Reviewed By: smessmer

Differential Revision: D9123652

ezyang added 2 commits August 1, 2018 18:51
Summary:
We want CaffeTypeId to be interconvertible with at::ScalarType, and
this means we should have the numbers line up exactly.  Fortunately
this is not too hard to do.

Differential Revision: D9123058

fbshipit-source-id: 5f5946606e5b1f39bebb8030837df70d09307768
Summary:
- Moved from namespace c10::guts to at
- I fixed the use sites, since there were only three of them
- Macro renamed from C10_ to AT_

Reviewed By: smessmer

Differential Revision: D9123652

fbshipit-source-id: d58f38c2b463257ea904cfc8e7cf09c44174a243
zdevito pushed a commit to zdevito/ATen that referenced this pull request Aug 2, 2018
Summary:
Pull Request resolved: pytorch/pytorch#10152

- Moved from namespace c10::guts to at
- I fixed the use sites, since there were only three of them
- Macro renamed from C10_ to AT_

Reviewed By: smessmer

Differential Revision: D9123652

fbshipit-source-id: bef3c0ace046ebadb82ad00ab73371f026749085
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary:
Pull Request resolved: pytorch#10152

- Moved from namespace c10::guts to at
- I fixed the use sites, since there were only three of them
- Macro renamed from C10_ to AT_

Reviewed By: smessmer

Differential Revision: D9123652

fbshipit-source-id: bef3c0ace046ebadb82ad00ab73371f026749085
@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.

1 participant