[PATCH] D71850: [X86] Add STRICT versions of CVTTP2SI, CVTTP2UI, CMPM, and CMPP.

Pengfei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 23 21:49:45 PST 2019


pengfei added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ISelDAGToDAG.cpp:546-549
     // The first operand of X86ISD::CMPM is chain, so we need to get the second
     // operand.
     if (Opcode == X86ISD::CMPM)
       OpVT = N->getOperand(1).getValueType();
----------------
This can be removed. Do we need handle strict cases?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71850





More information about the llvm-commits mailing list