[PATCH] D68237: [PowerPC] Handle f16 as a storage type only

[ChenZheng via Phabricator via llvm-commits]

shchenz added inline comments.


================
Comment at: lib/Target/PowerPC/PPCISelLowering.cpp:184
+    setTruncStoreAction(MVT::f32, MVT::f16, Expand);
+  }
+
----------------
Do we need to handle ppcf128 also? 


================
Comment at: lib/Target/PowerPC/PPCInstrVSX.td:114
                         [SDNPHasChain, SDNPMayLoad, SDNPMemOperand]>;
+def extloadf16 : PatFrag<(ops node:$ptr), (extload node:$ptr)> {
+  let IsLoad = 1;
----------------
Guard under IsISA3_0?


================
Comment at: lib/Target/PowerPC/PPCInstrVSX.td:3263
             (v2i64 (XXPERMDIs (VEXTSH2Ds (LXSIHZX xoaddr:$src)), 0))>;
 
+  // Load/convert and convert/store patterns for f16.
----------------
Guard under IsISA3_0?


================
Comment at: test/CodeGen/PowerPC/handle-f16-storage-type.ll:8
+; Function Attrs: nounwind readonly
+define dso_local double @loadd(i16* nocapture readonly %a) local_unnamed_addr #0 {
+; P8-LABEL: loadd:
----------------
`#0 `, seems all the function attributes are not defined?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D68237