[PATCH] D19922: [AArch64] Fix DAG selection for cmps for fp16 type

[Weiming Zhao via llvm-commits]

weimingz added a comment.

Thank you, Tim! And do we need to guard the promotion of fp16->fp32 with !HasFullFP16() check? If some subtarget has full  fp16 support, the promotion is not needed, is it?


================
Comment at: lib/Target/AArch64/AArch64ISelLowering.cpp:1412
@@ -1411,3 +1411,3 @@
     SDValue LHS = Val->getOperand(0);
     SDValue RHS = Val->getOperand(1);
     ISD::CondCode CC = cast<CondCodeSDNode>(Val->getOperand(2))->get();
----------------
Operands get from a chained SETCC


Repository:
  rL LLVM

http://reviews.llvm.org/D19922