[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