fprintf(stderr, "parse error: %s\n", message);
}
-static void parse_warning(const char *message)
+static void parser_print_warning_prefix_pos(
+ const source_position_t source_position)
+{
+ parser_print_prefix_pos(source_position);
+ fputs("warning: ", stderr);
+}
+
+static void parse_warning_pos(const source_position_t source_position,
+ const char *const message)
{
- parser_print_prefix_pos(token.source_position);
+ parser_print_prefix_pos(source_position);
fprintf(stderr, "warning: %s\n", message);
}
+static void parse_warning(const char *message)
+{
+ parse_warning_pos(token.source_position, message);
+}
+
static void parse_error_expected(const char *message, ...)
{
va_list args;
fprintf(stderr, "\n");
}
+static void print_type_quoted(type_t *type)
+{
+ fputc('\'', stderr);
+ print_type(type);
+ fputc('\'', stderr);
+}
+
static void type_error(const char *msg, const source_position_t source_position,
type_t *type)
{
parser_print_error_prefix_pos(source_position);
fprintf(stderr, "%s, but found type ", msg);
- print_type(type);
+ print_type_quoted(type);
fputc('\n', stderr);
- error();
}
static void type_error_incompatible(const char *msg,
{
parser_print_error_prefix_pos(source_position);
fprintf(stderr, "%s, incompatible types: ", msg);
- print_type(type1);
+ print_type_quoted(type1);
fprintf(stderr, " - ");
- print_type(type2);
+ print_type_quoted(type2);
fprintf(stderr, ")\n");
- error();
}
static void eat_block(void)
context_t *parent_context)
{
symbol_t *symbol = declaration->symbol;
- namespace_t namespace = declaration->namespace;
- assert(declaration->source_position.input_name != NULL);
+ namespace_t namespace = (namespace_t)declaration->namespace;
/* a declaration should be only pushed once */
assert(declaration->parent_context == NULL);
parser_print_error_prefix_pos(declaration->source_position);
fprintf(stderr, "definition of symbol %s%s with type ",
get_namespace_prefix(namespace), symbol->string);
- error();
- print_type(declaration->type);
+ print_type_quoted(declaration->type);
fputc('\n', stderr);
parser_print_error_prefix_pos(
previous_declaration->source_position);
fprintf(stderr, "is incompatible with previous declaration "
"of type ");
- print_type(previous_declaration->type);
+ print_type_quoted(previous_declaration->type);
fputc('\n', stderr);
+ } else {
+ const storage_class_t old_storage = previous_declaration->storage_class;
+ const storage_class_t new_storage = declaration->storage_class;
+ if (current_function == NULL) {
+ if (old_storage != STORAGE_CLASS_STATIC &&
+ new_storage == STORAGE_CLASS_STATIC) {
+ parser_print_error_prefix_pos(declaration->source_position);
+ fprintf(stderr,
+ "static declaration of '%s' follows non-static declaration\n",
+ symbol->string);
+ parser_print_error_prefix_pos(previous_declaration->source_position);
+ fprintf(stderr, "previous declaration of '%s' was here\n",
+ symbol->string);
+ } else {
+ if (old_storage == STORAGE_CLASS_EXTERN) {
+ if (new_storage == STORAGE_CLASS_NONE) {
+ previous_declaration->storage_class = STORAGE_CLASS_NONE;
+ }
+ } else {
+ parser_print_warning_prefix_pos(declaration->source_position);
+ fprintf(stderr, "redundant declaration for '%s'\n",
+ symbol->string);
+ parser_print_warning_prefix_pos(previous_declaration->source_position);
+ fprintf(stderr, "previous declaration of '%s' was here\n",
+ symbol->string);
+ }
+ }
+ } else {
+ if (old_storage == STORAGE_CLASS_EXTERN &&
+ new_storage == STORAGE_CLASS_EXTERN) {
+ parser_print_warning_prefix_pos(declaration->source_position);
+ fprintf(stderr, "redundant extern declaration for '%s'\n",
+ symbol->string);
+ parser_print_warning_prefix_pos(previous_declaration->source_position);
+ fprintf(stderr, "previous declaration of '%s' was here\n",
+ symbol->string);
+ } else {
+ parser_print_error_prefix_pos(declaration->source_position);
+ if (old_storage == new_storage) {
+ fprintf(stderr, "redeclaration of '%s'\n", symbol->string);
+ } else {
+ fprintf(stderr, "redeclaration of '%s' with different linkage\n", symbol->string);
+ }
+ parser_print_error_prefix_pos(previous_declaration->source_position);
+ fprintf(stderr, "previous declaration of '%s' was here\n",
+ symbol->string);
+ }
+ }
}
return previous_declaration;
}
entry.symbol = symbol;
entry.old_declaration = symbol->declaration;
entry.namespace = namespace;
- ARR_APP1(*stack_ptr, entry);
+ ARR_APP1(stack_entry_t, *stack_ptr, entry);
/* replace/add declaration into declaration list of the symbol */
if(symbol->declaration == NULL) {
symbol->declaration = declaration;
} else {
- declaration_t *iter = symbol->declaration;
- for( ; iter != NULL; iter = iter->symbol_next) {
- declaration_t *symbol_next = iter->symbol_next;
- if(symbol_next == NULL) {
- iter->symbol_next = declaration;
- assert(declaration->symbol_next == NULL);
- break;
- }
- if(symbol_next->namespace == namespace) {
- iter->symbol_next = declaration;
- declaration->symbol_next = symbol_next->symbol_next;
+ declaration_t *iter_last = NULL;
+ declaration_t *iter = symbol->declaration;
+ for( ; iter != NULL; iter_last = iter, iter = iter->symbol_next) {
+ /* replace an entry? */
+ if(iter->namespace == namespace) {
+ if(iter_last == NULL) {
+ symbol->declaration = declaration;
+ } else {
+ iter_last->symbol_next = declaration;
+ }
+ declaration->symbol_next = iter->symbol_next;
break;
}
}
+ if(iter == NULL) {
+ assert(iter_last->symbol_next == NULL);
+ iter_last->symbol_next = declaration;
+ }
}
return declaration;
static declaration_t *environment_push(declaration_t *declaration)
{
+ assert(declaration->source_position.input_name != NULL);
return stack_push(&environment_stack, declaration, context);
}
declaration_t *old_declaration = entry->old_declaration;
symbol_t *symbol = entry->symbol;
- namespace_t namespace = entry->namespace;
+ namespace_t namespace = (namespace_t)entry->namespace;
/* replace/remove declaration */
declaration_t *declaration = symbol->declaration;
symbol->declaration = declaration->symbol_next;
} else {
symbol->declaration = old_declaration;
- assert(old_declaration->symbol_next ==
- declaration->symbol_next);
}
} else {
- for(; declaration != NULL; declaration = declaration->symbol_next) {
- declaration_t *symbol_next = declaration->symbol_next;
- if(symbol_next->namespace == namespace) {
- declaration->symbol_next = old_declaration;
- assert(old_declaration->symbol_next
- == symbol_next->symbol_next);
+ declaration_t *iter_last = declaration;
+ declaration_t *iter = declaration->symbol_next;
+ for( ; iter != NULL; iter_last = iter, iter = iter->symbol_next) {
+ /* replace an entry? */
+ if(iter->namespace == namespace) {
+ assert(iter_last != NULL);
+ iter_last->symbol_next = old_declaration;
+ old_declaration->symbol_next = iter->symbol_next;
break;
}
}
- assert(declaration != NULL);
+ assert(iter != NULL);
}
}
return (expression_t*) cast;
}
+static bool is_null_expression(const expression_t *const expr)
+{
+ if (expr->type != EXPR_CONST) return false;
+
+ type_t *const type = skip_typeref(expr->datatype);
+ if (!is_type_integer(type)) return false;
+
+ const const_t *const const_expr = (const const_t*)expr;
+ return const_expr->v.int_value == 0;
+}
+
static expression_t *create_implicit_cast(expression_t *expression,
type_t *dest_type)
{
- assert(expression->datatype != NULL);
type_t *source_type = expression->datatype;
+ if(source_type == NULL)
+ return expression;
+
source_type = skip_typeref(source_type);
dest_type = skip_typeref(dest_type);
- if(expression->datatype == dest_type)
+ if(source_type == dest_type)
return expression;
if(dest_type->type == TYPE_ATOMIC) {
return create_cast_expression(expression, dest_type);
}
if(dest_type->type == TYPE_POINTER) {
- if(source_type->type == TYPE_POINTER) {
- if(!pointers_compatible(source_type, dest_type)) {
- type_error_incompatible("can't implicitely cast types",
- expression->source_position,
- source_type, dest_type);
- } else {
- return create_cast_expression(expression, dest_type);
+ pointer_type_t *pointer_type
+ = (pointer_type_t*) dest_type;
+ switch (source_type->type) {
+ case TYPE_ATOMIC:
+ if (is_null_expression(expression)) {
+ return create_cast_expression(expression, dest_type);
+ }
+ break;
+
+ case TYPE_POINTER:
+ if (pointers_compatible(source_type, dest_type)) {
+ return create_cast_expression(expression, dest_type);
+ }
+ break;
+
+ case TYPE_ARRAY: {
+ array_type_t *const array_type = (array_type_t*) source_type;
+ if (types_compatible(array_type->element_type,
+ pointer_type->points_to)) {
+ return create_cast_expression(expression, dest_type);
+ }
+ break;
}
+
+ default:
+ panic("casting of non-atomic types not implemented yet");
}
+
+ type_error_incompatible("can't implicitely cast types",
+ expression->source_position,
+ source_type, dest_type);
+ return expression;
}
panic("casting of non-atomic types not implemented yet");
const char *context)
{
type_t *orig_type_right = (*right)->datatype;
- type_t *type_left = skip_typeref(orig_type_left);
- type_t *type_right = skip_typeref(orig_type_right);
+
+ if(orig_type_right == NULL)
+ return;
+
+ type_t *const type_left = skip_typeref(orig_type_left);
+ type_t *const type_right = skip_typeref(orig_type_right);
if(type_left == type_right) {
/* fine */
- } else if(is_type_arithmetic(type_left) && is_type_arithmetic(type_right)) {
+ } else if ((is_type_arithmetic(type_left) && is_type_arithmetic(type_right)) ||
+ (type_left->type == TYPE_POINTER && is_null_expression(*right)) ||
+ (type_left->type == TYPE_POINTER && type_right->type == TYPE_POINTER)) {
*right = create_implicit_cast(*right, type_left);
- } else if(type_left->type == TYPE_POINTER
- && type_right->type == TYPE_POINTER) {
- /* TODO */
} else {
/* TODO: improve error message */
parser_print_error_prefix();
fprintf(stderr, "incompatible types in %s\n", context);
parser_print_error_prefix();
- print_type(type_left);
+ print_type_quoted(type_left);
fputs(" <- ", stderr);
- print_type(type_right);
+ print_type_quoted(type_right);
fputs("\n", stderr);
}
return parse_sub_expression(2);
}
-static void parse_compound_type_entries(void);
-static declaration_t *parse_declarator(storage_class_t storage_class,
- type_t *type, int may_be_abstract);
-static declaration_t *record_declaration(declaration_t *declaration);
-
typedef struct declaration_specifiers_t declaration_specifiers_t;
struct declaration_specifiers_t {
storage_class_t storage_class;
+ bool is_inline;
type_t *type;
};
+static void parse_compound_type_entries(void);
+static declaration_t *parse_declarator(
+ const declaration_specifiers_t *specifiers, type_t *type,
+ bool may_be_abstract);
+static declaration_t *record_declaration(declaration_t *declaration);
+
static const char *parse_string_literals(void)
{
assert(token.type == T_STRING_LITERAL);
designator = allocate_ast_zero(sizeof(designator[0]));
next_token();
if(token.type != T_IDENTIFIER) {
- parse_error_expected("problem while parsing designator",
+ parse_error_expected("while parsing designator",
T_IDENTIFIER, 0);
return NULL;
}
break;
if(token.type != ',') {
- parse_error_expected("problem while parsing initializer list",
- ',', '}', 0);
+ parse_error_expected("while parsing initializer list", ',', '}', 0);
eat_block();
return result;
}
}
} else if(token.type != '{') {
if(is_struct) {
- parse_error_expected("problem while parsing struct type specifier",
+ parse_error_expected("while parsing struct type specifier",
T_IDENTIFIER, '{', 0);
} else {
- parse_error_expected("problem while parsing union type specifier",
+ parse_error_expected("while parsing union type specifier",
T_IDENTIFIER, '{', 0);
}
}
declaration->source_position = token.source_position;
declaration->symbol = symbol;
+ record_declaration(declaration);
}
if(token.type == '{') {
is_struct ? "struct" : "union", symbol->string);
declaration->context.declarations = NULL;
}
- record_declaration(declaration);
declaration->init.is_defined = true;
int top = environment_top();
declaration_t *entry = allocate_ast_zero(sizeof(entry[0]));
if(token.type != T_IDENTIFIER) {
- parse_error_expected("problem while parsing enum entry",
- T_IDENTIFIER, 0);
+ parse_error_expected("while parsing enum entry", T_IDENTIFIER, 0);
eat_block();
return;
}
declaration = get_declaration(symbol, NAMESPACE_ENUM);
} else if(token.type != '{') {
- parse_error_expected("problem while parsing enum type specifier",
+ parse_error_expected("while parsing enum type specifier",
T_IDENTIFIER, '{', 0);
return NULL;
} else {
MATCH_TYPE_QUALIFIER(T_const, TYPE_QUALIFIER_CONST);
MATCH_TYPE_QUALIFIER(T_restrict, TYPE_QUALIFIER_RESTRICT);
MATCH_TYPE_QUALIFIER(T_volatile, TYPE_QUALIFIER_VOLATILE);
- MATCH_TYPE_QUALIFIER(T_inline, TYPE_QUALIFIER_INLINE);
case T___extension__:
/* TODO */
#ifdef PROVIDE_IMAGINARY
MATCH_SPECIFIER(T__Imaginary, SPECIFIER_IMAGINARY, "_Imaginary")
#endif
+ case T_inline:
+ next_token();
+ specifiers->is_inline = true;
+ break;
+
case T_long:
next_token();
if(type_specifiers & SPECIFIER_LONG_LONG) {
}
}
- type->qualifiers = type_qualifiers;
+ type->qualifiers = (type_qualifier_t)type_qualifiers;
type_t *result = typehash_insert(type);
if(newtype && result != (type_t*) type) {
specifiers->type = result;
}
-static type_qualifier_t parse_type_qualifiers(void)
+static unsigned parse_type_qualifiers(void)
{
- type_qualifier_t type_qualifiers = 0;
+ unsigned type_qualifiers = TYPE_QUALIFIER_NONE;
while(true) {
switch(token.type) {
MATCH_TYPE_QUALIFIER(T_const, TYPE_QUALIFIER_CONST);
MATCH_TYPE_QUALIFIER(T_restrict, TYPE_QUALIFIER_RESTRICT);
MATCH_TYPE_QUALIFIER(T_volatile, TYPE_QUALIFIER_VOLATILE);
- MATCH_TYPE_QUALIFIER(T_inline, TYPE_QUALIFIER_INLINE);
default:
return type_qualifiers;
{
while(true) {
if(token.type != T_IDENTIFIER) {
- parse_error_expected("problem while parsing parameter identifier "
- "list", T_IDENTIFIER, 0);
+ parse_error_expected("while parsing parameter identifier list",
+ T_IDENTIFIER, 0);
return;
}
next_token();
parse_declaration_specifiers(&specifiers);
- declaration_t *declaration = parse_declarator(specifiers.storage_class,
- specifiers.type, 1);
+ declaration_t *declaration
+ = parse_declarator(&specifiers, specifiers.type, true);
/* TODO check declaration constraints for parameters */
if(declaration->storage_class == STORAGE_CLASS_TYPEDEF) {
}
typedef enum {
+ CONSTRUCT_INVALID,
CONSTRUCT_POINTER,
CONSTRUCT_FUNCTION,
CONSTRUCT_ARRAY
parsed_pointer_t *pointer = obstack_alloc(&temp_obst, sizeof(pointer[0]));
memset(pointer, 0, sizeof(pointer[0]));
- pointer->type_qualifiers = parse_type_qualifiers();
+ pointer->construct_type.type = CONSTRUCT_POINTER;
+ pointer->type_qualifiers = parse_type_qualifiers();
return (construct_type_t*) pointer;
}
parsed_array_t *array = obstack_alloc(&temp_obst, sizeof(array[0]));
memset(array, 0, sizeof(array[0]));
+ array->construct_type.type = CONSTRUCT_ARRAY;
if(token.type == T_static) {
array->is_static = true;
default:
if(may_be_abstract)
break;
- parse_error_expected("problem while parsing declarator", T_IDENTIFIER,
- '(', 0);
+ parse_error_expected("while parsing declarator", T_IDENTIFIER, '(', 0);
}
while(true) {
array_type_t *array_type;
switch(iter->type) {
+ case CONSTRUCT_INVALID:
+ panic("invalid type construction found");
case CONSTRUCT_FUNCTION:
construct_function_type = (construct_function_type_t*) iter;
function_type = construct_function_type->function_type;
return type;
}
-static declaration_t *parse_declarator(storage_class_t storage_class,
- type_t *type, int may_be_abstract)
+static declaration_t *parse_declarator(
+ const declaration_specifiers_t *specifiers,
+ type_t *type, bool may_be_abstract)
{
declaration_t *declaration = allocate_ast_zero(sizeof(declaration[0]));
- declaration->storage_class = storage_class;
+ declaration->storage_class = specifiers->storage_class;
+ declaration->is_inline = specifiers->is_inline;
construct_type_t *construct_type
= parse_inner_declarator(declaration, may_be_abstract);
parser_print_error_prefix_pos(previous->source_position);
fprintf(stderr, "this is the location of the previous "
"definition.\n");
- error();
}
static void parse_init_declarators(const declaration_specifiers_t *specifiers)
{
while(true) {
declaration_t *ndeclaration
- = parse_declarator(specifiers->storage_class, specifiers->type, 0);
+ = parse_declarator(specifiers, specifiers->type, false);
declaration_t *declaration = record_declaration(ndeclaration);
+
+ type_t *type = declaration->type;
+ if(type->type != TYPE_FUNCTION && declaration->is_inline) {
+ parser_print_warning_prefix_pos(declaration->source_position);
+ fprintf(stderr, "variable ‘%s’ declared ‘inline’\n",
+ declaration->symbol->string);
+ }
+
if(token.type == '=') {
next_token();
/* TODO (bitfields) */
} else {
declaration_t *declaration
- = parse_declarator(specifiers->storage_class,
- specifiers->type, 1);
+ = parse_declarator(specifiers, specifiers->type, true);
/* TODO: check constraints for struct declarations */
/* TODO: check for doubled fields */
parse_declaration_specifiers(&specifiers);
if(token.type == ';') {
+ if (specifiers.storage_class != STORAGE_CLASS_NONE) {
+ parse_warning_pos(source_position,
+ "useless keyword in empty declaration");
+ }
+ switch (specifiers.type->type) {
+ case TYPE_COMPOUND_STRUCT:
+ case TYPE_COMPOUND_UNION: {
+ const compound_type_t *const comp_type =
+ (const compound_type_t*)specifiers.type;
+ if (comp_type->declaration->symbol == NULL) {
+ parse_warning_pos(source_position,
+ "unnamed struct/union that defines no instances");
+ }
+ break;
+ }
+
+ case TYPE_ENUM: break;
+
+ default:
+ parse_warning_pos(source_position, "empty declaration");
+ break;
+ }
+
next_token();
declaration_t *declaration = allocate_ast_zero(sizeof(declaration[0]));
expression_parser_function_t expression_parsers[T_LAST_TOKEN];
+static expression_t *make_invalid_expression(void)
+{
+ expression_t *expression = allocate_ast_zero(sizeof(expression[0]));
+ expression->type = EXPR_INVALID;
+ expression->source_position = token.source_position;
+ return expression;
+}
+
static expression_t *expected_expression_error(void)
{
parser_print_error_prefix();
print_token(stderr, & token);
fprintf(stderr, "\n");
- expression_t *expression = allocate_ast_zero(sizeof(expression[0]));
- expression->type = EXPR_INVALID;
next_token();
- return expression;
+ return make_invalid_expression();
}
static expression_t *parse_string_const(void)
statement_expression_t *expression
= allocate_ast_zero(sizeof(expression[0]));
expression->expression.type = EXPR_STATEMENT;
- expression->statement = parse_compound_statement();
+
+ statement_t *statement = parse_compound_statement();
+ expression->statement = statement;
+ if(statement == NULL) {
+ expect(')');
+ return NULL;
+ }
+
+ assert(statement->type == STATEMENT_COMPOUND);
+ compound_statement_t *compound_statement
+ = (compound_statement_t*) statement;
/* find last statement and use it's type */
const statement_t *last_statement = NULL;
- const statement_t *statement = expression->statement;
- for( ; statement != NULL; statement = statement->next) {
- last_statement = statement;
+ const statement_t *iter = compound_statement->statements;
+ for( ; iter != NULL; iter = iter->next) {
+ last_statement = iter;
}
if(last_statement->type == STATEMENT_EXPRESSION) {
designator_t *result = allocate_ast_zero(sizeof(result[0]));
if(token.type != T_IDENTIFIER) {
- parse_error_expected("problem while parsing member designator",
+ parse_error_expected("while parsing member designator",
T_IDENTIFIER, 0);
eat_brace();
return NULL;
if(token.type == '.') {
next_token();
if(token.type != T_IDENTIFIER) {
- parse_error_expected("problem while parsing member designator",
- T_IDENTIFIER, 0);
+ parse_error_expected("while parsing member designator",
+ T_IDENTIFIER, 0);
eat_brace();
return NULL;
}
fprintf(stderr, "\n");
eat_statement();
- expression_t *expression = allocate_ast_zero(sizeof(expression[0]));
- expression->type = EXPR_INVALID;
- expression->datatype = type_void;
-
- return expression;
+ return make_invalid_expression();
}
static expression_t *parse_array_expression(unsigned precedence,
array_access->array_ref = array_ref;
array_access->index = parse_expression();
- type_t *array_type = array_ref->datatype;
- if(array_type != NULL) {
- if(array_type->type == TYPE_POINTER) {
- pointer_type_t *pointer = (pointer_type_t*) array_type;
+ type_t *type = array_ref->datatype;
+ if(type != NULL) {
+ if(type->type == TYPE_POINTER) {
+ pointer_type_t *pointer = (pointer_type_t*) type;
array_access->expression.datatype = pointer->points_to;
+ } else if(type->type == TYPE_ARRAY) {
+ array_type_t *array_type = (array_type_t*) type;
+ array_access->expression.datatype = array_type->element_type;
} else {
parser_print_error_prefix();
fprintf(stderr, "array access on object with non-pointer type ");
- print_type(array_type);
+ print_type_quoted(type);
fprintf(stderr, "\n");
}
}
expression_t *compound)
{
(void) precedence;
-
assert(token.type == '.' || token.type == T_MINUSGREATER);
+
+ bool is_pointer = (token.type == T_MINUSGREATER);
next_token();
select_expression_t *select = allocate_ast_zero(sizeof(select[0]));
select->expression.type = EXPR_SELECT;
select->compound = compound;
- /* TODO: datatype */
-
if(token.type != T_IDENTIFIER) {
- parse_error_expected("Problem while parsing select", T_IDENTIFIER, 0);
+ parse_error_expected("while parsing select", T_IDENTIFIER, 0);
return (expression_t*) select;
}
- select->symbol = token.v.symbol;
+ symbol_t *symbol = token.v.symbol;
+ select->symbol = symbol;
next_token();
+ type_t *type = compound->datatype;
+ if(type == NULL)
+ return make_invalid_expression();
+
+ type_t *type_left = type;
+ if(is_pointer) {
+ if(type->type != TYPE_POINTER) {
+ parser_print_error_prefix();
+ fprintf(stderr, "left hand side of '->' is not a pointer, but ");
+ print_type_quoted(type);
+ fputc('\n', stderr);
+ return make_invalid_expression();
+ }
+ pointer_type_t *pointer_type = (pointer_type_t*) type;
+ type_left = pointer_type->points_to;
+ }
+ type_left = skip_typeref(type_left);
+
+ if(type_left->type != TYPE_COMPOUND_STRUCT
+ && type_left->type != TYPE_COMPOUND_UNION) {
+ parser_print_error_prefix();
+ fprintf(stderr, "request for member '%s' in something not a struct or "
+ "union, but ", symbol->string);
+ print_type_quoted(type_left);
+ fputc('\n', stderr);
+ return make_invalid_expression();
+ }
+
+ compound_type_t *compound_type = (compound_type_t*) type_left;
+ declaration_t *declaration = compound_type->declaration;
+
+ if(!declaration->init.is_defined) {
+ parser_print_error_prefix();
+ fprintf(stderr, "request for member '%s' of incomplete type ",
+ symbol->string);
+ print_type_quoted(type_left);
+ fputc('\n', stderr);
+ return make_invalid_expression();
+ }
+
+ declaration_t *iter = declaration->context.declarations;
+ for( ; iter != NULL; iter = iter->next) {
+ if(iter->symbol == symbol) {
+ break;
+ }
+ }
+ if(iter == NULL) {
+ parser_print_error_prefix();
+ print_type_quoted(type_left);
+ fprintf(stderr, " has no memeber named '%s'\n", symbol->string);
+ return make_invalid_expression();
+ }
+
+ select->compound_entry = iter;
+ select->expression.datatype = iter->type;
return (expression_t*) select;
}
fputs("called object '", stderr);
print_expression(expression);
fputs("' (type ", stderr);
- print_type(type);
+ print_type_quoted(type);
fputs("is not a function\n", stderr);
function_type = NULL;
parameter = parameter->next, argument = argument->next) {
type_t *expected_type = parameter->type;
/* TODO report context in error messages */
- argument->expression = create_implicit_cast(argument->expression,
- expected_type);
+ argument->expression = create_implicit_cast(argument->expression,
+ expected_type);
}
/* too few parameters */
if(parameter != NULL) {
for( ; argument != NULL; argument = argument->next) {
type_t *type = argument->expression->datatype;
+ if(type == NULL)
+ continue;
+
if(is_type_integer(type)) {
type = promote_integer(type);
} else if(type == type_float) {
/* 6.5.15.2 */
type_t *condition_type = conditional->condition->datatype;
if(condition_type != NULL) {
- if(!is_type_scalar(condition_type)) {
+ if(!is_type_scalar(skip_typeref(condition_type))) {
type_error("expected a scalar type", expression->source_position,
condition_type);
}
if(false_type == NULL)
return (expression_t*) conditional;
+ type_t *const skipped_true_type = skip_typeref(true_type);
+ type_t *const skipped_false_type = skip_typeref(false_type);
+
/* 6.5.15.3 */
- if(true_type == false_type) {
- conditional->expression.datatype = true_type;
- } else if(is_type_arithmetic(true_type) && is_type_arithmetic(false_type)) {
- type_t *result = get_type_after_conversion(true_type, false_type);
+ if (skipped_true_type == skipped_false_type) {
+ conditional->expression.datatype = skipped_true_type;
+ } else if (is_type_arithmetic(skipped_true_type) &&
+ is_type_arithmetic(skipped_false_type)) {
+ type_t *const result = get_type_after_conversion(skipped_true_type,
+ skipped_false_type);
/* TODO: create implicit convs if necessary */
conditional->expression.datatype = result;
- } else if(true_type->type == TYPE_POINTER &&
- false_type->type == TYPE_POINTER &&
+ } else if (skipped_true_type->type == TYPE_POINTER &&
+ skipped_false_type->type == TYPE_POINTER &&
true /* TODO compatible points_to types */) {
/* TODO */
- } else if(/* (is_null_ptr_const(true_type) && false_type->type == TYPE_POINTER)
- || (is_null_ptr_const(false_type) &&
- true_type->type == TYPE_POINTER) TODO*/ false) {
+ } else if(/* (is_null_ptr_const(skipped_true_type) &&
+ skipped_false_type->type == TYPE_POINTER)
+ || (is_null_ptr_const(skipped_false_type) &&
+ skipped_true_type->type == TYPE_POINTER) TODO*/ false) {
/* TODO */
} else if(/* 1 is pointer to object type, other is void* */ false) {
/* TODO */
} else {
- type_error_incompatible("problem while parsing conditional",
+ type_error_incompatible("while parsing conditional",
expression->source_position, true_type,
- false_type);
+ skipped_false_type);
}
return (expression_t*) conditional;
return parse_sub_expression(precedence);
}
-static type_t *get_unexpr_arithmetic_type(const expression_t *expression)
+static void semantic_incdec(unary_expression_t *expression)
{
- /* TODO */
- return expression->datatype;
+ type_t *orig_type = expression->value->datatype;
+ if(orig_type == NULL)
+ return;
+
+ type_t *type = skip_typeref(orig_type);
+ if(!is_type_arithmetic(type) && type->type != TYPE_POINTER) {
+ /* TODO: improve error message */
+ parser_print_error_prefix();
+ fprintf(stderr, "operation needs an arithmetic or pointer type\n");
+ return;
+ }
+
+ expression->expression.datatype = orig_type;
}
-static type_t *get_unexpr_dereference_type(const expression_t *expression)
+static void semantic_unexpr_arithmetic(unary_expression_t *expression)
{
- type_t *expression_type = expression->datatype;
+ type_t *orig_type = expression->value->datatype;
+ if(orig_type == NULL)
+ return;
- if(expression_type->type == TYPE_POINTER) {
- pointer_type_t *pointer_type = (pointer_type_t*) expression_type;
- return pointer_type->points_to;
+ type_t *type = skip_typeref(orig_type);
+ if(!is_type_arithmetic(type)) {
+ /* TODO: improve error message */
+ parser_print_error_prefix();
+ fprintf(stderr, "operation needs an arithmetic type\n");
+ return;
}
- panic("deref TODO...");
- return NULL;
+
+ expression->expression.datatype = orig_type;
}
-static type_t *get_unexpr_take_addr_type(const expression_t *expression)
+static void semantic_dereference(unary_expression_t *expression)
{
- type_t *type = expression->datatype;
- return make_pointer_type(type, 0);
+ type_t *orig_type = expression->value->datatype;
+ if(orig_type == NULL)
+ return;
+
+ type_t *type = skip_typeref(orig_type);
+ if(type->type != TYPE_POINTER) {
+ /* TODO: improve error message */
+ parser_print_error_prefix();
+ fprintf(stderr, "operation needs a pointer type\n");
+ return;
+ }
+
+ pointer_type_t *pointer_type = (pointer_type_t*) type;
+ expression->expression.datatype = pointer_type->points_to;
+}
+
+static void semantic_take_addr(unary_expression_t *expression)
+{
+ type_t *orig_type = expression->value->datatype;
+ if(orig_type == NULL)
+ return;
+
+ expression_t *value = expression->value;
+ if(value->type == EXPR_REFERENCE) {
+ reference_expression_t *reference = (reference_expression_t*) value;
+ declaration_t *declaration = reference->declaration;
+ if(declaration != NULL) {
+ declaration->address_taken = 1;
+ }
+ }
+
+ expression->expression.datatype = make_pointer_type(orig_type, 0);
}
-#define CREATE_UNARY_EXPRESSION_PARSER(token_type, unexpression_type, tfunc) \
+#define CREATE_UNARY_EXPRESSION_PARSER(token_type, unexpression_type, sfunc) \
static expression_t *parse_##unexpression_type(unsigned precedence) \
{ \
eat(token_type); \
unary_expression->expression.type = EXPR_UNARY; \
unary_expression->type = unexpression_type; \
unary_expression->value = parse_sub_expression(precedence); \
- unary_expression->expression.datatype = tfunc(unary_expression->value); \
+ \
+ sfunc(unary_expression); \
\
return (expression_t*) unary_expression; \
}
-CREATE_UNARY_EXPRESSION_PARSER('-', UNEXPR_NEGATE, get_unexpr_arithmetic_type)
-CREATE_UNARY_EXPRESSION_PARSER('+', UNEXPR_PLUS, get_unexpr_arithmetic_type)
-CREATE_UNARY_EXPRESSION_PARSER('!', UNEXPR_NOT, get_unexpr_arithmetic_type)
-CREATE_UNARY_EXPRESSION_PARSER('*', UNEXPR_DEREFERENCE,
- get_unexpr_dereference_type)
-CREATE_UNARY_EXPRESSION_PARSER('&', UNEXPR_TAKE_ADDRESS,
- get_unexpr_take_addr_type)
+CREATE_UNARY_EXPRESSION_PARSER('-', UNEXPR_NEGATE, semantic_unexpr_arithmetic)
+CREATE_UNARY_EXPRESSION_PARSER('+', UNEXPR_PLUS, semantic_unexpr_arithmetic)
+CREATE_UNARY_EXPRESSION_PARSER('!', UNEXPR_NOT, semantic_unexpr_arithmetic)
+CREATE_UNARY_EXPRESSION_PARSER('*', UNEXPR_DEREFERENCE, semantic_dereference)
+CREATE_UNARY_EXPRESSION_PARSER('&', UNEXPR_TAKE_ADDRESS, semantic_take_addr)
CREATE_UNARY_EXPRESSION_PARSER('~', UNEXPR_BITWISE_NEGATE,
- get_unexpr_arithmetic_type)
+ semantic_unexpr_arithmetic)
CREATE_UNARY_EXPRESSION_PARSER(T_PLUSPLUS, UNEXPR_PREFIX_INCREMENT,
- get_unexpr_arithmetic_type)
+ semantic_incdec)
CREATE_UNARY_EXPRESSION_PARSER(T_MINUSMINUS, UNEXPR_PREFIX_DECREMENT,
- get_unexpr_arithmetic_type)
+ semantic_incdec)
#define CREATE_UNARY_POSTFIX_EXPRESSION_PARSER(token_type, unexpression_type, \
- tfunc) \
+ sfunc) \
static expression_t *parse_##unexpression_type(unsigned precedence, \
expression_t *left) \
{ \
unary_expression->expression.type = EXPR_UNARY; \
unary_expression->type = unexpression_type; \
unary_expression->value = left; \
- unary_expression->expression.datatype = tfunc(left); \
+ \
+ sfunc(unary_expression); \
\
return (expression_t*) unary_expression; \
}
CREATE_UNARY_POSTFIX_EXPRESSION_PARSER(T_PLUSPLUS, UNEXPR_POSTFIX_INCREMENT,
- get_unexpr_arithmetic_type)
+ semantic_incdec)
CREATE_UNARY_POSTFIX_EXPRESSION_PARSER(T_MINUSMINUS, UNEXPR_POSTFIX_DECREMENT,
- get_unexpr_arithmetic_type)
+ semantic_incdec)
static type_t *semantic_arithmetic(type_t *type_left, type_t *type_right)
{
{
expression_t *left = expression->left;
expression_t *right = expression->right;
- type_t *type_left = skip_typeref(left->datatype);
- type_t *type_right = skip_typeref(right->datatype);
+ type_t *orig_type_left = left->datatype;
+ type_t *orig_type_right = right->datatype;
+
+ if(orig_type_left == NULL || orig_type_right == NULL)
+ return;
+
+ type_t *type_left = skip_typeref(orig_type_left);
+ type_t *type_right = skip_typeref(orig_type_right);
if(!is_type_arithmetic(type_left) || !is_type_arithmetic(type_right)) {
/* TODO: improve error message */
expression->expression.datatype = arithmetic_type;
}
+static void semantic_shift_op(binary_expression_t *expression)
+{
+ expression_t *left = expression->left;
+ expression_t *right = expression->right;
+ type_t *orig_type_left = left->datatype;
+ type_t *orig_type_right = right->datatype;
+
+ if(orig_type_left == NULL || orig_type_right == NULL)
+ return;
+
+ type_t *type_left = skip_typeref(orig_type_left);
+ type_t *type_right = skip_typeref(orig_type_right);
+
+ if(!is_type_integer(type_left) || !is_type_integer(type_right)) {
+ /* TODO: improve error message */
+ parser_print_error_prefix();
+ fprintf(stderr, "operation needs integer types\n");
+ return;
+ }
+
+ type_left = promote_integer(type_left);
+ type_right = promote_integer(type_right);
+
+ expression->left = create_implicit_cast(left, type_left);
+ expression->right = create_implicit_cast(right, type_right);
+ expression->expression.datatype = type_left;
+}
+
static void semantic_add(binary_expression_t *expression)
{
expression_t *left = expression->left;
expression_t *right = expression->right;
type_t *orig_type_left = left->datatype;
type_t *orig_type_right = right->datatype;
- type_t *type_left = skip_typeref(orig_type_left);
- type_t *type_right = skip_typeref(orig_type_right);
+
+ if(orig_type_left == NULL || orig_type_right == NULL)
+ return;
+
+ type_t *type_left = skip_typeref(orig_type_left);
+ type_t *type_right = skip_typeref(orig_type_right);
/* § 5.6.5 */
if(is_type_arithmetic(type_left) && is_type_arithmetic(type_right)) {
} else {
parser_print_error_prefix();
fprintf(stderr, "invalid operands to binary + (");
- print_type(orig_type_left);
+ print_type_quoted(orig_type_left);
fprintf(stderr, ", ");
- print_type(orig_type_right);
+ print_type_quoted(orig_type_right);
fprintf(stderr, ")\n");
}
}
expression_t *right = expression->right;
type_t *orig_type_left = left->datatype;
type_t *orig_type_right = right->datatype;
+
+ if(orig_type_left == NULL || orig_type_right == NULL)
+ return;
+
type_t *type_left = skip_typeref(orig_type_left);
type_t *type_right = skip_typeref(orig_type_right);
if(!pointers_compatible(type_left, type_right)) {
parser_print_error_prefix();
fprintf(stderr, "pointers to incompatible objects to binary - (");
- print_type(orig_type_left);
+ print_type_quoted(orig_type_left);
fprintf(stderr, ", ");
- print_type(orig_type_right);
+ print_type_quoted(orig_type_right);
fprintf(stderr, ")\n");
} else {
expression->expression.datatype = type_ptrdiff_t;
} else {
parser_print_error_prefix();
fprintf(stderr, "invalid operands to binary - (");
- print_type(orig_type_left);
+ print_type_quoted(orig_type_left);
fprintf(stderr, ", ");
- print_type(orig_type_right);
+ print_type_quoted(orig_type_right);
fprintf(stderr, ")\n");
}
}
static void semantic_comparison(binary_expression_t *expression)
{
- expression_t *left = expression->left;
- expression_t *right = expression->right;
- type_t *type_left = left->datatype;
- type_t *type_right = right->datatype;
+ expression_t *left = expression->left;
+ expression_t *right = expression->right;
+ type_t *orig_type_left = left->datatype;
+ type_t *orig_type_right = right->datatype;
+
+ if(orig_type_left == NULL || orig_type_right == NULL)
+ return;
+
+ type_t *type_left = skip_typeref(orig_type_left);
+ type_t *type_right = skip_typeref(orig_type_right);
/* TODO non-arithmetic types */
if(is_type_arithmetic(type_left) && is_type_arithmetic(type_right)) {
expression->left = create_implicit_cast(left, arithmetic_type);
expression->right = create_implicit_cast(right, arithmetic_type);
expression->expression.datatype = arithmetic_type;
+ } else if (type_left->type == TYPE_POINTER &&
+ type_right->type == TYPE_POINTER) {
+ /* TODO check compatibility */
+ } else if (type_left->type == TYPE_POINTER) {
+ expression->right = create_implicit_cast(right, type_left);
+ } else if (type_right->type == TYPE_POINTER) {
+ expression->left = create_implicit_cast(left, type_right);
+ } else {
+ type_error_incompatible("invalid operands in comparison",
+ expression->expression.source_position,
+ type_left, type_right);
}
expression->expression.datatype = type_int;
}
static void semantic_arithmetic_assign(binary_expression_t *expression)
{
- expression_t *left = expression->left;
- expression_t *right = expression->right;
- type_t *type_left = left->datatype;
- type_t *type_right = right->datatype;
+ expression_t *left = expression->left;
+ expression_t *right = expression->right;
+ type_t *orig_type_left = left->datatype;
+ type_t *orig_type_right = right->datatype;
+
+ if(orig_type_left == NULL || orig_type_right == NULL)
+ return;
+
+ type_t *type_left = skip_typeref(orig_type_left);
+ type_t *type_right = skip_typeref(orig_type_right);
if(!is_type_arithmetic(type_left) || !is_type_arithmetic(type_right)) {
/* TODO: improve error message */
static void semantic_logical_op(binary_expression_t *expression)
{
- /* TODO */
+ expression_t *left = expression->left;
+ expression_t *right = expression->right;
+ type_t *orig_type_left = left->datatype;
+ type_t *orig_type_right = right->datatype;
+
+ if(orig_type_left == NULL || orig_type_right == NULL)
+ return;
+
+ type_t *type_left = skip_typeref(orig_type_left);
+ type_t *type_right = skip_typeref(orig_type_right);
+
+ if (!is_type_scalar(type_left) || !is_type_scalar(type_right)) {
+ /* TODO: improve error message */
+ parser_print_error_prefix();
+ fprintf(stderr, "operation needs scalar types\n");
+ return;
+ }
+
expression->expression.datatype = type_int;
}
expression_t *left = expression->left;
type_t *type_left = left->datatype;
- semantic_assign(type_left, &expression->right, "assignment");
+ if(type_left != NULL) {
+ semantic_assign(type_left, &expression->right, "assignment");
+ }
expression->expression.datatype = type_left;
}
CREATE_BINEXPR_PARSER(T_PIPEPIPE, BINEXPR_LOGICAL_OR, semantic_logical_op, 1)
/* TODO shift has a bit special semantic */
CREATE_BINEXPR_PARSER(T_LESSLESS, BINEXPR_SHIFTLEFT,
- semantic_binexpr_arithmetic, 1)
+ semantic_shift_op, 1)
CREATE_BINEXPR_PARSER(T_GREATERGREATER, BINEXPR_SHIFTRIGHT,
- semantic_binexpr_arithmetic, 1)
+ semantic_shift_op, 1)
CREATE_BINEXPR_PARSER(T_PLUSEQUAL, BINEXPR_ADD_ASSIGN,
semantic_arithmetic_assign, 0)
CREATE_BINEXPR_PARSER(T_MINUSEQUAL, BINEXPR_SUB_ASSIGN,
left = parser->infix_parser(parser->infix_precedence, left);
assert(left != NULL);
- assert(left->type != EXPR_INVALID);
+ assert(left->type != EXPR_UNKNOWN);
left->source_position = source_position;
}
{
assert(token.type == T_IDENTIFIER);
symbol_t *symbol = token.v.symbol;
+ next_token();
declaration_t *label = get_label(symbol);
label_statement->statement.type = STATEMENT_LABEL;
label_statement->statement.source_position = token.source_position;
+ label_statement->label = label;
expect(':');
return NULL;
}
symbol_t *symbol = token.v.symbol;
+ next_token();
declaration_t *label = get_label(symbol);
expect(';');
- return NULL;
+ return (statement_t*) statement;
}
static statement_t *parse_continue(void)
parse_warning("'return' with a value, in function returning void");
return_value = NULL;
} else {
- semantic_assign(return_type, &return_value, "'return'");
+ if(return_type != NULL) {
+ semantic_assign(return_type, &return_value, "'return'");
+ }
}
} else {
return_value = NULL;
projects / cparser / blobdiff