[PATCH] D35035: [InstCombine] Prevent memcpy generation for small data size

DIVYA SHANMUGHAN via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 14 08:44:30 PDT 2017


DIVYA marked an inline comment as done.
DIVYA added inline comments.


================
Comment at: test/Transforms/InstCombine/memcpy-to-load.ll:81
+
+; If there is no datalayout, then all memcpy of size less than 16 (and power-of-2) will be expanded inline with load/store
+
----------------
spatel wrote:
> That is the existing behavior, but why is that valid now that we're using the datalayout?
 In test cases when the target datalayout is not specified, DL.getLargestLegalIntTypeSizeInBits() returns 0.For those cases it will have the previous behavious before applying the patch


https://reviews.llvm.org/D35035





More information about the llvm-commits mailing list