This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Fix constant bus use check with subregisters
ClosedPublic

Authored by arsenm on Feb 10 2016, 2:54 PM.

Details

Reviewers
tstellarAMD
Summary

If the two operands to an instruction were both
subregisters of the same super register, it would incorrectly
think this counted as the same constant bus use.

This fixes the verifier error in fmin_legacy.ll which
was missing -verify-machineinstrs.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 47536.Feb 10 2016, 2:54 PM
arsenm retitled this revision from to AMDGPU: Fix constant bus use check with subregisters.
arsenm updated this object.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.
tstellarAMD accepted this revision.Feb 10 2016, 6:09 PM
tstellarAMD edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Feb 10 2016, 6:09 PM
arsenm closed this revision.Feb 10 2016, 10:20 PM

r260495