[PATCH] D69115: [ARM] Use isFMAFasterThanFMulAndFAdd for MVE

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 17 09:37:08 PDT 2019


dmgreen created this revision.
dmgreen added reviewers: t.p.northover, samparker, SjoerdMeijer, simon_tatham, ostannard.
Herald added subscribers: hiraditya, kristof.beyls.
Herald added a project: LLVM.

The Arm backend will usually return false for isFMAFasterThanFMulAndFAdd, where both the fused VFMA.f32 and a non-fused VMLA.f32 are usually available for scalar code. For MVE we don't have the non-fused version though. It makes more sense for isFMAFasterThanFMulAndFAdd to return true, allowing us to simplify some of the existing ISel patterns.

The tests here are that non of the existing tests failed, and so we are still selecting VFMA and VFMS.


https://reviews.llvm.org/D69115

Files:
  llvm/lib/Target/ARM/ARMISelLowering.cpp
  llvm/lib/Target/ARM/ARMISelLowering.h
  llvm/lib/Target/ARM/ARMInstrMVE.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69115.225445.patch
Type: text/x-patch
Size: 4384 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191017/8e7f5741/attachment.bin>


More information about the llvm-commits mailing list