squiid/llvm
1
0
Fork 0
Code Issues Pull requests Activity

[mlir][sparse] fix unsigned comparison bug in assert

Browse source 
Reviewed By: bixia, wrengr

Differential Revision: https://reviews.llvm.org/D126007
This commit is contained in:
Aart Bik 2022-05-19 12:23:52 -07:00
parent 9ece051847
commit 3b13f8805c
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
mlir/lib/ExecutionEngine/SparseTensorUtils.cpp
View file

@ -885,7 +885,7 @@ private:
const uint64_t pstop = static_cast<uint64_t>(pointers_d[parentPos + 1]);
// Loop-invariant code for looking up the `d`-level coordinates/indices.
const std::vector<I> &indices_d = src.indices[d];
assert(pstop - 1 < indices_d.size() && "Index position is out of bounds");
assert(pstop <= indices_d.size() && "Index position is out of bounds");
uint64_t &cursor_reord_d = this->cursor[this->reord[d]];
for (uint64_t pos = pstart; pos < pstop; pos++) {
cursor_reord_d = static_cast<uint64_t>(indices_d[pos]);