================
@@ -1926,12 +1930,13 @@ DWARFASTParserClang::ParseStructureLikeDIE(const
SymbolContext &sc,
// TypeSystemClang is always in C++ mode, but some compilers such as
// GCC and Clang give empty structs a size of 0 in C mode (in contrast to
// the size of 1 for empty structs that would be computed in C++ mode).
- if (attrs.byte_size) {
+ if (attrs.byte_size || attrs.alignment) {
clang::RecordDecl *record_decl =
TypeSystemClang::GetAsRecordDecl(clang_type);
if (record_decl) {
ClangASTImporter::LayoutInfo layout;
- layout.bit_size = *attrs.byte_size * 8;
+ layout.bit_size = attrs.byte_size.value_or(0) * 8;
+ layout.alignment = attrs.alignment.value_or(0) * 8;
----------------
clayborg wrote:
This looks fine to me. If we have a byte size computed by clang, it was always
using it and it helped with C structs being computed in a C++ type system when
the size was zero or 1. Now we will also respect the alignment. Also the
ClangASTImporter::LayoutInfo::bit_size is initialized to zero in the header
file, so even though we will make it into this case now if we have only
alignment,it shouldn't change anything AFAIK.
https://github.com/llvm/llvm-project/pull/73307
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
