This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Handle splitting 64-bit private accesses when necessary
AbandonedPublic

Authored by arsenm on Feb 10 2016, 7:55 PM.

Details

Reviewers
tstellarAMD

Diff Detail

Event Timeline

arsenm updated this revision to Diff 47587.Feb 10 2016, 7:55 PM
arsenm retitled this revision from to AMDGPU: Handle splitting 64-bit private accesses when necessary.
arsenm updated this object.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.

I'm going to try moving this into the lowering and try to fix the problem with ExpandUnalignedLoad, or do this in a post-legalize target dag combine. The problem with this is the pointer add for the split is not folded into the previous pointer add, which sometimes breaks the addressing mode matching

tstellarAMD accepted this revision.Apr 21 2016, 10:25 AM
tstellarAMD edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Apr 21 2016, 10:25 AM
arsenm abandoned this revision.May 16 2016, 3:24 PM

Replaced with custom lowering. Tests committed r269712