[PATCH] D72139: [ARM] Use isFMAFasterThanFMulAndFAdd for scalars as well as MVE vectors

[Sam Parker via Phabricator via llvm-commits]

samparker added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:15024
+  case MVT::f16:
+    return Subtarget->hasFullFP16() && !Subtarget->isTargetDarwin() &&
+           Subtarget->useFPVFMx();
----------------
Is there a way this logic can sit in Subtarget to avoid it being a tablegen predicate as well as code here? I'm hopeless with our FP architectures... does FullFP16 infer VFP4?


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

https://reviews.llvm.org/D72139