This is an archive of the discontinued LLVM Phabricator instance.

R600/SI: Report offset in correct units for st64 DS instructions
ClosedPublic

Authored by arsenm on Sep 9 2014, 10:59 AM.

Details

Reviewers
arsenm
Summary

Need to convert the 64 element offset into bytes, not just the element
size like the normal case instructions.

Noticed by inspection. This can't be hit now because
st64 instructions aren't emitted during instruction selection,
and the post-RA scheduler isn't enabled.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 13483.Sep 9 2014, 10:59 AM
arsenm retitled this revision from to R600/SI: Report offset in correct units for st64 DS instructions .
arsenm updated this object.
arsenm edited the test plan for this revision. (Show Details)
arsenm added a subscriber: Unknown Object (MLST).
arsenm accepted this revision.Sep 10 2014, 4:36 PM
arsenm added a reviewer: arsenm.

r217560

This revision is now accepted and ready to land.Sep 10 2014, 4:36 PM
arsenm closed this revision.Sep 10 2014, 4:36 PM