This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU/SI: Make sure to emit TargetConstant nodes when matching ds_*permute
ClosedPublic

Authored by tstellarAMD on May 5 2016, 2:06 PM.

Details

Summary

This fixes a bug with ds_*permute instructions where if it was passed a
constant address, then the offset operand would get assigned a register
operand instead of an immediate.

Diff Detail

Event Timeline

tstellarAMD retitled this revision from to AMDGPU/SI: Make sure to emit TargetConstant nodes when matching ds_*permute.
tstellarAMD updated this object.
tstellarAMD added reviewers: arsenm, scchan.
tstellarAMD added a subscriber: llvm-commits.
arsenm accepted this revision.May 5 2016, 2:09 PM
arsenm edited edge metadata.

LGTM

This revision is now accepted and ready to land.May 5 2016, 2:09 PM
This revision was automatically updated to reflect the committed changes.