[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