This is an archive of the discontinued LLVM Phabricator instance.

Adjust printQualifiedName to handle unscoped enums in a way similar to anonymous namespaces.
ClosedPublic

Authored by saugustine on Nov 6 2015, 11:51 AM.

Reviewers

klimek
rsmith

Commits

Summary

Adjust printQualifiedName to handle unscoped enums in a way similar to anonymous namespaces.

saugustine retitled this revision from to Adjust printQualifiedName to handle unscoped enums in a way similar to anonymous namespaces..

saugustine updated this object.

This is related to this bug report:

Handle unscoped enum with name. Also add test cases.

rsmith edited edge metadata.

This revision is now accepted and ready to land.Nov 6 2015, 4:43 PM

This revision was automatically updated to reflect the committed changes.

Path

Size

lib/

AST/

9 lines

Show First 20 Lines • Show All 1,424 Lines • ▼ Show 20 Lines	if (const ClassTemplateSpecializationDecl *Spec

if (FT->isVariadic()) {		if (FT->isVariadic()) {
if (NumParams > 0)		if (NumParams > 0)
OS << ", ";		OS << ", ";
OS << "...";		OS << "...";
}		}
}		}
OS << ')';		OS << ')';
		} else if (const EnumDecl ED = dyn_cast<EnumDecl>(I)) {
		// C++ [dcl.enum]p10: Each enum-name and each unscoped
		// enumerator is declared in the scope that immediately contains
		// the enum-specifier. Each scoped enumerator is declared in the
		// scope of the enumeration.
		if (ED->isScoped())
		OS << *ED;
		else
		continue;
} else {		} else {
OS << cast<NamedDecl>(I);		OS << cast<NamedDecl>(I);
}		}
OS << "::";		OS << "::";
}		}

if (getDeclName())		if (getDeclName())
OS << *this;		OS << *this;
▲ Show 20 Lines • Show All 2,607 Lines • Show Last 20 Lines