[llvm-commits] patch: partial DSE

Eric Christopher echristo at apple.com
Tue Nov 1 15:15:17 PDT 2011


>> 
> 
> There aren't actually any vector types involved here; the issue is
> that, for example, a 32 byte memset is cheaper than a 31-byte memset
> under the default settings on x86-64.  I'm not sure what the right
> approach is here.

Good point, I'd taken the 16 as a 16-byte vector issue and not the sizes.
This is probably related to the patches that Lang did recently trying to figure
out what the best number of "loads" is for a memset/memcpy as well.

Maybe a TLI.shouldCallMemcpy(Ty) type of call?

Not sure.

-eric



More information about the llvm-commits mailing list