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

Sebastian Pop via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 14 12:27:47 PDT 2018


sebpop added a comment.

In https://reviews.llvm.org/D45098#1053266, @efriedma wrote:

> Looking at the generated code a bit, it looks like we do a really terrible job lowering memcpy; we don't form ldp/stp at all, ever.  We should probably fix that before we mess with the threshold here; it could substantially change the codesize/performance impact of this change.


Eli, could you please look at the patch that Sirish has posted https://reviews.llvm.org/D46477 to make the memcpy lowering produce ldp/stp?
Are there other things to be fixed to land this change?


https://reviews.llvm.org/D45098





More information about the llvm-commits mailing list