[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