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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 14 12:40:42 PDT 2018


efriedma added a comment.

It looks like https://reviews.llvm.org/D46477 has review comments which are waiting to be addressed?  But the approach of changing the chains seems reasonable.

Increasing the maximum number of inlined instructions for memcpy and memset from 4 to 16 seems right; it's probably a good performance/size tradeoff.  Not sure about memmove; you also have to worry about register pressure for that.


https://reviews.llvm.org/D45098





More information about the llvm-commits mailing list