This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Fix assert when legalizing atomic operands
ClosedPublic

Authored by arsenm on Nov 3 2015, 11:05 AM.

Details

Reviewers
tstellarAMD
Summary

The operand layout is slightly different for the atomic
opcodes from the usual MUBUF loads and stores.

This should only fix it on SI/CI. VI is still broken
because it still emits the addr64 repalcement.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 39096.Nov 3 2015, 11:05 AM
arsenm retitled this revision from to AMDGPU: Fix assert when legalizing atomic operands.
arsenm updated this object.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.
tstellarAMD accepted this revision.Nov 4 2015, 6:32 PM
tstellarAMD edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Nov 4 2015, 6:32 PM
arsenm closed this revision.Nov 4 2015, 6:50 PM

r252140