Skip to content

Use angle brackets instead of quotes for includes.#10153

Closed
ezyang wants to merge 3 commits intopytorch:masterfrom
ezyang:export-D9123768
Closed

Use angle brackets instead of quotes for includes.#10153
ezyang wants to merge 3 commits intopytorch:masterfrom
ezyang:export-D9123768

Conversation

@ezyang
Copy link
Contributor

@ezyang ezyang commented Aug 2, 2018

Reviewed By: smessmer

Differential Revision: D9123768

ezyang added 3 commits August 1, 2018 18:52
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: f169df2f9962598b82f5d5287f15bc3d24ab5da7
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_

Differential Revision: D9123652

fbshipit-source-id: 88c76a0802ed5f92efacce0cd79ed2cb35f544a0
Reviewed By: smessmer

Differential Revision: D9123768

fbshipit-source-id: 2c556c9357f0f233d7130ae97b8ef1af7e8806d7
@ezyang
Copy link
Contributor Author

ezyang commented Aug 2, 2018

@pytorchbot retest this please

2 similar comments
@ezyang
Copy link
Contributor Author

ezyang commented Aug 2, 2018

@pytorchbot retest this please

@ezyang
Copy link
Contributor Author

ezyang commented Aug 2, 2018

@pytorchbot retest this please

zdevito pushed a commit to zdevito/ATen that referenced this pull request Aug 2, 2018
Summary: Pull Request resolved: pytorch/pytorch#10153

Reviewed By: smessmer

Differential Revision: D9123768

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

Reviewed By: smessmer

Differential Revision: D9123768

fbshipit-source-id: 0970552ba4d5772fb3cef2db3af3181d98f85140
@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