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

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added a comment.

In D101109#3218469 <https://reviews.llvm.org/D101109#3218469>, @ye-luo wrote:

> 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.

Ok, thanks for letting me know!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101109