This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU/SI: Handle s_getreg hazard in GCNHazardRecognizer
ClosedPublic

Authored by tstellarAMD on Oct 12 2016, 11:05 AM.

Event Timeline

tstellarAMD retitled this revision from to AMDGPU/SI: Handle s_getreg hazard in GCNHazardRecognizer.
tstellarAMD updated this object.
tstellarAMD added a reviewer: arsenm.
tstellarAMD added a subscriber: llvm-commits.

Fix whitespace.

arsenm accepted this revision.Oct 12 2016, 11:33 AM
arsenm edited edge metadata.

LGTM

lib/Target/AMDGPU/GCNHazardRecognizer.cpp
320

HWReg != getHWReg looks confusing to me. getHWReg(TII, *MI) != GetRegHWReg?

This revision is now accepted and ready to land.Oct 12 2016, 11:33 AM
tstellarAMD edited edge metadata.

Refactor to make it easier to share code when adding support for other hazards.

This revision was automatically updated to reflect the committed changes.