[PATCH] D135876: [Instcombine] Scalarize vscale splats of vectorized BinOps

Matt Devereau via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 19 08:15:32 PDT 2022


MattDevereau updated this revision to Diff 468921.
MattDevereau retitled this revision from "[VectorCombine] Scalarize binops with insertelt nested inside splats" to "[Instcombine] Scalarize vscale splats of vectorized BinOps".
MattDevereau edited the summary of this revision.
MattDevereau added a comment.
Herald added a subscriber: nlopes.

Moved VectorCombine tests that are no longer necessary into InstCombine
Deleted all logic from VectorCombine, and have instead moved BinOp scalarizing logic into InstCombine. I've only seen this pattern emerge in InstCombine for scalable vectors, so this InstCombine is currently guarded by a scalable vector check.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135876

Files:
  llvm/lib/Transforms/InstCombine/InstCombineInternal.h
  llvm/lib/Transforms/InstCombine/InstCombineVectorOps.cpp
  llvm/test/Transforms/InstCombine/vscale-insert-shuffle-binop.ll
  llvm/test/Transforms/VectorCombine/AArch64/insert-shuffle-binop.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D135876.468921.patch
Type: text/x-patch
Size: 27727 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221019/c50cd492/attachment.bin>


More information about the llvm-commits mailing list