[PATCH] D107344: [SLP]Do not emit extra shuffle for insertelements vectorization.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 4 11:09:02 PDT 2021


ABataev added a comment.

In D107344#2926201 <https://reviews.llvm.org/D107344#2926201>, @srj wrote:

>> In D107344#2926191 <https://reviews.llvm.org/D107344#2926191>, @saugustine wrote:
>>
>>> As this change is causing miscompiles on halide, I think it wise to revert until fixed. Any objections?
>
> To be fair, it's (potentially) causing miscompiles everywhere; Halide just exercises vectorization a lot more than most LLVM clients :-)
>
> In D107344#2926195 <https://reviews.llvm.org/D107344#2926195>, @ABataev wrote:
>
>> Doing it already, will be reverted soon.
>
> Thanks! Please cc me on the updated patch and I will test it locally to verify that Halide is happy with it.

Sure


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107344



More information about the llvm-commits mailing list