This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Mark s_barrier as a high latency instruction
ClosedPublic

Authored by arsenm on Aug 26 2015, 10:21 PM.

Details

Reviewers
tstellarAMD
Summary

These were marked as WriteSALU, which is low latency.
I'm guessing at the value to use, but it should probably
be considered the highest latency instruction.

I'm not sure this has any actual effect since hasSideEffects
probably is preventing any moving of these.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 33300.Aug 26 2015, 10:21 PM
arsenm retitled this revision from to AMDGPU: Mark s_barrier as a high latency instruction.
arsenm updated this object.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.
tstellarAMD accepted this revision.Sep 8 2015, 12:08 PM
tstellarAMD edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Sep 8 2015, 12:08 PM
arsenm closed this revision.Sep 8 2015, 12:56 PM

r247060