[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