[PATCH] D45098: [AArch64] fix PR32384: bump the number of stores per memset/memcpy

[Evandro Menezes via Phabricator via llvm-commits]

evandro added a comment.

In https://reviews.llvm.org/D45098#1111593, @evandro wrote:

> In https://reviews.llvm.org/D45098#1111446, @efriedma wrote:
>
> > Those numbers look very different from the ones before. Is r332482 making this less profitable somehow?  Or is the change all noise?
>
>
> I'll have to take a closer look at these CPU2000 results, but in proprietary benchmarks this change is still beneficial.


It's come down to noise.  CPU2000 just lacks enough C++ code where this change has shown to be more beneficial.  Again, in proprietary benchmarks, this change has yielded aggregate improvement around 1% overall and just shy of 5% in a few cases.


https://reviews.llvm.org/D45098