intel/llvm
1
0
Fork 0
mirror of https://github.com/intel/llvm.git synced 2026-01-23 07:58:23 +08:00
Code Issues Packages Projects Releases Wiki Activity

[mlir][Linalg] Fix warnings/errors on D155518.

Browse Source 
This fixes some warnings (that were caught as errors) in https://reviews.llvm.org/D155518/.

Differential Revision: https://reviews.llvm.org/D155738
This commit is contained in:
Mahesh Ravishankar
2023-07-19 18:45:10 +00:00
parent 09fa41be98
commit 2efe88b50e
2 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
mlir/lib/Dialect/Linalg/Transforms/DropUnitDims.cpp
View File

@@ -361,7 +361,7 @@ static UnitExtentReplacementInfo dropUnitExtentFromOperandMetadata(
return false;
};
int64_t dim = 0;
unsigned dim = 0;
while (dim < operandShape.size() && isUnitDim(dim))
reassociationGroup.push_back(dim++);
while (dim < operandShape.size()) {
@@ -409,7 +409,6 @@ LogicalResult linalg::dropUnitDims(RewriterBase &rewriter, GenericOp genericOp,
llvm::SmallDenseSet<unsigned> unitDimsFilter(allowedUnitDims.begin(),
allowedUnitDims.end());
llvm::SmallDenseSet<unsigned> unitDims;
ArrayAttr iteratorTypes = genericOp.getIteratorTypes();
for (const auto &expr : enumerate(invertedMap.getResults())) {
if (AffineDimExpr dimExpr = expr.value().dyn_cast<AffineDimExpr>()) {
if (dims[dimExpr.getPosition()] == 1 &&

2
mlir/test/lib/Dialect/Linalg/TestLinalgDropUnitDims.cpp
View File

@@ -58,7 +58,7 @@ struct TestLinalgDropUnitDims
for (auto genericOp : genericOps) {
rewriter.setInsertionPoint(genericOp);
dropOutermostUnitDims(rewriter, genericOp);
(void)dropOutermostUnitDims(rewriter, genericOp);
}
}
};