caio/clang-p2996
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[mlir][tensor] Fix gcc build (ValueBoundsOpInterface)

Browse Source 
The order of evaluation of a sum (e.g., `a() + b()`) is unspecified in
C++. clang evaluates left-to-right. gcc evaluate right-to-left. This led
to slighly different (but equivalent) affine_map in a test and the
FileCheck did not match anymore.
This commit is contained in:
Matthias Springer
2023-04-07 10:33:13 +09:00
parent 5e2afe5c66
commit 06d0cd1e81
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
mlir/lib/Dialect/Tensor/IR/ValueBoundsOpInterfaceImpl.cpp
View File

@@ -86,10 +86,10 @@ struct PadOpInterface
auto padOp = cast<PadOp>(op);
assert(value == padOp.getResult() && "invalid value");
AffineExpr expr = cstr.getExpr(padOp.getSource(), dim) +
cstr.getExpr(padOp.getMixedLowPad()[dim]) +
cstr.getExpr(padOp.getMixedHighPad()[dim]);
cstr.bound(value)[dim] == expr;
AffineExpr srcSize = cstr.getExpr(padOp.getSource(), dim);
AffineExpr lowPad = cstr.getExpr(padOp.getMixedLowPad()[dim]);
AffineExpr highPad = cstr.getExpr(padOp.getMixedHighPad()[dim]);
cstr.bound(value)[dim] == srcSize + lowPad + highPad;
}
};