This is an archive of the discontinued LLVM Phabricator instance.

Differential D30149

AMDGPU/SI: Fix SGPR usage count in SI scheduler
ClosedPublic

Authored by axeldavy on Feb 18 2017, 1:28 PM.

Details

Reviewers
arsenm
tstellar
Commits
rGf7d1023a730a: [AMDGPU] Fix SGPR usage count in SI scheduler
rL298710: [AMDGPU] Fix SGPR usage count in SI scheduler
Summary

The max number of SGPRs used was incorrectly
computed.

Diff Detail

Event Timeline

axeldavy created this revision.Feb 18 2017, 1:28 PM
Herald added subscribers: tpr, tony-tye, yaxunl and 4 others. · View Herald TranscriptFeb 18 2017, 1:28 PM
axeldavy added a subscriber: llvm-commits.Feb 18 2017, 2:33 PM
arsenm accepted this revision.Feb 21 2017, 3:35 PM

LGTM

This revision is now accepted and ready to land.Feb 21 2017, 3:35 PM
t-tye added a subscriber: t-tye.Mar 22 2017, 6:39 PM
tony-tye removed a subscriber: tony-tye.Mar 22 2017, 6:50 PM
Closed by commit rL298710: [AMDGPU] Fix SGPR usage count in SI scheduler (authored by vpykhtin). · Explain WhyMar 24 2017, 9:58 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
lib/
Target/
AMDGPU/
4 lines

Diff 89042

lib/Target/AMDGPU/SIMachineScheduler.cpp

Context not available.
VregCurrentUsage, SregCurrentUsage); VregCurrentUsage, SregCurrentUsage);
if (VregCurrentUsage > maxVregUsage) if (VregCurrentUsage > maxVregUsage)
maxVregUsage = VregCurrentUsage; maxVregUsage = VregCurrentUsage;
if (VregCurrentUsage > maxSregUsage) if (SregCurrentUsage > maxSregUsage)
maxSregUsage = VregCurrentUsage; maxSregUsage = SregCurrentUsage;
DEBUG( DEBUG(
dbgs() << "Picking New Blocks\n"; dbgs() << "Picking New Blocks\n";
dbgs() << "Available: "; dbgs() << "Available: ";
Context not available.