[PATCH] D50191: [ARM] FP16: support VFMA

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 2 14:18:06 PDT 2018


efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: lib/Target/ARM/ARMInstrNEON.td:4742
+          (VFMAhq QPR:$src1, QPR:$Vn, QPR:$Vm)>,
+          Requires<[HasFullFP16]>;
 def : Pat<(v2f32 (fma DPR:$Vn, DPR:$Vm, DPR:$src1)),
----------------
I guess this should also check HasNEON, for completeness? Unlikely to make a difference in practice.


https://reviews.llvm.org/D50191





More information about the llvm-commits mailing list