[PATCH] D55655: [DAGCombiner] after simplifying demanded elements of vector operand of extract, revisit the extract

[Sanjay Patel via Phabricator via llvm-commits]

spatel created this revision.
spatel added reviewers: andreadb, craig.topper, RKSimon.
Herald added a subscriber: mcrosier.

I noticed this as part of a patch for PR39973:
https://bugs.llvm.org/show_bug.cgi?id=39973

And there might be some interplay with D55600 <https://reviews.llvm.org/D55600>, but I haven't checked yet. I think this bug is unique to extract_element, but I'll do an audit to be sure.

These are all wins or neutral from what I can tell.


https://reviews.llvm.org/D55655

Files:
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  test/CodeGen/X86/extractelement-load.ll
  test/CodeGen/X86/insertps-combine.ll
  test/CodeGen/X86/masked_load.ll
  test/CodeGen/X86/masked_store.ll
  test/CodeGen/X86/shrink_vmul-widen.ll
  test/CodeGen/X86/vec_extract-sse4.ll
  test/CodeGen/X86/vector-reduce-mul-widen.ll
  test/CodeGen/X86/vector-reduce-mul.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55655.178063.patch
Type: text/x-patch
Size: 10953 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181213/9af9e707/attachment.bin>