[PATCH] D100302: [VectorCombine] Run load/extract scalarization after scalarizing store.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 21 07:00:58 PDT 2021


fhahn abandoned this revision.
fhahn added a comment.

In D100302#2950426 <https://reviews.llvm.org/D100302#2950426>, @spatel wrote:

> In D100302#2950411 <https://reviews.llvm.org/D100302#2950411>, @fhahn wrote:
>
>> @spatel did you by any chance already think/consider this for VectorCombine? Or do you have any concerns?
>
> No concerns. This has come up before (we were walking the instruction bottom-up at some point instead of top-down), but we just didn't have enough motivation with the existing transforms to justify implementing a worklist. If you want to try it, that would be great!

I put up a patch: D110171 <https://reviews.llvm.org/D110171>

I'll abandon the patch here for now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100302



More information about the llvm-commits mailing list