[llvm] [SLPVectorizer] Clear `TreeEntryToStridedPtrInfoMap`. (PR #160544)
Mikhail Gudim via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 26 10:51:16 PDT 2025
mgudim wrote:
I am out of ideas.
I don't see how to add asserts to the code so it doesn't look stupid. Current code is like this:
"buildTree(...) {
deleteTree(...);
...
buildTreeRec(...)
}
"
adding asserts after deleteTree is like this code:
```
int x = 2;
assert (x == 2);
```
I thought that maybe I can set up a unit test. There is no public header to include which would contain `BoUpSLP`.
Constructing test by hand to try to make SLP crash is very hard.
There won't be a memory leak because the map gets deleted automatically with `BoUpSLP` object so setting up some kind of valgrind - based test is also not an option.
Can you suggest something else?
https://github.com/llvm/llvm-project/pull/160544
More information about the llvm-commits
mailing list