[PATCH] D20176: [MemCpyOpt] Use MaxIntSize in byte instead of bit

[Chad Rosier via llvm-commits]

mcrosier added a comment.

In your test case aren't we merging these stores into a memset to be later lowered to a series of stores (or single 64-bit store) on most targets?  The final code generated will be better in this case, because it can merge the 3 stores.  In other cases I'm concerned the rather small memset might perturb optimizations that don't know how to deal with memsets.  Don't we have a pass that merges these type of things (e.g., SLP vectorizer or the counterpart to the LoadMerge pass or something)?


http://reviews.llvm.org/D20176