[PATCH] D52081: [InstCombine] do not expand 8 byte memcpy if optimising for minsize

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 14 02:58:19 PDT 2018


lebedev.ri added a comment.

Two questions:

1. Should this be somehow more parametrized, rather than hardcoding magical `Size > 4`? Some target info?
2. Is there some reverse transform, that tries to actually for `memcpy` from such expanded load+store pairs? Shoukd it e disabled too, to avoid looping?


https://reviews.llvm.org/D52081





More information about the llvm-commits mailing list