projects / platform / upstream / llvm.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: d159c37) | patch
[mlir][sparse] Simplifying Merger::expContainsTensor
authorwren romano <2998727+wrengr@users.noreply.github.com>
Wed, 29 Mar 2023 23:36:14 +0000 (16:36 -0700)
committerwren romano <2998727+wrengr@users.noreply.github.com>
Fri, 7 Apr 2023 19:32:47 +0000 (12:32 -0700)
commit686ef4b4a8dd0af5639ebac9edb491386a3b6932
tree1bf0a6d693f9df84d5cc39f7d0c8bc7c10fb2a7ctree | snapshot
parentd159c37235b1ca191cfa1cf43039710307abea6fcommit | diff
[mlir][sparse] Simplifying Merger::expContainsTensor

`expContainsTensor` used to call `expIsTensor` to short-circuit the recursive calls; however, the very first thing `expContainsTensor` does is to check `expIsTensor`, so the short-circuiting code just causes the function to check that condition redundantly.

Depends On D146684

Reviewed By: aartbik

Differential Revision: https://reviews.llvm.org/D146688
mlir/lib/Dialect/SparseTensor/Utils/Merger.cpp diff | blob | history
Domain: System / Toolchain;