[PATCH] D42889: [LoopIdiomRecognize] Add support for memmove. Fixes PR25165

Eric Fiselier via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 8 16:50:42 PST 2018


EricWF added a comment.

In https://reviews.llvm.org/D42889#998260, @efriedma wrote:

> This transform isn't legal.  Consider:
>
>   #include <assert.h>
>   void not_memmove(int *x, int *y, int n) {
>     for (int i = 0; i < n; ++i)
>       x[i] = y[i];
>   }


Woops. Thanks for the correction. I'll attempt to clean this up.


https://reviews.llvm.org/D42889





More information about the llvm-commits mailing list