Skip to content

Commit c19fdcf

Browse files
committed
Sema: Make output for DebugGenericSignatures more legible
1 parent f883ca6 commit c19fdcf

File tree

3 files changed

+9
-4
lines changed

3 files changed

+9
-4
lines changed

lib/Sema/TypeCheckDeclPrimary.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2061,6 +2061,7 @@ class DeclChecker : public DeclVisitor<DeclChecker> {
20612061
GenericSignature::get({PD->getProtocolSelfType()},
20622062
PD->getRequirementSignature());
20632063

2064+
llvm::errs() << "\n";
20642065
llvm::errs() << "Protocol requirement signature:\n";
20652066
PD->dumpRef(llvm::errs());
20662067
llvm::errs() << "\n";

lib/Sema/TypeCheckGeneric.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -469,12 +469,13 @@ GenericSignature TypeChecker::checkGenericSignature(
469469
// Debugging of the generic signature builder and generic signature
470470
// generation.
471471
if (dc->getASTContext().TypeCheckerOpts.DebugGenericSignatures) {
472+
llvm::errs() << "\n";
472473
if (auto *VD = dyn_cast_or_null<ValueDecl>(dc->getAsDecl())) {
473474
VD->dumpRef(llvm::errs());
475+
llvm::errs() << "\n";
474476
} else {
475477
dc->printContext(llvm::errs());
476478
}
477-
llvm::errs() << "\n";
478479
llvm::errs() << "Generic signature: ";
479480
sig->print(llvm::errs());
480481
llvm::errs() << "\n";
@@ -591,8 +592,8 @@ GenericSignatureRequest::evaluate(Evaluator &evaluator,
591592
// Debugging of the generic signature builder and generic signature
592593
// generation.
593594
if (GC->getASTContext().TypeCheckerOpts.DebugGenericSignatures) {
594-
PD->printContext(llvm::errs());
595595
llvm::errs() << "\n";
596+
PD->printContext(llvm::errs());
596597
llvm::errs() << "Generic signature: ";
597598
sig->print(llvm::errs());
598599
llvm::errs() << "\n";

lib/Sema/TypeCheckProtocol.cpp

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5115,12 +5115,15 @@ void TypeChecker::checkConformancesInContext(DeclContext *dc,
51155115
// Check all conformances.
51165116
groupChecker.checkAllConformances();
51175117

5118-
if (Context.TypeCheckerOpts.DebugGenericSignatures) {
5118+
if (Context.TypeCheckerOpts.DebugGenericSignatures &&
5119+
!conformances.empty()) {
51195120
// Now that they're filled out, print out information about the conformances
51205121
// here, when requested.
5122+
llvm::errs() << "\n";
5123+
dc->printContext(llvm::errs());
51215124
for (auto conformance : conformances) {
5122-
dc->printContext(llvm::errs());
51235125
conformance->dump(llvm::errs());
5126+
llvm::errs() << "\n";
51245127
}
51255128
}
51265129

0 commit comments

Comments
 (0)