[PATCH] D101109: [SLP]Improve multinode analysis.

Mon Jan 3 15:21:31 PST 2022

ye-luo added a comment.

In D101109#3213634 <https://reviews.llvm.org/D101109#3213634>, @ABataev wrote:

> In D101109#3213510 <https://reviews.llvm.org/D101109#3213510>, @ye-luo wrote:
>
>> This patch caused many test failure in my application on Power9. Although this patch sounds like affecting SLP, adding -fno-slp-vectorize doesn't improve the pass rate but changing -O3 to -O0 does.
>
> Hi, do you have a reproducer?

Initially I was not sure where the issue is from and just reported my observation. After a careful inspection, I found it is an interaction between clang and the random number generator in the boost libraries. Since I had little knowledge about the inside details of the library, I decided not to debug it. Instead I just moved my application out of boost and the RNG from C++ standard library works well with Clang. So I won't work on an reproducer. If boost developers find an issue, they will report bugs. Right now assume everything is good.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101109/new/

https://reviews.llvm.org/D101109