Skip to content

Commit e16e068

Browse files
committed
[lldb][TypeSystemClang][NFC] Remove obsolete commented code
There's no need for these. They are misleading at best, and just clutter the implementation.
1 parent 99f2bc2 commit e16e068

File tree

1 file changed

+0
-97
lines changed

1 file changed

+0
-97
lines changed

lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp

Lines changed: 0 additions & 97 deletions
Original file line numberDiff line numberDiff line change
@@ -6879,46 +6879,6 @@ size_t TypeSystemClang::GetIndexOfChildMemberWithName(
68796879
name, omit_empty_base_classes, child_indexes);
68806880
} break;
68816881

6882-
case clang::Type::ConstantArray: {
6883-
// const clang::ConstantArrayType *array =
6884-
// llvm::cast<clang::ConstantArrayType>(parent_qual_type.getTypePtr());
6885-
// const uint64_t element_count =
6886-
// array->getSize().getLimitedValue();
6887-
//
6888-
// if (idx < element_count)
6889-
// {
6890-
// std::pair<uint64_t, unsigned> field_type_info =
6891-
// ast->getTypeInfo(array->getElementType());
6892-
//
6893-
// char element_name[32];
6894-
// ::snprintf (element_name, sizeof (element_name),
6895-
// "%s[%u]", parent_name ? parent_name : "", idx);
6896-
//
6897-
// child_name.assign(element_name);
6898-
// assert(field_type_info.first % 8 == 0);
6899-
// child_byte_size = field_type_info.first / 8;
6900-
// child_byte_offset = idx * child_byte_size;
6901-
// return array->getElementType().getAsOpaquePtr();
6902-
// }
6903-
} break;
6904-
6905-
// case clang::Type::MemberPointerType:
6906-
// {
6907-
// MemberPointerType *mem_ptr_type =
6908-
// llvm::cast<MemberPointerType>(qual_type.getTypePtr());
6909-
// clang::QualType pointee_type =
6910-
// mem_ptr_type->getPointeeType();
6911-
//
6912-
// if (TypeSystemClang::IsAggregateType
6913-
// (pointee_type.getAsOpaquePtr()))
6914-
// {
6915-
// return GetIndexOfChildWithName (ast,
6916-
// mem_ptr_type->getPointeeType().getAsOpaquePtr(),
6917-
// name);
6918-
// }
6919-
// }
6920-
// break;
6921-
//
69226882
case clang::Type::LValueReference:
69236883
case clang::Type::RValueReference: {
69246884
const clang::ReferenceType *reference_type =
@@ -7058,46 +7018,6 @@ TypeSystemClang::GetIndexOfChildWithName(lldb::opaque_compiler_type_t type,
70587018
name, omit_empty_base_classes);
70597019
} break;
70607020

7061-
case clang::Type::ConstantArray: {
7062-
// const clang::ConstantArrayType *array =
7063-
// llvm::cast<clang::ConstantArrayType>(parent_qual_type.getTypePtr());
7064-
// const uint64_t element_count =
7065-
// array->getSize().getLimitedValue();
7066-
//
7067-
// if (idx < element_count)
7068-
// {
7069-
// std::pair<uint64_t, unsigned> field_type_info =
7070-
// ast->getTypeInfo(array->getElementType());
7071-
//
7072-
// char element_name[32];
7073-
// ::snprintf (element_name, sizeof (element_name),
7074-
// "%s[%u]", parent_name ? parent_name : "", idx);
7075-
//
7076-
// child_name.assign(element_name);
7077-
// assert(field_type_info.first % 8 == 0);
7078-
// child_byte_size = field_type_info.first / 8;
7079-
// child_byte_offset = idx * child_byte_size;
7080-
// return array->getElementType().getAsOpaquePtr();
7081-
// }
7082-
} break;
7083-
7084-
// case clang::Type::MemberPointerType:
7085-
// {
7086-
// MemberPointerType *mem_ptr_type =
7087-
// llvm::cast<MemberPointerType>(qual_type.getTypePtr());
7088-
// clang::QualType pointee_type =
7089-
// mem_ptr_type->getPointeeType();
7090-
//
7091-
// if (TypeSystemClang::IsAggregateType
7092-
// (pointee_type.getAsOpaquePtr()))
7093-
// {
7094-
// return GetIndexOfChildWithName (ast,
7095-
// mem_ptr_type->getPointeeType().getAsOpaquePtr(),
7096-
// name);
7097-
// }
7098-
// }
7099-
// break;
7100-
//
71017021
case clang::Type::LValueReference:
71027022
case clang::Type::RValueReference: {
71037023
const clang::ReferenceType *reference_type =
@@ -7118,23 +7038,6 @@ TypeSystemClang::GetIndexOfChildWithName(lldb::opaque_compiler_type_t type,
71187038
if (pointee_type.IsAggregateType()) {
71197039
return pointee_type.GetIndexOfChildWithName(name,
71207040
omit_empty_base_classes);
7121-
} else {
7122-
// if (parent_name)
7123-
// {
7124-
// child_name.assign(1, '*');
7125-
// child_name += parent_name;
7126-
// }
7127-
//
7128-
// // We have a pointer to an simple type
7129-
// if (idx == 0)
7130-
// {
7131-
// std::pair<uint64_t, unsigned> clang_type_info
7132-
// = ast->getTypeInfo(pointee_type);
7133-
// assert(clang_type_info.first % 8 == 0);
7134-
// child_byte_size = clang_type_info.first / 8;
7135-
// child_byte_offset = 0;
7136-
// return pointee_type.getAsOpaquePtr();
7137-
// }
71387041
}
71397042
} break;
71407043

0 commit comments

Comments
 (0)