[PATCH] D57348: [CodeGen][X86] Don't scalarize vector saturating add/sub

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 28 13:10:55 PST 2019


RKSimon added a comment.

This might be a little premature as we haven't got PR40442 yet.



================
Comment at: lib/CodeGen/SelectionDAG/TargetLowering.cpp:5325
-         "Expected operands to be integers. Vector of int arguments should "
-         "already be unrolled.");
   assert(LHS.getValueType() == RHS.getValueType() &&
----------------
You can keep this but just change to isInteger()?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57348





More information about the llvm-commits mailing list