[PATCH] D53314: [ARM][NFCI] Do not fuse VADD and VMUL, continued (1/2)

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 16 02:14:40 PDT 2018


samparker added inline comments.


================
Comment at: lib/Target/ARM/ARMInstrInfo.td:363
+                           "!(TM.Options.AllowFPOpFusion == FPOpFusion::Fast &&"
+                           "  Subtarget->hasVFP4())) ||"
+                           "MF->getFunction().optForMinSize()">;
----------------
I think moving VFP4 check into the useFPVMLx method would help make this easier to read.


https://reviews.llvm.org/D53314





More information about the llvm-commits mailing list