This is an archive of the discontinued LLVM Phabricator instance.

Differential D132895

[ModuloScheduler] Fix missing LLVM_DEBUG
ClosedPublic

Authored by dpenry on Aug 29 2022, 3:55 PM.

Details

Reviewers
dmgreen
Commits
rG9aca7b02170c: [ModuloScheduler] Fix missing LLVM_DEBUG
Summary

Guard a debug message with LLVM_DEBUG

Diff Detail

Event Timeline

dpenry created this revision.Aug 29 2022, 3:55 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 29 2022, 3:55 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
dpenry requested review of this revision.Aug 29 2022, 3:55 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 29 2022, 3:55 PM
Herald added a subscriber: llvm-commits. · View Herald Transcript
dpenry added a reviewer: dmgreen.Aug 29 2022, 3:56 PM
Harbormaster completed remote builds in B184034: Diff 456484.Aug 29 2022, 5:40 PM
This revision was not accepted when it landed; it landed in state Needs Review.Aug 30 2022, 9:21 AM
Closed by commit rG9aca7b02170c: [ModuloScheduler] Fix missing LLVM_DEBUG (authored by dpenry). · Explain Why
This revision was automatically updated to reflect the committed changes.
dpenry added a commit: rG9aca7b02170c: [ModuloScheduler] Fix missing LLVM_DEBUG.

Revision Contents

PathSize
llvm/
lib/
CodeGen/
2 lines

Diff 456697

llvm/lib/CodeGen/MachinePipeliner.cpp

Show First 20 Lines • Show All 2,095 Lines • ▼ Show 20 Linesbool SwingSchedulerDAG::schedulePipeline(SMSchedule &Schedule) {
LLVM_DEBUG(dbgs() << "Schedule Found? " << scheduleFound LLVM_DEBUG(dbgs() << "Schedule Found? " << scheduleFound
<< " (II=" << Schedule.getInitiationInterval() << " (II=" << Schedule.getInitiationInterval()
<< ")\n"); << ")\n");
if (scheduleFound) { if (scheduleFound) {
scheduleFound = LoopPipelinerInfo->shouldUseSchedule(*this, Schedule); scheduleFound = LoopPipelinerInfo->shouldUseSchedule(*this, Schedule);
if (!scheduleFound) if (!scheduleFound)
dbgs() << "Target rejected schedule\n"; LLVM_DEBUG(dbgs() << "Target rejected schedule\n");
} }
if (scheduleFound) { if (scheduleFound) {
Schedule.finalizeSchedule(this); Schedule.finalizeSchedule(this);
Pass.ORE->emit([&]() { Pass.ORE->emit([&]() {
return MachineOptimizationRemarkAnalysis( return MachineOptimizationRemarkAnalysis(
DEBUG_TYPE, "schedule", Loop.getStartLoc(), Loop.getHeader()) DEBUG_TYPE, "schedule", Loop.getStartLoc(), Loop.getHeader())
<< "Schedule found with Initiation Interval: " << "Schedule found with Initiation Interval: "
▲ Show 20 Lines • Show All 1,053 Lines • Show Last 20 Lines