Skip to content
This repository was archived by the owner on Apr 2, 2020. It is now read-only.

Commit cb46c32

Browse files
authored
Merge pull request #127 from slavapestov/small-lldb-swift-cleanup
[Swift] Use llvm casts instead of looking at {Decl,TypeBase}::getKind()
2 parents 9c7cbb4 + e4a7e2b commit cb46c32

File tree

1 file changed

+3
-9
lines changed

1 file changed

+3
-9
lines changed

source/Symbol/SwiftASTContext.cpp

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -433,7 +433,7 @@ CachedMemberInfo *SwiftASTContext::GetCachedMemberInfo(void *type) {
433433
}
434434

435435
for (auto decl : nominal_decl->getMembers()) {
436-
if (decl->getKind() == swift::DeclKind::Var) {
436+
if (swift::isa<swift::VarDecl>(decl)) {
437437
swift::VarDecl *var_decl = llvm::cast<swift::VarDecl>(decl);
438438
if (var_decl->hasStorage() && !var_decl->isStatic()) {
439439
MemberInfo member_info(MemberType::Field);
@@ -494,7 +494,7 @@ CachedMemberInfo *SwiftASTContext::GetCachedMemberInfo(void *type) {
494494

495495
for (auto decl : t_decl->getMembers()) {
496496
// Find ivars that aren't properties
497-
if (decl->getKind() == swift::DeclKind::Var) {
497+
if (swift::isa<swift::VarDecl>(decl)) {
498498
swift::VarDecl *var_decl = llvm::cast<swift::VarDecl>(decl);
499499
if (var_decl->hasStorage() && !var_decl->isStatic()) {
500500
MemberInfo member_info(MemberType::Field);
@@ -4008,13 +4008,7 @@ ConstString SwiftASTContext::GetMangledTypeName(swift::TypeBase *type_base) {
40084008

40094009
swift::Type swift_type(type_base);
40104010

4011-
bool has_archetypes = false;
4012-
4013-
swift_type.visit([&has_archetypes](swift::Type part_type) -> void {
4014-
if (part_type->getKind() == swift::TypeKind::Archetype) {
4015-
has_archetypes = true;
4016-
}
4017-
});
4011+
bool has_archetypes = swift_type->hasArchetype();
40184012

40194013
if (!has_archetypes) {
40204014
swift::Mangle::Mangler mangler(true);

0 commit comments

Comments
 (0)