[PATCH] D59787: [ARM] Implement TTI::getMemcpyCost

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer updated this revision to Diff 193888.
SjoerdMeijer added a comment.

Run code-size benchmark CSiBe, triggered an assert in function FindOptimalMemOpLowering, and fixed that.

This are the code-size results:

  Before  After    Diff 

csibe/OpenTCP-1.0.4       21879   21895    0.07%
csibe/bzip2-1.0.2         47350   47390    0.08%
csibe/cg_compiler_opensrc 96212   96184   -0.03%
csibe/compiler            19286   19297    0.06%
csibe/jikespg-1.3         170538  170199  -0.20%
csibe/jpeg-6b             96431   96441    0.01%
csibe/libpng-1.2.5        77702   77902    0.26%
csibe/lwip-0.5.3.preproc  68611   68627    0.02%
csibe/mpeg2dec-0.3.1      38988   38924   -0.16%
csibe/mpgcut-1.1          7196    7198     0.03%
csibe/teem-1.6.0-src      1033815 1034595  0.08%
csibe/ttt-0.10.1.preproc  12711   12703   -0.06%
csibe/unrarlib-0.4.0      10394   10415    0.20%
csibe/zlib-1.1.4          29229   29228    0.00%

csibe                     1730342 1730998  0.04%
total                     1730343 1730999  0.04%

Overall, small differences and the changes are neutral, but I will look into libpng to see what is happening there.


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

https://reviews.llvm.org/D59787

Files:
  lib/Analysis/TargetTransformInfo.cpp
  lib/CodeGen/SelectionDAG/TargetLowering.cpp
  lib/Target/ARM/ARMTargetTransformInfo.cpp
  lib/Target/ARM/ARMTargetTransformInfo.h
  test/Analysis/CostModel/ARM/memcpy.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D59787.193888.patch
Type: text/x-patch
Size: 16834 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190405/c576967d/attachment.bin>