[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