This is an archive of the discontinued LLVM Phabricator instance.

Differential D67478

[ScheduleDAGMILive] Fix typo in comment.
ClosedPublic

Authored by xmj on Sep 11 2019, 8:57 PM.

Details

Reviewers
atrick
Commits
rG4b191770f49f: [ScheduleDAGMILive] Fix typo in comment.
rL371916: [ScheduleDAGMILive] Fix typo in comment.

Diff Detail

Repository
rL LLVM

Event Timeline

xmj created this revision.Sep 11 2019, 8:57 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 11 2019, 8:57 PM
Herald added subscribers: llvm-commits, javed.absar, hiraditya, MatzeB. · View Herald Transcript
atrick accepted this revision.Sep 12 2019, 10:51 AM

lgtm

This revision is now accepted and ready to land.Sep 12 2019, 10:51 AM
Closed by commit rL371916: [ScheduleDAGMILive] Fix typo in comment. (authored by xmj). · Explain WhySep 13 2019, 8:27 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
llvm/
trunk/
lib/
CodeGen/
2 lines

Diff 220202

llvm/trunk/lib/CodeGen/MachineScheduler.cpp

Show First 20 Lines • Show All 979 Lines • ▼ Show 20 Linesvoid ScheduleDAGMILive::enterRegion(MachineBasicBlock *bb,
ShouldTrackPressure = SchedImpl->shouldTrackPressure(); ShouldTrackPressure = SchedImpl->shouldTrackPressure();
ShouldTrackLaneMasks = SchedImpl->shouldTrackLaneMasks(); ShouldTrackLaneMasks = SchedImpl->shouldTrackLaneMasks();
assert((!ShouldTrackLaneMasks || ShouldTrackPressure) && assert((!ShouldTrackLaneMasks || ShouldTrackPressure) &&
"ShouldTrackLaneMasks requires ShouldTrackPressure"); "ShouldTrackLaneMasks requires ShouldTrackPressure");
} }
// Setup the register pressure trackers for the top scheduled top and bottom // Setup the register pressure trackers for the top scheduled and bottom
// scheduled regions. // scheduled regions.
void ScheduleDAGMILive::initRegPressure() { void ScheduleDAGMILive::initRegPressure() {
VRegUses.clear(); VRegUses.clear();
VRegUses.setUniverse(MRI.getNumVirtRegs()); VRegUses.setUniverse(MRI.getNumVirtRegs());
for (SUnit &SU : SUnits) for (SUnit &SU : SUnits)
collectVRegUses(SU); collectVRegUses(SU);
TopRPTracker.init(&MF, RegClassInfo, LIS, BB, RegionBegin, TopRPTracker.init(&MF, RegClassInfo, LIS, BB, RegionBegin,
▲ Show 20 Lines • Show All 2,760 Lines • Show Last 20 Lines