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

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added a comment.

One more thought then on this:

> If we want to distinguish optimizing for size from optimizing for speed, that belongs in the backend

that means we shouldn't be doing any memcpy lowering at all in InstCombiner::SimplifyAnyMemTransfer. In other words, if I rewrite this patch to actually strip out any memcpy lowering here in InstCombine, would that be an acceptable approach?


https://reviews.llvm.org/D52081