HomePhabricator

[ARM] Fix lowering of misaligned memcpy/memset

Description

[ARM] Fix lowering of misaligned memcpy/memset

Currently getOptimalMemOpType returns i32 for large enough sizes without
checking for alignment, leading to poor code generation when misaligned accesses
aren't permitted as we generate a word store then later split it up into byte
stores. This means we inadvertantly go over the MaxStoresPerMemcpy limit and for
memset we splat the memset value into a word then immediately split it up
again.

Fix this by leaving it up to FindOptimalMemOpLowering to figure out which type
to use, but also fix a bug there where it wasn't correctly checking if
misaligned memory accesses are allowed.

Differential Revision: https://reviews.llvm.org/D33442

Details

Committed
john.brawnMay 26 2017, 6:59 AM
Differential Revision
D33442: [ARM] Fix lowering of misaligned memcpy/memset
Parents
rL303989: nits in wide-integer-cmp.ll . NFC
Branches
Unknown
Tags
Unknown