Page MenuHomePhabricator

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

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

Diff Detail

Event Timeline

mbrkusanin created this revision.Tue, Oct 6, 6:22 AM
mbrkusanin requested review of this revision.Tue, Oct 6, 6:22 AM
foad added a comment.Tue, Oct 6, 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.Tue, Oct 6, 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.Tue, Oct 6, 10:07 AM
This revision is now accepted and ready to land.Tue, Oct 6, 10:07 AM