[PATCH] D30225: [LIR] re-enable generation of memmove with runtime checks

[Sebastian Pop via Phabricator via llvm-commits]

sebpop added a comment.

In https://reviews.llvm.org/D30225#689790, @efriedma wrote:

> > With the patch and compiling with "-mllvm -stats" the spec 2006:
> >  Number of memcpy's formed from loop load+stores: 42
> > 
> > Before the patch on spec 2006:
> >  Number of memcpy's formed from loop load+stores: 98
>
> This looks bad.


I've been trying to understand why the number of memcpys are changing:
the patch should only add new memmoves with runtime checks.
I then decided to rerun the spec2006, and it looks like the specmake was the problem
it truncated some of the stderr output.

With the patch I see the following on cpu2006:
Number of memcpy's formed from loop load+stores: 98
Number of memset's formed from loop stores: 1398
Number of memmove's formed from loop load+stores: 155


https://reviews.llvm.org/D30225