[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