[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