This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Fix alignments in test
ClosedPublic

Authored by arsenm on Feb 17 2016, 11:32 AM.

Details

Reviewers
tstellarAMD
Summary

I don't think this test was intending to test unaligned load/store.
Change it to use the natural alignment to avoid regressing.

Also adds missing SI checks.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 48223.Feb 17 2016, 11:32 AM
arsenm retitled this revision from to AMDGPU: Fix alignments in test.
arsenm updated this object.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.
tstellarAMD accepted this revision.Feb 17 2016, 11:37 AM
tstellarAMD edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Feb 17 2016, 11:37 AM
arsenm closed this revision.Feb 22 2016, 1:09 PM

r261571