[PATCH] D26714: [AMDGPU] Handle f16 select{_cc}

Konstantin Zhuravlyov via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 15 19:26:35 PST 2016


kzhuravl added inline comments.


================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:292
     // F16 - VOP2 Actions.
+    setOperationAction(ISD::SELECT_CC, MVT::f16, Expand);
     setOperationAction(ISD::FMAXNUM, MVT::f16, Legal);
----------------
arsenm wrote:
> Also should have a test for BR_CC, which probably also needs this
Thanks, I will do it in a follow up patch.


Repository:
  rL LLVM

https://reviews.llvm.org/D26714





More information about the llvm-commits mailing list