[PATCH] D63447: [ARM] Put some of the TTI costmodel behind hasNeon calls.

[Dave Green via Phabricator via llvm-commits]

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

This puts some of the calls in ARMTargetTransformInfo.cpp behind hasNeon() checks, now that we have MVE coming. I have also regenerated a large number of cost model checks, which I show the differences of here (which is really the meat of this patch).

Many of the costs will not be very precise yet, there are updates for MVE coming. They look like they are mostly moving in the correct direction. Broadcasts (VDUPs) and divs both have patches to adjust the costs specifically.


https://reviews.llvm.org/D63447

Files:
  llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
  llvm/test/Analysis/CostModel/ARM/cast.ll
  llvm/test/Analysis/CostModel/ARM/divrem.ll
  llvm/test/Analysis/CostModel/ARM/load_store.ll
  llvm/test/Analysis/CostModel/ARM/select.ll
  llvm/test/Analysis/CostModel/ARM/shuffle.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63447.205115.patch
Type: text/x-patch
Size: 64875 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190617/56f87b14/attachment.bin>