[PATCH] D144958: [SLP]Initial support for reshuffling of non-starting buildvector/gather nodes.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 17 16:32:45 PDT 2023


ABataev added a comment.

In D144958#4203422 <https://reviews.llvm.org/D144958#4203422>, @rupprecht wrote:

> We're still seeing issues with this at trunk, even after the followup assertion fix in 874c49f55454cb285282e6d184f809945c0beca1 <https://reviews.llvm.org/rG874c49f55454cb285282e6d184f809945c0beca1>.
>
> One strange thing I noticed is an entire method that now gets replaced w/ `unreachable`. This may just be a bug in `SimplifyCFGPass` that this patch triggers: https://godbolt.org/z/1G6PTd6os
>
> (That method isn't the one that's causing problems for us, but maybe it has the same root cause)

Thanks for the reproducer, will check it on Monday.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144958



More information about the llvm-commits mailing list