This is an archive of the discontinued LLVM Phabricator instance.

R600/SI: Consider adjacent offsets in getLdStBaseRegImmOfs
ClosedPublic

Authored by arsenm on Jul 29 2014, 2:45 PM.

Details

Reviewers
arsenm
Summary

We can treat ds_read2_* as a single offset if the offsets are adjacent.

No test since emission of read2 instructions for partially
aligned loads isn't implemented yet.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 11995.Jul 29 2014, 2:45 PM
arsenm retitled this revision from to R600/SI: Consider adjacent offsets in getLdStBaseRegImmOfs .
arsenm updated this object.
arsenm edited the test plan for this revision. (Show Details)
arsenm added a subscriber: Unknown Object (MLST).
arsenm updated this revision to Diff 11998.Jul 29 2014, 3:37 PM

Use size of data0 operand if it's a write2 instead of the first, non-result operand to get the element size

arsenm accepted this revision.Jul 29 2014, 6:10 PM
arsenm added a reviewer: arsenm.

r214269

This revision is now accepted and ready to land.Jul 29 2014, 6:10 PM
arsenm closed this revision.Jul 29 2014, 6:10 PM