[PATCH] D45315: [X86] Improve unsigned saturation downconvert detection.

Artur Gainullin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 14 07:23:17 PDT 2018


ArturGainullin added a comment.

Simon, thanks! I agree that operands have same type. Do you think that it is better to create new node?   Or the latest uploaded version where I update the operand of smax is acceptable?

  DAG.UpdateNodeOperands(In.getNode(), SMin, In.getOperand(1));
  return In;


Repository:
  rL LLVM

https://reviews.llvm.org/D45315





More information about the llvm-commits mailing list