This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Handle promoting memmove
ClosedPublic

Authored by arsenm on Feb 1 2016, 4:33 PM.

Details

Reviewers
tstellarAMD
Summary

Also add missing tests for the others.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 46592.Feb 1 2016, 4:33 PM
arsenm retitled this revision from to AMDGPU: Handle promoting memmove.
arsenm updated this object.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.
mgrang added a subscriber: mgrang.Feb 1 2016, 4:37 PM
mgrang added inline comments.
lib/Target/AMDGPU/AMDGPUPromoteAlloca.cpp
587

Minor nit pick with spacing.
MemMove->getLength() and MemMove->isVolatile() are not aligned with MemMove->getRawDest().

arsenm updated this revision to Diff 46597.Feb 1 2016, 5:01 PM

Fix spacing. Also add another easy to handle intrinsic

tstellarAMD accepted this revision.Feb 2 2016, 8:45 AM
tstellarAMD edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Feb 2 2016, 8:45 AM
arsenm closed this revision.Feb 2 2016, 12:36 PM

r259558