Skip to content

Move TensorTypeId, TensorTypeIdRegistration and flat_hash_map to ATen/core#10263

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

Move TensorTypeId, TensorTypeIdRegistration and flat_hash_map to ATen/core#10263
ezyang wants to merge 2 commits intopytorch:masterfrom
ezyang:export-D9181321

Conversation

@ezyang
Copy link
Contributor

@ezyang ezyang commented Aug 6, 2018

Summary:
Auxiliary changes that were needed:

  • Add DISABLE_COPY_AND_ASSIGN to CoreAPI.h (maybe we should rename this file
    now)

Differential Revision: D9181321

ezyang added 2 commits August 6, 2018 13:30
Differential Revision: D9157408

fbshipit-source-id: 7da24258ffdc7e37359f55e6ec90607d448ef6bb
…/core (pytorch#10263)

Summary:
Pull Request resolved: pytorch#10263

Auxiliary changes that were needed:
- Add DISABLE_COPY_AND_ASSIGN to CoreAPI.h (maybe we should rename this file
  now)

Differential Revision: D9181321

fbshipit-source-id: f2fe619cf9be50517cb1d66c351209646597f0c4
@ezyang
Copy link
Contributor Author

ezyang commented Aug 6, 2018

This commit revealed that ska flat_hash_map doesn't (1) work on Android, (2) work on PyTorch Windows

Since tensor type ID is not a perf crit path, I'm just going to get rid of the dependence of TensorTypeId on flat_hash_map, but we might need to get rid of flat_hash_map entirely if no one is willing to fix it to be more portable.

CC @smessmer

zdevito pushed a commit to zdevito/ATen that referenced this pull request Aug 7, 2018
…/core (#10263)

Summary:
Pull Request resolved: pytorch/pytorch#10263

Auxiliary changes that were needed:
- Add DISABLE_COPY_AND_ASSIGN to CoreAPI.h (maybe we should rename this file
  now)

Reviewed By: dzhulgakov

Differential Revision: D9181321

fbshipit-source-id: 975687068285b5a94a57934817c960aeea2bbafa
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
…/core (pytorch#10263)

Summary:
Pull Request resolved: pytorch#10263

Auxiliary changes that were needed:
- Add DISABLE_COPY_AND_ASSIGN to CoreAPI.h (maybe we should rename this file
  now)

Reviewed By: dzhulgakov

Differential Revision: D9181321

fbshipit-source-id: 975687068285b5a94a57934817c960aeea2bbafa
@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