[PATCH] D63326: [X86] Replace any_extend* vector extensions with zero_extend* equivalents

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 14 03:47:01 PDT 2019


RKSimon created this revision.
RKSimon added reviewers: craig.topper, spatel, lebedev.ri.
Herald added a project: LLVM.

First step toward addressing the vector-reduce-mul-widen.ll regression in D63281 <https://reviews.llvm.org/D63281> - we should replace ANY_EXTEND/ANY_EXTEND_VECTOR_INREG in X86ISelDAGToDAG to avoid having to add duplicate patterns when treating any extensions as legal.

In future patches this will also allow us to keep any extension nodes around a lot longer in the DAG, which should mean that we can keep better track of undef elements that otherwise become zeros that we think we have to keep......

Would you guys prefer I keep SS41I_pmovx_avx2_patterns_base/SS41I_pmovx_avx2_patterns separate for now or merge them as part of this patch?


Repository:
  rL LLVM

https://reviews.llvm.org/D63326

Files:
  lib/Target/X86/X86ISelDAGToDAG.cpp
  lib/Target/X86/X86InstrAVX512.td
  lib/Target/X86/X86InstrSSE.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63326.204735.patch
Type: text/x-patch
Size: 4335 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190614/090db969/attachment.bin>


More information about the llvm-commits mailing list