pytorch
a649d808 - Added fast path in the case of no hooks (#52576)

Commit
3 years ago
Added fast path in the case of no hooks (#52576) Summary: See the discussion here: https://github.com/pytorch/pytorch/pull/50431 ~~Not completely done yet - need to figure out the backwards compatibility stuff as well as `RemovableHandle`.~~ ~~Also, this concretely breaks Torchscript (which tries to script the properties), and more generally, probably requires modifying Torchscript hook support: https://github.com/pytorch/pytorch/issues/34329~~ Just kidding, I think all problems are solved :) Another thing I could do in this PR is to simply replace all the `len(x) > 0` checks with the faster checks. That's about 1.5-2k more Python instructions and .4 - .5 microseconds slower. Pull Request resolved: https://github.com/pytorch/pytorch/pull/52576 Reviewed By: ailzhang Differential Revision: D26650352 Pulled By: Chillee fbshipit-source-id: 0fd73e916354b9e306701a8a396c5dc051e69f0d
Author
Parents
Loading