X-Git-Url: http://nsz.repo.hu/git/?a=blobdiff_plain;f=type.c;h=7b938a997f5c714af8f58730757d3175e465ac17;hb=779610287a11b207e958e31a29f0dd9ea5459e39;hp=667a912be41a78d20e40c36d8862e79b7e4e0a4b;hpb=6d7d61de445629e1590d3a2ba2990ef7a651368a;p=cparser diff --git a/type.c b/type.c index 667a912..7b938a9 100644 --- a/type.c +++ b/type.c @@ -1,27 +1,12 @@ /* * This file is part of cparser. - * Copyright (C) 2007-2009 Matthias Braun - * - * This program is free software; you can redistribute it and/or - * modify it under the terms of the GNU General Public License - * as published by the Free Software Foundation; either version 2 - * of the License, or (at your option) any later version. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software - * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA - * 02111-1307, USA. + * Copyright (C) 2012 Matthias Braun */ #include #include -#include +#include "adt/bitfiddle.h" #include "type_t.h" #include "types.h" #include "entity_t.h" @@ -32,206 +17,228 @@ #include "lang_features.h" #include "warning.h" #include "diagnostic.h" -#include "driver/firm_cmdline.h" +#include "printer.h" +#include "separator_t.h" /** The default calling convention. */ cc_kind_t default_calling_convention = CC_CDECL; -static struct obstack _type_obst; -static FILE *out; -struct obstack *type_obst = &_type_obst; -static bool print_implicit_array_size = false; +static struct obstack type_obst; +static bool print_implicit_array_size = false; static void intern_print_type_pre(const type_t *type); static void intern_print_type_post(const type_t *type); -typedef struct atomic_type_properties_t atomic_type_properties_t; -struct atomic_type_properties_t { - unsigned size; /**< type size in bytes */ - unsigned alignment; /**< type alignment in bytes */ - unsigned flags; /**< type flags from atomic_type_flag_t */ -}; +/** + * Returns the size of a type node. + * + * @param kind the type kind + */ +static size_t get_type_struct_size(type_kind_t kind) +{ + static const size_t sizes[] = { + [TYPE_ATOMIC] = sizeof(atomic_type_t), + [TYPE_IMAGINARY] = sizeof(atomic_type_t), + [TYPE_COMPLEX] = sizeof(atomic_type_t), + [TYPE_COMPOUND_STRUCT] = sizeof(compound_type_t), + [TYPE_COMPOUND_UNION] = sizeof(compound_type_t), + [TYPE_ENUM] = sizeof(enum_type_t), + [TYPE_FUNCTION] = sizeof(function_type_t), + [TYPE_POINTER] = sizeof(pointer_type_t), + [TYPE_REFERENCE] = sizeof(reference_type_t), + [TYPE_ARRAY] = sizeof(array_type_t), + [TYPE_TYPEDEF] = sizeof(typedef_type_t), + [TYPE_TYPEOF] = sizeof(typeof_type_t), + }; + assert((size_t)kind < lengthof(sizes)); + assert(sizes[kind] != 0); + return sizes[kind]; +} + +type_t *allocate_type_zero(type_kind_t kind) +{ + size_t const size = get_type_struct_size(kind); + type_t *const res = obstack_alloc(&type_obst, size); + memset(res, 0, size); + res->base.kind = kind; + + return res; +} /** * Properties of atomic types. */ -static atomic_type_properties_t atomic_type_properties[ATOMIC_TYPE_LAST+1] = { - //ATOMIC_TYPE_INVALID = 0, +atomic_type_properties_t atomic_type_properties[ATOMIC_TYPE_LAST+1] = { [ATOMIC_TYPE_VOID] = { - .size = 0, - .alignment = 0, - .flags = ATOMIC_TYPE_FLAG_NONE - }, - [ATOMIC_TYPE_WCHAR_T] = { - .size = (unsigned)-1, - .alignment = (unsigned)-1, - /* signed flag will be set when known */ - .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC, + .size = 1, + .alignment = 1, + .flags = ATOMIC_TYPE_FLAG_NONE, + .rank = 0, }, - [ATOMIC_TYPE_CHAR] = { + [ATOMIC_TYPE_BOOL] = { .size = 1, .alignment = 1, - /* signed flag will be set when known */ .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC, + .rank = 1, + }, + [ATOMIC_TYPE_CHAR] = { + .size = 1, + .alignment = 1, + .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC, + .rank = 2, }, [ATOMIC_TYPE_SCHAR] = { - .size = 1, - .alignment = 1, - .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC - | ATOMIC_TYPE_FLAG_SIGNED, + .size = 1, + .alignment = 1, + .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC + | ATOMIC_TYPE_FLAG_SIGNED, + .rank = 2, }, [ATOMIC_TYPE_UCHAR] = { - .size = 1, - .alignment = 1, - .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC, + .size = 1, + .alignment = 1, + .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC, + .rank = 2, }, [ATOMIC_TYPE_SHORT] = { .size = 2, .alignment = 2, .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC - | ATOMIC_TYPE_FLAG_SIGNED + | ATOMIC_TYPE_FLAG_SIGNED, + .rank = 3, }, [ATOMIC_TYPE_USHORT] = { .size = 2, .alignment = 2, .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC, + .rank = 3, }, [ATOMIC_TYPE_INT] = { .size = (unsigned) -1, .alignment = (unsigned) -1, .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC | ATOMIC_TYPE_FLAG_SIGNED, + .rank = 4, }, [ATOMIC_TYPE_UINT] = { .size = (unsigned) -1, .alignment = (unsigned) -1, .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC, + .rank = 4, }, [ATOMIC_TYPE_LONG] = { .size = (unsigned) -1, .alignment = (unsigned) -1, .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC | ATOMIC_TYPE_FLAG_SIGNED, + .rank = 5, }, [ATOMIC_TYPE_ULONG] = { .size = (unsigned) -1, .alignment = (unsigned) -1, .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC, + .rank = 5, }, [ATOMIC_TYPE_LONGLONG] = { - .size = (unsigned) -1, - .alignment = (unsigned) -1, + .size = 8, + .alignment = 8, .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC | ATOMIC_TYPE_FLAG_SIGNED, + .rank = 6, }, [ATOMIC_TYPE_ULONGLONG] = { - .size = (unsigned) -1, - .alignment = (unsigned) -1, - .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC, - }, - [ATOMIC_TYPE_BOOL] = { - .size = (unsigned) -1, - .alignment = (unsigned) -1, + .size = 8, + .alignment = 8, .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC, + .rank = 6, }, [ATOMIC_TYPE_FLOAT] = { .size = 4, - .alignment = (unsigned) -1, + .alignment = 4, .flags = ATOMIC_TYPE_FLAG_FLOAT | ATOMIC_TYPE_FLAG_ARITHMETIC | ATOMIC_TYPE_FLAG_SIGNED, + .rank = 0, }, [ATOMIC_TYPE_DOUBLE] = { .size = 8, - .alignment = (unsigned) -1, + .alignment = 8, .flags = ATOMIC_TYPE_FLAG_FLOAT | ATOMIC_TYPE_FLAG_ARITHMETIC | ATOMIC_TYPE_FLAG_SIGNED, + .rank = 0, }, - [ATOMIC_TYPE_LONG_DOUBLE] = { - .size = 12, - .alignment = (unsigned) -1, - .flags = ATOMIC_TYPE_FLAG_FLOAT | ATOMIC_TYPE_FLAG_ARITHMETIC - | ATOMIC_TYPE_FLAG_SIGNED, + [ATOMIC_TYPE_WCHAR_T] = { + .size = (unsigned)-1, + .alignment = (unsigned)-1, + .flags = ATOMIC_TYPE_FLAG_INTEGER | ATOMIC_TYPE_FLAG_ARITHMETIC, + .rank = (unsigned)-1, }, - /* complex and imaginary types are set in init_types */ +}; +atomic_type_properties_t pointer_properties = { + .size = 4, + .alignment = 4, + .flags = ATOMIC_TYPE_FLAG_NONE, }; -void init_types(void) +void init_types(unsigned machine_size) { - obstack_init(type_obst); + obstack_init(&type_obst); atomic_type_properties_t *props = atomic_type_properties; - if (char_is_signed) { - props[ATOMIC_TYPE_CHAR].flags |= ATOMIC_TYPE_FLAG_SIGNED; - } + /* atempt to set some sane defaults based on machine size */ unsigned int_size = machine_size < 32 ? 2 : 4; unsigned long_size = machine_size < 64 ? 4 : 8; - unsigned llong_size = machine_size < 32 ? 4 : 8; - - props[ATOMIC_TYPE_INT].size = int_size; - props[ATOMIC_TYPE_INT].alignment = int_size; - props[ATOMIC_TYPE_UINT].size = int_size; - props[ATOMIC_TYPE_UINT].alignment = int_size; - props[ATOMIC_TYPE_LONG].size = long_size; - props[ATOMIC_TYPE_LONG].alignment = long_size; - props[ATOMIC_TYPE_ULONG].size = long_size; - props[ATOMIC_TYPE_ULONG].alignment = long_size; - props[ATOMIC_TYPE_LONGLONG].size = llong_size; - props[ATOMIC_TYPE_LONGLONG].alignment = llong_size; - props[ATOMIC_TYPE_ULONGLONG].size = llong_size; - props[ATOMIC_TYPE_ULONGLONG].alignment = llong_size; - - /* TODO: backend specific, need a way to query the backend for this. - * The following are good settings for x86 */ - props[ATOMIC_TYPE_FLOAT].alignment = 4; - props[ATOMIC_TYPE_DOUBLE].alignment = 4; - props[ATOMIC_TYPE_LONG_DOUBLE].alignment = 4; - props[ATOMIC_TYPE_LONGLONG].alignment = 4; - props[ATOMIC_TYPE_ULONGLONG].alignment = 4; - if (firm_opt.os_support == OS_SUPPORT_MACHO) { - props[ATOMIC_TYPE_LONG_DOUBLE].size = 16; - props[ATOMIC_TYPE_LONG_DOUBLE].alignment = 16; - } - - /* TODO: make this configurable for platforms which do not use byte sized - * bools. */ - props[ATOMIC_TYPE_BOOL] = props[ATOMIC_TYPE_UCHAR]; - props[ATOMIC_TYPE_WCHAR_T] = props[wchar_atomic_kind]; + props[ATOMIC_TYPE_INT].size = int_size; + props[ATOMIC_TYPE_INT].alignment = int_size; + props[ATOMIC_TYPE_UINT].size = int_size; + props[ATOMIC_TYPE_UINT].alignment = int_size; + props[ATOMIC_TYPE_LONG].size = long_size; + props[ATOMIC_TYPE_LONG].alignment = long_size; + props[ATOMIC_TYPE_ULONG].size = long_size; + props[ATOMIC_TYPE_ULONG].alignment = long_size; + + pointer_properties.size = long_size; + pointer_properties.alignment = long_size; + pointer_properties.struct_alignment = long_size; + + props[ATOMIC_TYPE_LONG_DOUBLE] = props[ATOMIC_TYPE_DOUBLE]; + props[ATOMIC_TYPE_WCHAR_T] = props[ATOMIC_TYPE_INT]; + + /* set struct alignments to the same value as alignment */ + for (size_t i = 0; i != lengthof(atomic_type_properties); ++i) { + props[i].struct_alignment = props[i].alignment; + } } void exit_types(void) { - obstack_free(type_obst, NULL); + obstack_free(&type_obst, NULL); } -void type_set_output(FILE *stream) +void print_type_qualifiers(type_qualifiers_t const qualifiers, QualifierSeparators const q) { - out = stream; -} - -void print_type_qualifiers(type_qualifiers_t qualifiers) -{ - int first = 1; + size_t sep = q & QUAL_SEP_START ? 0 : 1; if (qualifiers & TYPE_QUALIFIER_CONST) { - fputs(" const" + first, out); - first = 0; + print_string(&" const"[sep]); + sep = 0; } if (qualifiers & TYPE_QUALIFIER_VOLATILE) { - fputs(" volatile" + first, out); - first = 0; + print_string(&" volatile"[sep]); + sep = 0; } if (qualifiers & TYPE_QUALIFIER_RESTRICT) { - fputs(" restrict" + first, out); - first = 0; + print_string(&" restrict"[sep]); + sep = 0; } + if (sep == 0 && q & QUAL_SEP_END) + print_char(' '); } const char *get_atomic_kind_name(atomic_type_kind_t kind) { - switch(kind) { - case ATOMIC_TYPE_INVALID: break; + switch (kind) { case ATOMIC_TYPE_VOID: return "void"; case ATOMIC_TYPE_WCHAR_T: return "wchar_t"; case ATOMIC_TYPE_BOOL: return c_mode & _CXX ? "bool" : "_Bool"; @@ -261,7 +268,7 @@ const char *get_atomic_kind_name(atomic_type_kind_t kind) static void print_atomic_kinds(atomic_type_kind_t kind) { const char *s = get_atomic_kind_name(kind); - fputs(s, out); + print_string(s); } /** @@ -271,9 +278,7 @@ static void print_atomic_kinds(atomic_type_kind_t kind) */ static void print_atomic_type(const atomic_type_t *type) { - print_type_qualifiers(type->base.qualifiers); - if (type->base.qualifiers != 0) - fputc(' ', out); + print_type_qualifiers(type->base.qualifiers, QUAL_SEP_END); print_atomic_kinds(type->akind); } @@ -282,12 +287,10 @@ static void print_atomic_type(const atomic_type_t *type) * * @param type The type. */ -static -void print_complex_type(const complex_type_t *type) +static void print_complex_type(const atomic_type_t *type) { - int empty = type->base.qualifiers == 0; - print_type_qualifiers(type->base.qualifiers); - fputs(" _Complex " + empty, out); + print_type_qualifiers(type->base.qualifiers, QUAL_SEP_END); + print_string("_Complex "); print_atomic_kinds(type->akind); } @@ -296,12 +299,10 @@ void print_complex_type(const complex_type_t *type) * * @param type The type. */ -static -void print_imaginary_type(const imaginary_type_t *type) +static void print_imaginary_type(const atomic_type_t *type) { - int empty = type->base.qualifiers == 0; - print_type_qualifiers(type->base.qualifiers); - fputs(" _Imaginary " + empty, out); + print_type_qualifiers(type->base.qualifiers, QUAL_SEP_END); + print_string("_Imaginary "); print_atomic_kinds(type->akind); } @@ -313,33 +314,28 @@ void print_imaginary_type(const imaginary_type_t *type) static void print_function_type_pre(const function_type_t *type) { switch (type->linkage) { - case LINKAGE_INVALID: - break; - case LINKAGE_C: if (c_mode & _CXX) - fputs("extern \"C\" ", out); + print_string("extern \"C\" "); break; case LINKAGE_CXX: if (!(c_mode & _CXX)) - fputs("extern \"C++\" ", out); + print_string("extern \"C++\" "); break; } - print_type_qualifiers(type->base.qualifiers); - if (type->base.qualifiers != 0) - fputc(' ', out); + print_type_qualifiers(type->base.qualifiers, QUAL_SEP_END); intern_print_type_pre(type->return_type); cc_kind_t cc = type->calling_convention; restart: switch (cc) { - case CC_CDECL: fputs(" __cdecl", out); break; - case CC_STDCALL: fputs(" __stdcall", out); break; - case CC_FASTCALL: fputs(" __fastcall", out); break; - case CC_THISCALL: fputs(" __thiscall", out); break; + case CC_CDECL: print_string(" __cdecl"); break; + case CC_STDCALL: print_string(" __stdcall"); break; + case CC_FASTCALL: print_string(" __fastcall"); break; + case CC_THISCALL: print_string(" __thiscall"); break; case CC_DEFAULT: if (default_calling_convention != CC_CDECL) { /* show the default calling convention if its not cdecl */ @@ -358,16 +354,12 @@ restart: static void print_function_type_post(const function_type_t *type, const scope_t *parameters) { - fputc('(', out); - bool first = true; + print_char('('); + separator_t sep = { "", ", " }; if (parameters == NULL) { function_parameter_t *parameter = type->parameters; - for( ; parameter != NULL; parameter = parameter->next) { - if (first) { - first = false; - } else { - fputs(", ", out); - } + for ( ; parameter != NULL; parameter = parameter->next) { + print_string(sep_next(&sep)); print_type(parameter->type); } } else { @@ -376,31 +368,23 @@ static void print_function_type_post(const function_type_t *type, if (parameter->kind != ENTITY_PARAMETER) continue; - if (first) { - first = false; - } else { - fputs(", ", out); - } - const type_t *const type = parameter->declaration.type; - if (type == NULL) { - fputs(parameter->base.symbol->string, out); + print_string(sep_next(&sep)); + const type_t *const param_type = parameter->declaration.type; + if (param_type == NULL) { + print_string(parameter->base.symbol->string); } else { - print_type_ext(type, parameter->base.symbol, NULL); + print_type_ext(param_type, parameter->base.symbol, NULL); } } } if (type->variadic) { - if (first) { - first = false; - } else { - fputs(", ", out); - } - fputs("...", out); + print_string(sep_next(&sep)); + print_string("..."); } - if (first && !type->unspecified_parameters) { - fputs("void", out); + if (sep_at_first(&sep) && !type->unspecified_parameters) { + print_string("void"); } - fputc(')', out); + print_char(')'); intern_print_type_post(type->return_type); } @@ -415,18 +399,15 @@ static void print_pointer_type_pre(const pointer_type_t *type) type_t const *const points_to = type->points_to; intern_print_type_pre(points_to); if (points_to->kind == TYPE_ARRAY || points_to->kind == TYPE_FUNCTION) - fputs(" (", out); + print_string(" ("); variable_t *const variable = type->base_variable; if (variable != NULL) { - fputs(" __based(", out); - fputs(variable->base.base.symbol->string, out); - fputs(") ", out); + print_string(" __based("); + print_string(variable->base.base.symbol->string); + print_string(") "); } - fputc('*', out); - type_qualifiers_t const qual = type->base.qualifiers; - if (qual != 0) - fputc(' ', out); - print_type_qualifiers(qual); + print_char('*'); + print_type_qualifiers(type->base.qualifiers, QUAL_SEP_START); } /** @@ -438,7 +419,7 @@ static void print_pointer_type_post(const pointer_type_t *type) { type_t const *const points_to = type->points_to; if (points_to->kind == TYPE_ARRAY || points_to->kind == TYPE_FUNCTION) - fputc(')', out); + print_char(')'); intern_print_type_post(points_to); } @@ -452,8 +433,8 @@ static void print_reference_type_pre(const reference_type_t *type) type_t const *const refers_to = type->refers_to; intern_print_type_pre(refers_to); if (refers_to->kind == TYPE_ARRAY || refers_to->kind == TYPE_FUNCTION) - fputs(" (", out); - fputc('&', out); + print_string(" ("); + print_char('&'); } /** @@ -465,7 +446,7 @@ static void print_reference_type_post(const reference_type_t *type) { type_t const *const refers_to = type->refers_to; if (refers_to->kind == TYPE_ARRAY || refers_to->kind == TYPE_FUNCTION) - fputc(')', out); + print_char(')'); intern_print_type_post(refers_to); } @@ -486,66 +467,41 @@ static void print_array_type_pre(const array_type_t *type) */ static void print_array_type_post(const array_type_t *type) { - fputc('[', out); + print_char('['); if (type->is_static) { - fputs("static ", out); + print_string("static "); } - print_type_qualifiers(type->base.qualifiers); - if (type->base.qualifiers != 0) - fputc(' ', out); + print_type_qualifiers(type->base.qualifiers, QUAL_SEP_END); if (type->size_expression != NULL && (print_implicit_array_size || !type->has_implicit_size)) { print_expression(type->size_expression); } - fputc(']', out); + print_char(']'); intern_print_type_post(type->element_type); } -/** - * Prints the postfix part of a bitfield type. - * - * @param type The array type. - */ -static void print_bitfield_type_post(const bitfield_type_t *type) -{ - fputs(" : ", out); - print_expression(type->size_expression); - intern_print_type_post(type->base_type); -} - -/** - * Prints an enum definition. - * - * @param declaration The enum's type declaration. - */ void print_enum_definition(const enum_t *enume) { - fputs("{\n", out); + print_string("{\n"); change_indent(1); entity_t *entry = enume->base.next; - for( ; entry != NULL && entry->kind == ENTITY_ENUM_VALUE; + for ( ; entry != NULL && entry->kind == ENTITY_ENUM_VALUE; entry = entry->base.next) { print_indent(); - fputs(entry->base.symbol->string, out); + print_string(entry->base.symbol->string); if (entry->enum_value.value != NULL) { - fputs(" = ", out); - - /* skip the implicit cast */ - expression_t *expression = entry->enum_value.value; - if (expression->kind == EXPR_UNARY_CAST_IMPLICIT) { - expression = expression->unary.value; - } - print_expression(expression); + print_string(" = "); + print_expression(entry->enum_value.value); } - fputs(",\n", out); + print_string(",\n"); } change_indent(-1); print_indent(); - fputc('}', out); + print_char('}'); } /** @@ -555,66 +511,56 @@ void print_enum_definition(const enum_t *enume) */ static void print_type_enum(const enum_type_t *type) { - int empty = type->base.qualifiers == 0; - print_type_qualifiers(type->base.qualifiers); - fputs(" enum " + empty, out); + print_type_qualifiers(type->base.base.qualifiers, QUAL_SEP_END); + print_string("enum "); enum_t *enume = type->enume; symbol_t *symbol = enume->base.symbol; if (symbol != NULL) { - fputs(symbol->string, out); + print_string(symbol->string); } else { print_enum_definition(enume); } } -/** - * Print the compound part of a compound type. - */ void print_compound_definition(const compound_t *compound) { - fputs("{\n", out); + print_string("{\n"); change_indent(1); entity_t *entity = compound->members.entities; - for( ; entity != NULL; entity = entity->base.next) { + for ( ; entity != NULL; entity = entity->base.next) { if (entity->kind != ENTITY_COMPOUND_MEMBER) continue; print_indent(); print_entity(entity); - fputc('\n', out); + print_char('\n'); } change_indent(-1); print_indent(); - fputc('}', out); + print_char('}'); if (compound->modifiers & DM_TRANSPARENT_UNION) { - fputs("__attribute__((__transparent_union__))", out); + print_string("__attribute__((__transparent_union__))"); } } /** * Prints a compound type. * + * @param kind The name of the compound kind. * @param type The compound type. */ -static void print_compound_type(const compound_type_t *type) +static void print_compound_type(char const *const kind, compound_type_t const *const type) { - int empty = type->base.qualifiers == 0; - print_type_qualifiers(type->base.qualifiers); - - if (type->base.kind == TYPE_COMPOUND_STRUCT) { - fputs(" struct " + empty, out); - } else { - assert(type->base.kind == TYPE_COMPOUND_UNION); - fputs(" union " + empty, out); - } + print_type_qualifiers(type->base.qualifiers, QUAL_SEP_END); + print_string(kind); compound_t *compound = type->compound; symbol_t *symbol = compound->base.symbol; if (symbol != NULL) { - fputs(symbol->string, out); + print_string(symbol->string); } else { print_compound_definition(compound); } @@ -627,10 +573,8 @@ static void print_compound_type(const compound_type_t *type) */ static void print_typedef_type_pre(const typedef_type_t *const type) { - print_type_qualifiers(type->base.qualifiers); - if (type->base.qualifiers != 0) - fputc(' ', out); - fputs(type->typedefe->base.symbol->string, out); + print_type_qualifiers(type->base.qualifiers, QUAL_SEP_END); + print_string(type->typedefe->base.symbol->string); } /** @@ -640,13 +584,13 @@ static void print_typedef_type_pre(const typedef_type_t *const type) */ static void print_typeof_type_pre(const typeof_type_t *const type) { - fputs("typeof(", out); + print_string("typeof("); if (type->expression != NULL) { print_expression(type->expression); } else { print_type(type->typeof_type); } - fputc(')', out); + print_char(')'); } /** @@ -656,55 +600,22 @@ static void print_typeof_type_pre(const typeof_type_t *const type) */ static void intern_print_type_pre(const type_t *const type) { - switch(type->kind) { - case TYPE_ERROR: - fputs("", out); - return; - case TYPE_INVALID: - fputs("", out); - return; - case TYPE_ENUM: - print_type_enum(&type->enumt); - return; - case TYPE_ATOMIC: - print_atomic_type(&type->atomic); - return; - case TYPE_COMPLEX: - print_complex_type(&type->complex); - return; - case TYPE_IMAGINARY: - print_imaginary_type(&type->imaginary); - return; - case TYPE_COMPOUND_STRUCT: - case TYPE_COMPOUND_UNION: - print_compound_type(&type->compound); - return; - case TYPE_BUILTIN: - fputs(type->builtin.symbol->string, out); - return; - case TYPE_FUNCTION: - print_function_type_pre(&type->function); - return; - case TYPE_POINTER: - print_pointer_type_pre(&type->pointer); - return; - case TYPE_REFERENCE: - print_reference_type_pre(&type->reference); - return; - case TYPE_BITFIELD: - intern_print_type_pre(type->bitfield.base_type); - return; - case TYPE_ARRAY: - print_array_type_pre(&type->array); - return; - case TYPE_TYPEDEF: - print_typedef_type_pre(&type->typedeft); - return; - case TYPE_TYPEOF: - print_typeof_type_pre(&type->typeoft); - return; + switch (type->kind) { + case TYPE_ARRAY: print_array_type_pre( &type->array); return; + case TYPE_ATOMIC: print_atomic_type( &type->atomic); return; + case TYPE_COMPLEX: print_complex_type( &type->atomic); return; + case TYPE_COMPOUND_STRUCT: print_compound_type("struct ", &type->compound); return; + case TYPE_COMPOUND_UNION: print_compound_type("union ", &type->compound); return; + case TYPE_ENUM: print_type_enum( &type->enumt); return; + case TYPE_ERROR: print_string(""); return; + case TYPE_FUNCTION: print_function_type_pre( &type->function); return; + case TYPE_IMAGINARY: print_imaginary_type( &type->atomic); return; + case TYPE_POINTER: print_pointer_type_pre( &type->pointer); return; + case TYPE_REFERENCE: print_reference_type_pre( &type->reference); return; + case TYPE_TYPEDEF: print_typedef_type_pre( &type->typedeft); return; + case TYPE_TYPEOF: print_typeof_type_pre( &type->typeoft); return; } - fputs("unknown", out); + print_string("unknown"); } /** @@ -714,7 +625,7 @@ static void intern_print_type_pre(const type_t *const type) */ static void intern_print_type_post(const type_t *const type) { - switch(type->kind) { + switch (type->kind) { case TYPE_FUNCTION: print_function_type_post(&type->function, NULL); return; @@ -727,29 +638,19 @@ static void intern_print_type_post(const type_t *const type) case TYPE_ARRAY: print_array_type_post(&type->array); return; - case TYPE_BITFIELD: - print_bitfield_type_post(&type->bitfield); - return; case TYPE_ERROR: - case TYPE_INVALID: case TYPE_ATOMIC: case TYPE_COMPLEX: case TYPE_IMAGINARY: case TYPE_ENUM: case TYPE_COMPOUND_STRUCT: case TYPE_COMPOUND_UNION: - case TYPE_BUILTIN: case TYPE_TYPEOF: case TYPE_TYPEDEF: break; } } -/** - * Prints a type. - * - * @param type The type. - */ void print_type(const type_t *const type) { print_type_ext(type, NULL, NULL); @@ -758,15 +659,10 @@ void print_type(const type_t *const type) void print_type_ext(const type_t *const type, const symbol_t *symbol, const scope_t *parameters) { - if (type == NULL) { - fputs("nil type", out); - return; - } - intern_print_type_pre(type); if (symbol != NULL) { - fputc(' ', out); - fputs(symbol->string, out); + print_char(' '); + print_string(symbol->string); } if (type->kind == TYPE_FUNCTION) { print_function_type_post(&type->function, parameters); @@ -775,59 +671,16 @@ void print_type_ext(const type_t *const type, const symbol_t *symbol, } } -/** - * Return the size of a type AST node. - * - * @param type The type. - */ -static size_t get_type_struct_size(const type_t *type) -{ - switch(type->kind) { - case TYPE_ATOMIC: return sizeof(atomic_type_t); - case TYPE_COMPLEX: return sizeof(complex_type_t); - case TYPE_IMAGINARY: return sizeof(imaginary_type_t); - case TYPE_COMPOUND_STRUCT: - case TYPE_COMPOUND_UNION: return sizeof(compound_type_t); - case TYPE_ENUM: return sizeof(enum_type_t); - case TYPE_FUNCTION: return sizeof(function_type_t); - case TYPE_POINTER: return sizeof(pointer_type_t); - case TYPE_REFERENCE: return sizeof(reference_type_t); - case TYPE_ARRAY: return sizeof(array_type_t); - case TYPE_BUILTIN: return sizeof(builtin_type_t); - case TYPE_TYPEDEF: return sizeof(typedef_type_t); - case TYPE_TYPEOF: return sizeof(typeof_type_t); - case TYPE_BITFIELD: return sizeof(bitfield_type_t); - case TYPE_ERROR: panic("error type found"); - case TYPE_INVALID: panic("invalid type found"); - } - panic("unknown type found"); -} - -/** - * Duplicates a type. - * - * @param type The type to copy. - * @return A copy of the type. - * - * @note This does not produce a deep copy! - */ type_t *duplicate_type(const type_t *type) { - size_t size = get_type_struct_size(type); + size_t size = get_type_struct_size(type->kind); - type_t *copy = obstack_alloc(type_obst, size); - memcpy(copy, type, size); + type_t *const copy = obstack_copy(&type_obst, type, size); copy->base.firm_type = NULL; return copy; } -/** - * Returns the unqualified type of a given type. - * - * @param type The type. - * @returns The unqualified type. - */ type_t *get_unqualified_type(type_t *type) { assert(!is_typeref(type)); @@ -857,7 +710,7 @@ type_t *get_qualified_type(type_t *orig_type, type_qualifiers_t const qual) copy = duplicate_type(type); copy->array.element_type = qual_element_type; } else if (is_type_valid(type)) { - if ((type->base.qualifiers & qual) == qual) + if ((type->base.qualifiers & qual) == (int)qual) return orig_type; copy = duplicate_type(type); @@ -869,17 +722,6 @@ type_t *get_qualified_type(type_t *orig_type, type_qualifiers_t const qual) return identify_new_type(copy); } -/** - * Check if a type is valid. - * - * @param type The type to check. - * @return true if type represents a valid type. - */ -bool type_valid(const type_t *type) -{ - return type->kind != TYPE_INVALID; -} - static bool test_atomic_type_flag(atomic_type_kind_t kind, atomic_type_flag_t flag) { @@ -887,45 +729,20 @@ static bool test_atomic_type_flag(atomic_type_kind_t kind, return (atomic_type_properties[kind].flags & flag) != 0; } -/** - * Returns true if the given type is an integer type. - * - * @param type The type to check. - * @return True if type is an integer type. - */ bool is_type_integer(const type_t *type) { assert(!is_typeref(type)); - - if (type->kind == TYPE_ENUM) - return true; - if (type->kind == TYPE_BITFIELD) - return true; - - if (type->kind != TYPE_ATOMIC) + if (!is_type_arithmetic(type)) return false; - return test_atomic_type_flag(type->atomic.akind, ATOMIC_TYPE_FLAG_INTEGER); } -/** - * Returns true if the given type is an enum type. - * - * @param type The type to check. - * @return True if type is an enum type. - */ bool is_type_enum(const type_t *type) { assert(!is_typeref(type)); return type->kind == TYPE_ENUM; } -/** - * Returns true if the given type is an floating point type. - * - * @param type The type to check. - * @return True if type is a floating point type. - */ bool is_type_float(const type_t *type) { assert(!is_typeref(type)); @@ -936,111 +753,64 @@ bool is_type_float(const type_t *type) return test_atomic_type_flag(type->atomic.akind, ATOMIC_TYPE_FLAG_FLOAT); } -/** - * Returns true if the given type is an complex type. - * - * @param type The type to check. - * @return True if type is a complex type. - */ bool is_type_complex(const type_t *type) { assert(!is_typeref(type)); - - if (type->kind != TYPE_ATOMIC) - return false; - - return test_atomic_type_flag(type->atomic.akind, ATOMIC_TYPE_FLAG_COMPLEX); + return type->kind == TYPE_COMPLEX; } -/** - * Returns true if the given type is a signed type. - * - * @param type The type to check. - * @return True if type is a signed type. - */ bool is_type_signed(const type_t *type) { assert(!is_typeref(type)); - - /* enum types are int for now */ - if (type->kind == TYPE_ENUM) - return true; - if (type->kind == TYPE_BITFIELD) - return is_type_signed(type->bitfield.base_type); - - if (type->kind != TYPE_ATOMIC) + if (!is_type_arithmetic(type)) return false; - return test_atomic_type_flag(type->atomic.akind, ATOMIC_TYPE_FLAG_SIGNED); } -/** - * Returns true if the given type represents an arithmetic type. - * - * @param type The type to check. - * @return True if type represents an arithmetic type. - */ bool is_type_arithmetic(const type_t *type) { assert(!is_typeref(type)); - switch(type->kind) { - case TYPE_BITFIELD: + switch (type->kind) { case TYPE_ENUM: return true; case TYPE_ATOMIC: - return test_atomic_type_flag(type->atomic.akind, ATOMIC_TYPE_FLAG_ARITHMETIC); case TYPE_COMPLEX: - return test_atomic_type_flag(type->complex.akind, ATOMIC_TYPE_FLAG_ARITHMETIC); case TYPE_IMAGINARY: - return test_atomic_type_flag(type->imaginary.akind, ATOMIC_TYPE_FLAG_ARITHMETIC); + return test_atomic_type_flag(type->atomic.akind, ATOMIC_TYPE_FLAG_ARITHMETIC); default: return false; } } -/** - * Returns true if the given type is an integer or float type. - * - * @param type The type to check. - * @return True if type is an integer or float type. - */ bool is_type_real(const type_t *type) { /* 6.2.5 (17) */ return is_type_integer(type) || is_type_float(type); } -/** - * Returns true if the given type represents a scalar type. - * - * @param type The type to check. - * @return True if type represents a scalar type. - */ bool is_type_scalar(const type_t *type) { assert(!is_typeref(type)); switch (type->kind) { - case TYPE_POINTER: return true; - case TYPE_BUILTIN: return is_type_scalar(type->builtin.real_type); - default: break; + case TYPE_POINTER: + case TYPE_ENUM: + return true; + case TYPE_ATOMIC: + case TYPE_COMPLEX: + case TYPE_IMAGINARY: + return test_atomic_type_flag(type->atomic.akind, ATOMIC_TYPE_FLAG_ARITHMETIC); + default: + return false; } - - return is_type_arithmetic(type); } -/** - * Check if a given type is incomplete. - * - * @param type The type to check. - * @return True if the given type is incomplete (ie. just forward). - */ bool is_type_incomplete(const type_t *type) { assert(!is_typeref(type)); - switch(type->kind) { + switch (type->kind) { case TYPE_COMPOUND_STRUCT: case TYPE_COMPOUND_UNION: { const compound_type_t *compound_type = &type->compound; @@ -1054,30 +824,22 @@ bool is_type_incomplete(const type_t *type) && !type->array.size_constant; case TYPE_ATOMIC: - return type->atomic.akind == ATOMIC_TYPE_VOID; - - case TYPE_COMPLEX: - return type->complex.akind == ATOMIC_TYPE_VOID; - case TYPE_IMAGINARY: - return type->imaginary.akind == ATOMIC_TYPE_VOID; + case TYPE_COMPLEX: + return type->atomic.akind == ATOMIC_TYPE_VOID; - case TYPE_BITFIELD: case TYPE_FUNCTION: case TYPE_POINTER: case TYPE_REFERENCE: - case TYPE_BUILTIN: case TYPE_ERROR: return false; case TYPE_TYPEDEF: case TYPE_TYPEOF: - panic("is_type_incomplete called without typerefs skipped"); - case TYPE_INVALID: - break; + panic("typedef not skipped"); } - panic("invalid type found"); + panic("invalid type"); } bool is_type_object(const type_t *type) @@ -1085,14 +847,6 @@ bool is_type_object(const type_t *type) return !is_type_function(type) && !is_type_incomplete(type); } -bool is_builtin_va_list(type_t *type) -{ - type_t *tp = skip_typeref(type); - - return tp->kind == type_valist->kind && - tp->builtin.symbol == type_valist->builtin.symbol; -} - /** * Check if two function types are compatible. */ @@ -1117,13 +871,14 @@ static bool function_types_compatible(const function_type_t *func1, if (cc1 != cc2) return false; - /* can parameters be compared? */ - if (func1->unspecified_parameters || func2->unspecified_parameters) - return true; - if (func1->variadic != func2->variadic) return false; + /* can parameters be compared? */ + if ((func1->unspecified_parameters && !func1->kr_style_parameters) + || (func2->unspecified_parameters && !func2->kr_style_parameters)) + return true; + /* TODO: handling of unspecified parameters not correct yet */ /* all argument types must be compatible */ @@ -1164,9 +919,6 @@ static bool array_types_compatible(const array_type_t *array1, return array1->size == array2->size; } -/** - * Check if two types are compatible. - */ bool types_compatible(const type_t *type1, const type_t *type2) { assert(!is_typeref(type1)); @@ -1176,66 +928,48 @@ bool types_compatible(const type_t *type1, const type_t *type2) if (type1 == type2) return true; - if (!is_type_valid(type1) || !is_type_valid(type2)) - return true; - - if (type1->base.qualifiers != type2->base.qualifiers) - return false; - if (type1->kind != type2->kind) - return false; - - switch (type1->kind) { - case TYPE_FUNCTION: - return function_types_compatible(&type1->function, &type2->function); - case TYPE_ATOMIC: - return type1->atomic.akind == type2->atomic.akind; - case TYPE_COMPLEX: - return type1->complex.akind == type2->complex.akind; - case TYPE_IMAGINARY: - return type1->imaginary.akind == type2->imaginary.akind; - case TYPE_ARRAY: - return array_types_compatible(&type1->array, &type2->array); - - case TYPE_POINTER: { - const type_t *const to1 = skip_typeref(type1->pointer.points_to); - const type_t *const to2 = skip_typeref(type2->pointer.points_to); - return types_compatible(to1, to2); - } - - case TYPE_REFERENCE: { - const type_t *const to1 = skip_typeref(type1->reference.refers_to); - const type_t *const to2 = skip_typeref(type2->reference.refers_to); - return types_compatible(to1, to2); - } + if (type1->base.qualifiers == type2->base.qualifiers && + type1->kind == type2->kind) { + switch (type1->kind) { + case TYPE_FUNCTION: + return function_types_compatible(&type1->function, &type2->function); + case TYPE_ATOMIC: + case TYPE_IMAGINARY: + case TYPE_COMPLEX: + return type1->atomic.akind == type2->atomic.akind; + case TYPE_ARRAY: + return array_types_compatible(&type1->array, &type2->array); - case TYPE_COMPOUND_STRUCT: - case TYPE_COMPOUND_UNION: { + case TYPE_POINTER: { + const type_t *const to1 = skip_typeref(type1->pointer.points_to); + const type_t *const to2 = skip_typeref(type2->pointer.points_to); + return types_compatible(to1, to2); + } + case TYPE_REFERENCE: { + const type_t *const to1 = skip_typeref(type1->reference.refers_to); + const type_t *const to2 = skip_typeref(type2->reference.refers_to); + return types_compatible(to1, to2); + } - break; - } - case TYPE_ENUM: - case TYPE_BUILTIN: - /* TODO: not implemented */ - break; + case TYPE_COMPOUND_STRUCT: + case TYPE_COMPOUND_UNION: + break; - case TYPE_BITFIELD: - /* not sure if this makes sense or is even needed, implement it if you - * really need it! */ - panic("type compatibility check for bitfield type"); + case TYPE_ENUM: + /* TODO: not implemented */ + break; - case TYPE_ERROR: - /* Hmm, the error type should be compatible to all other types */ - return true; - case TYPE_INVALID: - panic("invalid type found in compatible types"); - case TYPE_TYPEDEF: - case TYPE_TYPEOF: - panic("typerefs not skipped in compatible types?!?"); + case TYPE_ERROR: + /* Hmm, the error type should be compatible to all other types */ + return true; + case TYPE_TYPEDEF: + case TYPE_TYPEOF: + panic("typeref not skipped"); + } } - /* TODO: incomplete */ - return false; + return !is_type_valid(type1) || !is_type_valid(type2); } /** @@ -1293,107 +1027,88 @@ type_t *skip_typeref(type_t *type) unsigned get_type_size(type_t *type) { switch (type->kind) { - case TYPE_INVALID: - break; case TYPE_ERROR: return 0; case TYPE_ATOMIC: + case TYPE_IMAGINARY: + case TYPE_ENUM: return get_atomic_type_size(type->atomic.akind); case TYPE_COMPLEX: - return get_atomic_type_size(type->complex.akind) * 2; - case TYPE_IMAGINARY: - return get_atomic_type_size(type->imaginary.akind); + return get_atomic_type_size(type->atomic.akind) * 2; case TYPE_COMPOUND_UNION: layout_union_type(&type->compound); return type->compound.compound->size; case TYPE_COMPOUND_STRUCT: layout_struct_type(&type->compound); return type->compound.compound->size; - case TYPE_ENUM: - return get_atomic_type_size(type->enumt.akind); case TYPE_FUNCTION: - return 0; /* non-const (but "address-const") */ + return 1; /* strange GNU extensions: sizeof(function) == 1 */ case TYPE_REFERENCE: case TYPE_POINTER: - /* TODO: make configurable by backend */ - return 4; + return pointer_properties.size; case TYPE_ARRAY: { /* TODO: correct if element_type is aligned? */ il_size_t element_size = get_type_size(type->array.element_type); return type->array.size * element_size; } - case TYPE_BITFIELD: - return 0; - case TYPE_BUILTIN: - return get_type_size(type->builtin.real_type); case TYPE_TYPEDEF: return get_type_size(type->typedeft.typedefe->type); case TYPE_TYPEOF: - if (type->typeoft.typeof_type) { - return get_type_size(type->typeoft.typeof_type); - } else { - return get_type_size(type->typeoft.expression->base.type); - } + return get_type_size(type->typeoft.typeof_type); } - panic("invalid type in get_type_size"); + panic("invalid type"); } unsigned get_type_alignment(type_t *type) { switch (type->kind) { - case TYPE_INVALID: - break; case TYPE_ERROR: return 0; case TYPE_ATOMIC: - return get_atomic_type_alignment(type->atomic.akind); - case TYPE_COMPLEX: - return get_atomic_type_alignment(type->complex.akind); case TYPE_IMAGINARY: - return get_atomic_type_alignment(type->imaginary.akind); + case TYPE_COMPLEX: + case TYPE_ENUM: + return get_atomic_type_alignment(type->atomic.akind); case TYPE_COMPOUND_UNION: layout_union_type(&type->compound); return type->compound.compound->alignment; case TYPE_COMPOUND_STRUCT: layout_struct_type(&type->compound); return type->compound.compound->alignment; - case TYPE_ENUM: - return get_atomic_type_alignment(type->enumt.akind); case TYPE_FUNCTION: - /* what is correct here? */ - return 4; + /* gcc says 1 here... */ + return 1; case TYPE_REFERENCE: case TYPE_POINTER: - /* TODO: make configurable by backend */ - return 4; + return pointer_properties.alignment; case TYPE_ARRAY: return get_type_alignment(type->array.element_type); - case TYPE_BITFIELD: - return 0; - case TYPE_BUILTIN: - return get_type_alignment(type->builtin.real_type); case TYPE_TYPEDEF: { - il_alignment_t alignment - = get_type_alignment(type->typedeft.typedefe->type); - if (type->typedeft.typedefe->alignment > alignment) - alignment = type->typedeft.typedefe->alignment; - - return alignment; + il_alignment_t const alignment = get_type_alignment(type->typedeft.typedefe->type); + return MAX(alignment, type->typedeft.typedefe->alignment); } case TYPE_TYPEOF: - if (type->typeoft.typeof_type) { - return get_type_alignment(type->typeoft.typeof_type); - } else { - return get_type_alignment(type->typeoft.expression->base.type); - } + return get_type_alignment(type->typeoft.typeof_type); } - panic("invalid type in get_type_alignment"); + panic("invalid type"); +} + +/** + * get alignment of a type when used inside a compound. + * Some ABIs are broken and alignment inside a compound is different from + * recommended alignment of a type + */ +static unsigned get_type_alignment_compound(type_t *const type) +{ + assert(!is_typeref(type)); + if (type->kind == TYPE_ATOMIC) + return atomic_type_properties[type->atomic.akind].struct_alignment; + return get_type_alignment(type); } decl_modifiers_t get_type_modifiers(const type_t *type) { - switch(type->kind) { - case TYPE_INVALID: + switch (type->kind) { case TYPE_ERROR: break; case TYPE_COMPOUND_STRUCT: @@ -1407,24 +1122,17 @@ decl_modifiers_t get_type_modifiers(const type_t *type) case TYPE_IMAGINARY: case TYPE_REFERENCE: case TYPE_POINTER: - case TYPE_BITFIELD: case TYPE_ARRAY: return 0; - case TYPE_BUILTIN: - return get_type_modifiers(type->builtin.real_type); case TYPE_TYPEDEF: { decl_modifiers_t modifiers = type->typedeft.typedefe->modifiers; modifiers |= get_type_modifiers(type->typedeft.typedefe->type); return modifiers; } case TYPE_TYPEOF: - if (type->typeoft.typeof_type) { - return get_type_modifiers(type->typeoft.typeof_type); - } else { - return get_type_modifiers(type->typeoft.expression->base.type); - } + return get_type_modifiers(type->typeoft.typeof_type); } - panic("invalid type found in get_type_modifiers"); + panic("invalid type"); } type_qualifiers_t get_type_qualifier(const type_t *type, bool skip_array_type) @@ -1478,26 +1186,6 @@ unsigned get_atomic_type_flags(atomic_type_kind_t kind) return atomic_type_properties[kind].flags; } -atomic_type_kind_t get_intptr_kind(void) -{ - if (machine_size <= 32) - return ATOMIC_TYPE_INT; - else if (machine_size <= 64) - return ATOMIC_TYPE_LONG; - else - return ATOMIC_TYPE_LONGLONG; -} - -atomic_type_kind_t get_uintptr_kind(void) -{ - if (machine_size <= 32) - return ATOMIC_TYPE_UINT; - else if (machine_size <= 64) - return ATOMIC_TYPE_ULONG; - else - return ATOMIC_TYPE_ULONGLONG; -} - /** * Find the atomic type kind representing a given size (signed). */ @@ -1507,7 +1195,7 @@ atomic_type_kind_t find_signed_int_atomic_type_kind_for_size(unsigned size) assert(size < 32); atomic_type_kind_t kind = kinds[size]; - if (kind == ATOMIC_TYPE_INVALID) { + if (kind == (atomic_type_kind_t)0) { static const atomic_type_kind_t possible_kinds[] = { ATOMIC_TYPE_SCHAR, ATOMIC_TYPE_SHORT, @@ -1535,7 +1223,7 @@ atomic_type_kind_t find_unsigned_int_atomic_type_kind_for_size(unsigned size) assert(size < 32); atomic_type_kind_t kind = kinds[size]; - if (kind == ATOMIC_TYPE_INVALID) { + if (kind == (atomic_type_kind_t)0) { static const atomic_type_kind_t possible_kinds[] = { ATOMIC_TYPE_UCHAR, ATOMIC_TYPE_USHORT, @@ -1562,7 +1250,7 @@ type_t *identify_new_type(type_t *type) { type_t *result = typehash_insert(type); if (result != type) { - obstack_free(type_obst, type); + obstack_free(&type_obst, type); } return result; } @@ -1575,10 +1263,7 @@ type_t *identify_new_type(type_t *type) */ type_t *make_atomic_type(atomic_type_kind_t akind, type_qualifiers_t qualifiers) { - type_t *type = obstack_alloc(type_obst, sizeof(atomic_type_t)); - memset(type, 0, sizeof(atomic_type_t)); - - type->kind = TYPE_ATOMIC; + type_t *const type = allocate_type_zero(TYPE_ATOMIC); type->base.qualifiers = qualifiers; type->atomic.akind = akind; @@ -1591,14 +1276,12 @@ type_t *make_atomic_type(atomic_type_kind_t akind, type_qualifiers_t qualifiers) * @param akind The kind of the atomic type. * @param qualifiers Type qualifiers for the new type. */ -type_t *make_complex_type(atomic_type_kind_t akind, type_qualifiers_t qualifiers) +type_t *make_complex_type(atomic_type_kind_t akind, + type_qualifiers_t qualifiers) { - type_t *type = obstack_alloc(type_obst, sizeof(complex_type_t)); - memset(type, 0, sizeof(complex_type_t)); - - type->kind = TYPE_COMPLEX; + type_t *const type = allocate_type_zero(TYPE_COMPLEX); type->base.qualifiers = qualifiers; - type->complex.akind = akind; + type->atomic.akind = akind; return identify_new_type(type); } @@ -1609,14 +1292,12 @@ type_t *make_complex_type(atomic_type_kind_t akind, type_qualifiers_t qualifiers * @param akind The kind of the atomic type. * @param qualifiers Type qualifiers for the new type. */ -type_t *make_imaginary_type(atomic_type_kind_t akind, type_qualifiers_t qualifiers) +type_t *make_imaginary_type(atomic_type_kind_t akind, + type_qualifiers_t qualifiers) { - type_t *type = obstack_alloc(type_obst, sizeof(imaginary_type_t)); - memset(type, 0, sizeof(imaginary_type_t)); - - type->kind = TYPE_IMAGINARY; + type_t *const type = allocate_type_zero(TYPE_IMAGINARY); type->base.qualifiers = qualifiers; - type->imaginary.akind = akind; + type->atomic.akind = akind; return identify_new_type(type); } @@ -1629,10 +1310,7 @@ type_t *make_imaginary_type(atomic_type_kind_t akind, type_qualifiers_t qualifie */ type_t *make_pointer_type(type_t *points_to, type_qualifiers_t qualifiers) { - type_t *type = obstack_alloc(type_obst, sizeof(pointer_type_t)); - memset(type, 0, sizeof(pointer_type_t)); - - type->kind = TYPE_POINTER; + type_t *const type = allocate_type_zero(TYPE_POINTER); type->base.qualifiers = qualifiers; type->pointer.points_to = points_to; type->pointer.base_variable = NULL; @@ -1647,11 +1325,8 @@ type_t *make_pointer_type(type_t *points_to, type_qualifiers_t qualifiers) */ type_t *make_reference_type(type_t *refers_to) { - type_t *type = obstack_alloc(type_obst, sizeof(reference_type_t)); - memset(type, 0, sizeof(reference_type_t)); - - type->kind = TYPE_REFERENCE; - type->base.qualifiers = 0; + type_t *const type = allocate_type_zero(TYPE_REFERENCE); + type->base.qualifiers = TYPE_QUALIFIER_NONE; type->reference.refers_to = refers_to; return identify_new_type(type); @@ -1667,10 +1342,7 @@ type_t *make_reference_type(type_t *refers_to) type_t *make_based_pointer_type(type_t *points_to, type_qualifiers_t qualifiers, variable_t *variable) { - type_t *type = obstack_alloc(type_obst, sizeof(pointer_type_t)); - memset(type, 0, sizeof(pointer_type_t)); - - type->kind = TYPE_POINTER; + type_t *const type = allocate_type_zero(TYPE_POINTER); type->base.qualifiers = qualifiers; type->pointer.points_to = points_to; type->pointer.base_variable = variable; @@ -1682,10 +1354,7 @@ type_t *make_based_pointer_type(type_t *points_to, type_t *make_array_type(type_t *element_type, size_t size, type_qualifiers_t qualifiers) { - type_t *type = obstack_alloc(type_obst, sizeof(array_type_t)); - memset(type, 0, sizeof(array_type_t)); - - type->kind = TYPE_ARRAY; + type_t *const type = allocate_type_zero(TYPE_ARRAY); type->base.qualifiers = qualifiers; type->array.element_type = element_type; type->array.size = size; @@ -1705,19 +1374,16 @@ static entity_t *pack_bitfield_members(il_size_t *struct_offset, entity_t *member; for (member = first; member != NULL; member = member->base.next) { if (member->kind != ENTITY_COMPOUND_MEMBER) + continue; + if (!member->compound_member.bitfield) break; - type_t *type = member->declaration.type; - if (type->kind != TYPE_BITFIELD) - break; - - type_t *base_type = skip_typeref(type->bitfield.base_type); - il_alignment_t base_alignment = get_type_alignment(base_type); + type_t *const base_type = skip_typeref(member->declaration.type); + il_alignment_t base_alignment = get_type_alignment_compound(base_type); il_alignment_t alignment_mask = base_alignment-1; - if (base_alignment > alignment) - alignment = base_alignment; + alignment = MAX(alignment, base_alignment); - size_t bit_size = type->bitfield.bit_size; + size_t bit_size = member->compound_member.bit_size; if (!packed) { bit_offset += (offset & alignment_mask) * BITS_PER_BYTE; offset &= ~alignment_mask; @@ -1730,8 +1396,14 @@ static entity_t *pack_bitfield_members(il_size_t *struct_offset, } } - member->compound_member.offset = offset; - member->compound_member.bit_offset = bit_offset; + if (byte_order_big_endian) { + size_t base_size = get_type_size(base_type) * BITS_PER_BYTE; + member->compound_member.offset = offset & ~alignment_mask; + member->compound_member.bit_offset = base_size - bit_offset - bit_size; + } else { + member->compound_member.offset = offset; + member->compound_member.bit_offset = bit_offset; + } bit_offset += bit_size; offset += bit_offset / BITS_PER_BYTE; @@ -1743,14 +1415,9 @@ static entity_t *pack_bitfield_members(il_size_t *struct_offset, *struct_offset = offset; *struct_alignment = alignment; - return member; } -/** - * Finish the construction of a struct type by calculating its size, offsets, - * alignment. - */ void layout_struct_type(compound_type_t *type) { assert(type->compound != NULL); @@ -1760,6 +1427,7 @@ void layout_struct_type(compound_type_t *type) return; if (type->compound->layouted) return; + compound->layouted = true; il_size_t offset = 0; il_alignment_t alignment = compound->alignment; @@ -1767,31 +1435,24 @@ void layout_struct_type(compound_type_t *type) entity_t *entry = compound->members.entities; while (entry != NULL) { - if (entry->kind != ENTITY_COMPOUND_MEMBER) { - entry = entry->base.next; - continue; - } + if (entry->kind != ENTITY_COMPOUND_MEMBER) + goto next; - type_t *m_type = entry->declaration.type; - type_t *skipped = skip_typeref(m_type); - if (! is_type_valid(skipped)) { - entry = entry->base.next; - continue; - } + type_t *const m_type = skip_typeref(entry->declaration.type); + if (!is_type_valid(m_type)) + goto next; - if (skipped->kind == TYPE_BITFIELD) { + if (entry->compound_member.bitfield) { entry = pack_bitfield_members(&offset, &alignment, compound->packed, entry); continue; } - il_alignment_t m_alignment = get_type_alignment(m_type); - if (m_alignment > alignment) - alignment = m_alignment; + il_alignment_t m_alignment = get_type_alignment_compound(m_type); + alignment = MAX(alignment, m_alignment); if (!compound->packed) { - il_size_t new_offset = (offset + m_alignment-1) & -m_alignment; - + il_size_t const new_offset = round_up2(offset, m_alignment); if (new_offset > offset) { need_pad = true; offset = new_offset; @@ -1801,36 +1462,29 @@ void layout_struct_type(compound_type_t *type) entry->compound_member.offset = offset; offset += get_type_size(m_type); +next: entry = entry->base.next; } if (!compound->packed) { - il_size_t new_offset = (offset + alignment-1) & -alignment; + il_size_t const new_offset = round_up2(offset, alignment); if (new_offset > offset) { need_pad = true; offset = new_offset; } } + position_t const *const pos = &compound->base.pos; if (need_pad) { - if (warning.padded) { - warningf(&compound->base.source_position, "'%T' needs padding", - type); - } - } else if (compound->packed && warning.packed) { - warningf(&compound->base.source_position, - "superfluous packed attribute on '%T'", type); + warningf(WARN_PADDED, pos, "'%T' needs padding", type); + } else if (compound->packed) { + warningf(WARN_PACKED, pos, "superfluous packed attribute on '%T'", type); } compound->size = offset; compound->alignment = alignment; - compound->layouted = true; } -/** - * Finish the construction of an union type by calculating - * its size and alignment. - */ void layout_union_type(compound_type_t *type) { assert(type->compound != NULL); @@ -1838,6 +1492,9 @@ void layout_union_type(compound_type_t *type) compound_t *compound = type->compound; if (! compound->complete) return; + if (compound->layouted) + return; + compound->layouted = true; il_size_t size = 0; il_alignment_t alignment = compound->alignment; @@ -1847,34 +1504,114 @@ void layout_union_type(compound_type_t *type) if (entry->kind != ENTITY_COMPOUND_MEMBER) continue; - type_t *m_type = entry->declaration.type; + type_t *m_type = skip_typeref(entry->declaration.type); if (! is_type_valid(skip_typeref(m_type))) continue; entry->compound_member.offset = 0; il_size_t m_size = get_type_size(m_type); - if (m_size > size) - size = m_size; - il_alignment_t m_alignment = get_type_alignment(m_type); - if (m_alignment > alignment) - alignment = m_alignment; + size = MAX(size, m_size); + il_alignment_t m_alignment = get_type_alignment_compound(m_type); + alignment = MAX(alignment, m_alignment); } - size = (size + alignment - 1) & -alignment; + size = round_up2(size, alignment); compound->size = size; compound->alignment = alignment; } +function_parameter_t *allocate_parameter(type_t *const type) +{ + function_parameter_t *const param = obstack_alloc(&type_obst, sizeof(*param)); + memset(param, 0, sizeof(*param)); + param->type = type; + return param; +} + +type_t *make_function_2_type(type_t *return_type, type_t *argument_type1, + type_t *argument_type2, decl_modifiers_t modifiers) +{ + function_parameter_t *const parameter2 = allocate_parameter(argument_type2); + function_parameter_t *const parameter1 = allocate_parameter(argument_type1); + parameter1->next = parameter2; + + type_t *type = allocate_type_zero(TYPE_FUNCTION); + type->function.return_type = return_type; + type->function.parameters = parameter1; + type->function.modifiers |= modifiers; + type->function.linkage = LINKAGE_C; + + return identify_new_type(type); +} + +type_t *make_function_1_type(type_t *return_type, type_t *argument_type, + decl_modifiers_t modifiers) +{ + function_parameter_t *const parameter = allocate_parameter(argument_type); + + type_t *type = allocate_type_zero(TYPE_FUNCTION); + type->function.return_type = return_type; + type->function.parameters = parameter; + type->function.modifiers |= modifiers; + type->function.linkage = LINKAGE_C; + + return identify_new_type(type); +} + +type_t *make_function_1_type_variadic(type_t *return_type, + type_t *argument_type, + decl_modifiers_t modifiers) +{ + function_parameter_t *const parameter = allocate_parameter(argument_type); + + type_t *type = allocate_type_zero(TYPE_FUNCTION); + type->function.return_type = return_type; + type->function.parameters = parameter; + type->function.variadic = true; + type->function.modifiers |= modifiers; + type->function.linkage = LINKAGE_C; + + return identify_new_type(type); +} + +type_t *make_function_0_type(type_t *return_type, decl_modifiers_t modifiers) +{ + type_t *type = allocate_type_zero(TYPE_FUNCTION); + type->function.return_type = return_type; + type->function.parameters = NULL; + type->function.modifiers |= modifiers; + type->function.linkage = LINKAGE_C; + + return identify_new_type(type); +} + +type_t *make_function_type(type_t *return_type, int n_types, + type_t *const *argument_types, + decl_modifiers_t modifiers) +{ + type_t *type = allocate_type_zero(TYPE_FUNCTION); + type->function.return_type = return_type; + type->function.modifiers |= modifiers; + type->function.linkage = LINKAGE_C; + + function_parameter_t **anchor = &type->function.parameters; + for (int i = 0; i < n_types; ++i) { + function_parameter_t *parameter = allocate_parameter(argument_types[i]); + *anchor = parameter; + anchor = ¶meter->next; + } + + return identify_new_type(type); +} + /** * Debug helper. Prints the given type to stdout. */ static __attribute__((unused)) void dbg_type(const type_t *type) { - FILE *old_out = out; - out = stderr; + print_to_file(stderr); print_type(type); - puts("\n"); + print_char('\n'); fflush(stderr); - out = old_out; }