Merge ivalue::Future's markCompleted and markCompletedWithDataPtrs (#56512)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/56512
I don't know if there was a reason to keep them separate, but since the former deferred to the latter, it seems to me that we can get the exact same behavior by merging them and making the `data_ptrs` argument optional (by giving it a default value).
ghstack-source-id: 127035767
Test Plan: Unit tests
Reviewed By: mrshenli
Differential Revision: D27861069
fbshipit-source-id: 93a49d6959b65a8d4ab9b31accce90bf30cd441e