The end-to-end test for this new feature also exposed a bug
in LLVM IR lowering (since then, fixed), where we need to account
for the min-poison bit as extra argument.
declare i32 @llvm.abs.i32(i32 <src>, i1 <is_int_min_poison>)
Paths
| Differential D131712
[mlir][sparse] enable integral abs recognition ClosedPublic Authored by aartbik on Aug 11 2022, 12:28 PM.
Details Summary The end-to-end test for this new feature also exposed a bug declare i32 @llvm.abs.i32(i32 <src>, i1 <is_int_min_poison>)
Diff Detail
Event TimelineThis revision is now accepted and ready to land.Aug 11 2022, 3:59 PM This revision was landed with ongoing or failed builds.Aug 12 2022, 11:36 AM Closed by commit rG8dd07e36ca0f: [mlir][sparse] enable integral abs recognition (authored by aartbik). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 452241 mlir/include/mlir/Dialect/SparseTensor/Utils/Merger.h
mlir/lib/Dialect/SparseTensor/Utils/Merger.cpp
mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_abs.mlir
mlir/unittests/Dialect/SparseTensor/MergerTest.cpp
|