[PATCH] D91084: [SVE][CodeGen] Add the ExtensionType flag to MGATHER

[Kerry McLaughlin via Phabricator via llvm-commits]

kmclaughlin updated this revision to Diff 307656.
kmclaughlin marked 5 inline comments as done.
kmclaughlin added a comment.

- Replaced calls to `TLI.isLoadExtLegal` with `TLI.isVectorLoadExtDesirable` in the masked gather combines, which also checks the type of load operation being used
- Removed unnecessary `!LegalOperations` from the zext_masked_gather combine and added a check for `hasOneUse()`, matching the sext_masked_gather combine


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

https://reviews.llvm.org/D91084

Files:
  llvm/include/llvm/CodeGen/SelectionDAG.h
  llvm/include/llvm/CodeGen/SelectionDAGNodes.h
  llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp
  llvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp
  llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
  llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
  llvm/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp
  llvm/lib/Target/X86/X86ISelLowering.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91084.307656.patch
Type: text/x-patch
Size: 14266 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201125/d2ae062d/attachment.bin>