Delete TensorOptions::operator== #25478
Description
Metadata
Metadata
Assignees
Labels
Not as big of a feature, but technically not a bug. Should be easy to fixRelated to internal abstractions in c10 and ATenWe think this is a small issue to fix. Consider knocking off high priority small issuesThis issue has been looked at a team member, and triaged and prioritized into an appropriate module
We defined an equality test on TensorOptions but we really shouldn't have; the semantics are completely non-obvious.
What we should define instead is probably something like
TensorAxeswhich is the tuple of(DeviceType, Layout, DType)which maps into TensorTypeId and uniquely specifies how dispatch computation happens. (Should Variable show up in here eventually? Probably.)