[PATCH] D56875: [DAGCombiner] narrow vector binop with 2 insert subvector operands

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 21 14:40:18 PST 2019


spatel updated this revision to Diff 182830.
spatel added a comment.
This revision is now accepted and ready to land.

Patch updated:

1. Compute the constant vector that we're inserting into rather than assuming it is undef.
2. Moved code comment as suggested.
3. Rebased with diffs for 'xor' tests.


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

https://reviews.llvm.org/D56875

Files:
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  test/CodeGen/X86/avx512-hadd-hsub.ll
  test/CodeGen/X86/scalarize-fp.ll
  test/CodeGen/X86/vector-partial-undef.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D56875.182830.patch
Type: text/x-patch
Size: 10142 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190121/26b59c8f/attachment.bin>


More information about the llvm-commits mailing list