This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Add test with redundant copies to temporary stack slot produced by expandUnalignedLoad
ClosedPublic

Authored by mbrkusanin on Oct 6 2020, 6:22 AM.

Diff Detail

Event Timeline

mbrkusanin created this revision.Oct 6 2020, 6:22 AM
mbrkusanin requested review of this revision.Oct 6 2020, 6:22 AM
foad added a comment.Oct 6 2020, 6:37 AM

Is "spill" the right name? They come from TargetLowering::expandUnalignedLoad copying the data to a temporary stack slot, not from the register allocator.

mbrkusanin updated this revision to Diff 296456.Oct 6 2020, 7:34 AM
mbrkusanin retitled this revision from [AMDGPU] Add test with spills for local loads and stores to [AMDGPU] Add test with redundant copies to temporary stack slot produced by expandUnalignedLoad.

I used "Spilling" as a term for storing and restoring of values regardless of who/what is causing it. But I guess we should be precise.

rampitec accepted this revision.Oct 6 2020, 10:07 AM
This revision is now accepted and ready to land.Oct 6 2020, 10:07 AM