[PATCH] D102834: [SLPVectorizer] Implement initial memory versioning.

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added a comment.

In D102834#2940056 <https://reviews.llvm.org/D102834#2940056>, @fhahn wrote:

> In D102834#2919138 <https://reviews.llvm.org/D102834#2919138>, @SjoerdMeijer wrote:
>
>> Sorry for the late reply, but just wanted to confirm this: yes, that `@f_alias` in `../AArch64/loadi8.ll` is a reproducer from x264.
>
> Great thanks. The code in the test should be transformed now. If you point me to the C code, I can check if it is transformed as expected now.

I believe that was function `mc_weight_w20()`.

And like I mentioned earlier, I expect this change to make quite some differences for x264. For example, I hope it will trigger on `quant_4x4()` too, although additional trick may be required for successful SLP vectorisation of that example (cost-model and or other things).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102834