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

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 16 05:45:44 PDT 2018


SjoerdMeijer updated this revision to Diff 169813.
SjoerdMeijer added a comment.
Herald added a subscriber: nhaehnle.

> I think moving VFP4 check into the useFPVMLx method would help make this easier to read.

Thanks for the suggestion. I had a look, and turns out we don't need it at all because the VFP checks and predicates are already on the rules. Thus we can simplify the UseFPVMLx predicate even more by removing the VFP check from it.


https://reviews.llvm.org/D53314

Files:
  lib/Target/ARM/ARMInstrInfo.td
  lib/Target/ARM/ARMInstrNEON.td
  lib/Target/ARM/ARMInstrVFP.td
  test/CodeGen/ARM/fmacs.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D53314.169813.patch
Type: text/x-patch
Size: 15546 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181016/96a6c6e1/attachment.bin>


More information about the llvm-commits mailing list