[PATCH] D150941: [LegalizeType][X86] Support WidenVecRes_AssertZext and SplitVecRes_AssertZext for ISD::AssertZext during LegalizeType procedure

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp:5131
 
+SDValue DAGTypeLegalizer::WidenVecRes_AssertZext(SDNode *N) {
+  SDValue InOp = GetWidenedVector(N->getOperand(0));
----------------
craig.topper wrote:
> The extra elements added by widening aren’t guaranteed to satisfy the Asserted property.
You can probably use ModifyToType to fill the new elements with zero


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150941