[PATCH] D40209: [DAGCombiner] eliminate shuffle of insert element

[Sanjay Patel via Phabricator via llvm-commits]

spatel updated this revision to Diff 124484.
spatel added a comment.

Patch updated:
Added a code comment to explain why we shouldn't need a legality check or hook to perform this fold. Not sure if that's sufficient. Let me know if there's another place or way to document the logic. Thanks!


https://reviews.llvm.org/D40209

Files:
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  test/CodeGen/X86/shuffle-of-insert.ll
  test/CodeGen/X86/sse41.ll
  test/CodeGen/X86/vector-shuffle-128-v2.ll
  test/CodeGen/X86/vector-shuffle-combining.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D40209.124484.patch
Type: text/x-patch
Size: 20307 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171127/8777a3fa/attachment.bin>