[PATCH] D72224: [LegalizeVectorOps] Improve handling of multi-result operations.

[Craig Topper via Phabricator via llvm-commits]

craig.topper updated this revision to Diff 236881.
craig.topper added a comment.

Add a LowerOperationWrapper helper that updates a Result vector. It looks a lot like X86's override of TargetLowering::LowerOperationWrapper except the bool result. The TargetLowering version doesn't allow for the possibility of keeping the existing node since it only called by the type legalizer today. I think in the future we could try to merge this together, but it will require changes to other targets.

Pass Results vector into the overflow operation expanders instead of returning std::pair. I plan to pass Results to all expanders in a follow up. Or possibly inline some of the smaller functions into the switch.


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

https://reviews.llvm.org/D72224

Files:
  llvm/lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp
  llvm/test/CodeGen/X86/avx512-cmp.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D72224.236881.patch
Type: text/x-patch
Size: 30803 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200108/aa2f2bbb/attachment.bin>