[mlir][sparse] Moving lexOrder from SparseTensorCOO to Element
authorwren romano <2998727+wrengr@users.noreply.github.com>
Tue, 22 Mar 2022 00:39:15 +0000 (17:39 -0700)
committerwren romano <2998727+wrengr@users.noreply.github.com>
Tue, 22 Mar 2022 20:07:05 +0000 (13:07 -0700)
commit110295ebb76150887a5a83733d7ddcf8506da9ad
treec83337d1a6b0a2216e8a4de4877ab03bbdb2ba37
parent3de6b1ce0dd1d7358c75f1bbafc6ee6e70a51875
[mlir][sparse] Moving lexOrder from SparseTensorCOO to Element

This is the more logical place for the function to live.  If/when we factor out a separate class for just the `Coordinates` themselves, then the definition should be moved to `Coordinates::lexOrder` (and `Element::lexOrder` would become a thin wrapper delegating to that function).

This is (tangentially) work towards fixing: https://github.com/llvm/llvm-project/issues/51652

Reviewed By: aartbik

Differential Revision: https://reviews.llvm.org/D122057
mlir/lib/ExecutionEngine/SparseTensorUtils.cpp