Skip to content

Commit

Permalink
[TargetPassConfig] Enable machine verification after miscellaneous pa…
Browse files Browse the repository at this point in the history
…sses

In a couple of places machine verification was disabled for no apparent
reason, probably just because an "addPass(..., false)" line was cut and
pasted from elsewhere.

After this patch the only remaining place where machine verification is
disabled in the generic TargetPassConfig code, is after addPreEmitPass.
  • Loading branch information
jayfoad committed Oct 7, 2021
1 parent c77a5c2 commit 097339b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions llvm/lib/CodeGen/TargetPassConfig.cpp
Expand Up @@ -1217,7 +1217,7 @@ void TargetPassConfig::addMachinePasses() {
// GC
if (addGCPasses()) {
if (PrintGCInfo)
addPass(createGCInfoPrinter(dbgs()), false);
addPass(createGCInfoPrinter(dbgs()));
}

// Basic block placement.
Expand Down Expand Up @@ -1507,7 +1507,7 @@ void TargetPassConfig::addMachineLateOptimization() {

/// Add standard GC passes.
bool TargetPassConfig::addGCPasses() {
addPass(&GCMachineCodeAnalysisID, false);
addPass(&GCMachineCodeAnalysisID);
return true;
}

Expand Down

0 comments on commit 097339b

Please sign in to comment.