-
Notifications
You must be signed in to change notification settings - Fork 13.9k
[mlir][sparse] implement sparse space collapse pass. #89003
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,199 @@ | ||
//===--------- SparseSpaceCollapse.cpp - Collapse Sparse Space Pass -------===// | ||
// | ||
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. | ||
// See https://llvm.org/LICENSE.txt for license information. | ||
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception | ||
// | ||
//===----------------------------------------------------------------------===// | ||
|
||
#include "mlir/Dialect/Func/IR/FuncOps.h" | ||
#include "mlir/IR/IRMapping.h" | ||
#include "mlir/Transforms/Passes.h" | ||
|
||
#include "mlir/Dialect/SparseTensor/IR/SparseTensor.h" | ||
#include "mlir/Dialect/SparseTensor/Transforms/Passes.h" | ||
|
||
namespace mlir { | ||
#define GEN_PASS_DEF_SPARSESPACECOLLAPSE | ||
#include "mlir/Dialect/SparseTensor/Transforms/Passes.h.inc" | ||
} // namespace mlir | ||
|
||
#define DEBUG_TYPE "sparse-space-collapse" | ||
|
||
using namespace mlir; | ||
using namespace sparse_tensor; | ||
|
||
namespace { | ||
|
||
struct CollapseSpaceInfo { | ||
ExtractIterSpaceOp space; | ||
IterateOp loop; | ||
}; | ||
|
||
bool isCollapsableLoops(LoopLikeOpInterface parent, LoopLikeOpInterface node) { | ||
auto pIterArgs = parent.getRegionIterArgs(); | ||
auto nInitArgs = node.getInits(); | ||
if (pIterArgs.size() != nInitArgs.size()) | ||
return false; | ||
|
||
// Two loops are collapsable if they are perfectly nested. | ||
auto pYields = parent.getYieldedValues(); | ||
auto nResult = node.getLoopResults().value(); | ||
|
||
bool yieldEq = | ||
llvm::all_of(llvm::zip_equal(pYields, nResult), [](auto zipped) { | ||
return std::get<0>(zipped) == std::get<1>(zipped); | ||
}); | ||
|
||
// Parent iter_args should be passed directly to the node's init_args. | ||
bool iterArgEq = | ||
llvm::all_of(llvm::zip_equal(pIterArgs, nInitArgs), [](auto zipped) { | ||
return std::get<0>(zipped) == std::get<1>(zipped); | ||
}); | ||
|
||
return yieldEq && iterArgEq; | ||
} | ||
|
||
bool legalToCollapse(SmallVectorImpl<CollapseSpaceInfo> &toCollapse, | ||
ExtractIterSpaceOp curSpace) { | ||
|
||
auto getIterateOpOverSpace = [](ExtractIterSpaceOp space) -> IterateOp { | ||
Value spaceVal = space.getExtractedSpace(); | ||
if (spaceVal.hasOneUse()) | ||
return llvm::dyn_cast<IterateOp>(*spaceVal.getUsers().begin()); | ||
return nullptr; | ||
}; | ||
|
||
if (toCollapse.empty()) { | ||
// Collapse root. | ||
if (auto itOp = getIterateOpOverSpace(curSpace)) { | ||
CollapseSpaceInfo &info = toCollapse.emplace_back(); | ||
info.space = curSpace; | ||
info.loop = itOp; | ||
return true; | ||
} | ||
return false; | ||
} | ||
|
||
auto parent = toCollapse.back().space; | ||
auto pItOp = toCollapse.back().loop; | ||
auto nItOp = getIterateOpOverSpace(curSpace); | ||
|
||
// Can only collapse spaces extracted from the same tensor. | ||
if (parent.getTensor() != curSpace.getTensor()) { | ||
LLVM_DEBUG({ | ||
llvm::dbgs() | ||
<< "failed to collpase spaces extracted from different tensors."; | ||
}); | ||
return false; | ||
PeimingLiu marked this conversation as resolved.
Show resolved
Hide resolved
|
||
} | ||
|
||
// Can only collapse consecutive simple iteration on one tensor (i.e., no | ||
// coiteration). | ||
if (!nItOp || nItOp->getBlock() != curSpace->getBlock() || | ||
pItOp.getIterator() != curSpace.getParentIter() || | ||
curSpace->getParentOp() != pItOp.getOperation()) { | ||
LLVM_DEBUG( | ||
{ llvm::dbgs() << "failed to collapse non-consecutive IterateOps."; }); | ||
return false; | ||
} | ||
|
||
if (pItOp && !isCollapsableLoops(pItOp, nItOp)) { | ||
LLVM_DEBUG({ | ||
llvm::dbgs() | ||
<< "failed to collapse IterateOps that are not perfectly nested."; | ||
}); | ||
return false; | ||
} | ||
|
||
CollapseSpaceInfo &info = toCollapse.emplace_back(); | ||
info.space = curSpace; | ||
info.loop = nItOp; | ||
return true; | ||
} | ||
|
||
void collapseSparseSpace(MutableArrayRef<CollapseSpaceInfo> toCollapse) { | ||
if (toCollapse.size() < 2) | ||
return; | ||
|
||
ExtractIterSpaceOp root = toCollapse.front().space; | ||
ExtractIterSpaceOp leaf = toCollapse.back().space; | ||
Location loc = root.getLoc(); | ||
|
||
assert(root->hasOneUse() && leaf->hasOneUse()); | ||
|
||
// Insert collapsed operation at the same scope as root operation. | ||
OpBuilder builder(root); | ||
|
||
// Construct the collapsed iteration space. | ||
auto collapsedSpace = builder.create<ExtractIterSpaceOp>( | ||
loc, root.getTensor(), root.getParentIter(), root.getLoLvl(), | ||
leaf.getHiLvl()); | ||
|
||
auto rItOp = llvm::cast<IterateOp>(*root->getUsers().begin()); | ||
auto innermost = toCollapse.back().loop; | ||
|
||
IRMapping mapper; | ||
mapper.map(leaf, collapsedSpace.getExtractedSpace()); | ||
for (auto z : llvm::zip_equal(innermost.getInitArgs(), rItOp.getInitArgs())) | ||
mapper.map(std::get<0>(z), std::get<1>(z)); | ||
|
||
auto cloned = llvm::cast<IterateOp>(builder.clone(*innermost, mapper)); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It seems like If that op were removed, you could also avoid the clone and, instead, create a new op and move the body region there, which would be cheaper... There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. since |
||
builder.setInsertionPointToStart(cloned.getBody()); | ||
|
||
LevelSet crdUsedLvls; | ||
unsigned shift = 0, argIdx = 1; | ||
for (auto info : toCollapse.drop_back()) { | ||
LevelSet set = info.loop.getCrdUsedLvls(); | ||
crdUsedLvls |= set.lshift(shift); | ||
shift += info.loop.getSpaceDim(); | ||
for (BlockArgument crd : info.loop.getCrds()) { | ||
BlockArgument collapsedCrd = cloned.getBody()->insertArgument( | ||
argIdx++, builder.getIndexType(), crd.getLoc()); | ||
crd.replaceAllUsesWith(collapsedCrd); | ||
} | ||
} | ||
crdUsedLvls |= innermost.getCrdUsedLvls().lshift(shift); | ||
cloned.getIterator().setType(collapsedSpace.getType().getIteratorType()); | ||
cloned.setCrdUsedLvls(crdUsedLvls); | ||
|
||
rItOp.replaceAllUsesWith(cloned.getResults()); | ||
// Erase collapsed loops. | ||
rItOp.erase(); | ||
root.erase(); | ||
} | ||
|
||
struct SparseSpaceCollapsePass | ||
: public impl::SparseSpaceCollapseBase<SparseSpaceCollapsePass> { | ||
SparseSpaceCollapsePass() = default; | ||
|
||
void runOnOperation() override { | ||
func::FuncOp func = getOperation(); | ||
|
||
// A naive (experimental) implementation to collapse consecutive sparse | ||
// spaces. It does NOT handle complex cases where multiple spaces are | ||
// extracted in the same basic block. E.g., | ||
// | ||
// %space1 = extract_space %t1 ... | ||
// %space2 = extract_space %t2 ... | ||
// sparse_tensor.iterate(%sp1) ... | ||
// | ||
SmallVector<CollapseSpaceInfo> toCollapse; | ||
func->walk([&](ExtractIterSpaceOp op) { | ||
PeimingLiu marked this conversation as resolved.
Show resolved
Hide resolved
|
||
if (!legalToCollapse(toCollapse, op)) { | ||
// if not legal to collapse one more space, collapse the existing ones | ||
// and clear. | ||
collapseSparseSpace(toCollapse); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I am wondering whether this couldn't be a pattern. I guess the answer is: currently no because we want to collect all collapsible loops in one go. Eventually, you might want to be able to merge an already collapsed iteration with the next collapsible level. In that case, you could solve the special case of only non-collapsed ops by repeatedly applying the previous logic. Maybe that could work as patterns then...? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You are correct, we can probably transfer it into a pattern. But I would also like to collapse all eligible loops in one run. |
||
toCollapse.clear(); | ||
} | ||
}); | ||
|
||
collapseSparseSpace(toCollapse); | ||
} | ||
}; | ||
|
||
} // namespace | ||
|
||
std::unique_ptr<Pass> mlir::createSparseSpaceCollapsePass() { | ||
return std::make_unique<SparseSpaceCollapsePass>(); | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,37 @@ | ||
// RUN: mlir-opt %s --sparse-space-collapse | FileCheck %s | ||
|
||
#COO = #sparse_tensor.encoding<{ | ||
map = (i, j) -> ( | ||
i : compressed(nonunique), | ||
j : singleton(soa) | ||
) | ||
}> | ||
|
||
// CHECK-LABEL: func.func @sparse_sparse_collapse( | ||
// CHECK-SAME: %[[VAL_0:.*]]: tensor<4x8xf32, #sparse>, | ||
// CHECK-SAME: %[[VAL_1:.*]]: index) { | ||
// CHECK: %[[VAL_3:.*]] = sparse_tensor.extract_iteration_space %[[VAL_0]] lvls = 0 to 2 : tensor<4x8xf32, #sparse> | ||
// CHECK: %[[VAL_4:.*]] = sparse_tensor.iterate %[[VAL_5:.*]] in %[[VAL_3]] at(%[[VAL_6:.*]], _) iter_args(%[[VAL_7:.*]] = %[[VAL_1]]) | ||
// CHECK: %[[VAL_8:.*]] = "test.op"(%[[VAL_7]]) : (index) -> index | ||
// CHECK: sparse_tensor.yield %[[VAL_8]] : index | ||
// CHECK: } | ||
// CHECK: "test.sink"(%[[VAL_4]]) : (index) -> () | ||
// CHECK: return | ||
// CHECK: } | ||
func.func @sparse_sparse_collapse(%sp : tensor<4x8xf32, #COO>, %i : index) { | ||
%l1 = sparse_tensor.extract_iteration_space %sp lvls = 0 | ||
: tensor<4x8xf32, #COO> | ||
-> !sparse_tensor.iter_space<#COO, lvls = 0> | ||
%r1 = sparse_tensor.iterate %it1 in %l1 at(%crd0) iter_args(%outer = %i): !sparse_tensor.iter_space<#COO, lvls = 0 to 1> -> index { | ||
%l2 = sparse_tensor.extract_iteration_space %sp at %it1 lvls = 1 | ||
: tensor<4x8xf32, #COO>, !sparse_tensor.iterator<#COO, lvls = 0 to 1> | ||
-> !sparse_tensor.iter_space<#COO, lvls = 1> | ||
%r2 = sparse_tensor.iterate %it2 in %l2 iter_args(%inner = %outer): !sparse_tensor.iter_space<#COO, lvls = 1 to 2> -> index { | ||
%k ="test.op"(%inner) : (index) -> index | ||
sparse_tensor.yield %k : index | ||
} | ||
sparse_tensor.yield %r2 : index | ||
} | ||
"test.sink"(%r1) : (index) -> () | ||
return | ||
} | ||
PeimingLiu marked this conversation as resolved.
Show resolved
Hide resolved
|
Uh oh!
There was an error while loading. Please reload this page.