diff options
author | Victor Westerhuis <victor@westerhu.is> | 2021-11-29 22:17:34 +0100 |
---|---|---|
committer | Victor Westerhuis <victor@westerhu.is> | 2021-11-29 22:47:34 +0100 |
commit | 938c4ff209056fb332e57ba03ca56aed820824e4 (patch) | |
tree | f082b816a4264a8efbe47b99aa69d4b92b27b529 | |
parent | fc61ecb3a22b89864916ec538eaf04840e7dd6b5 (diff) | |
download | colm-938c4ff209056fb332e57ba03ca56aed820824e4.tar.gz |
Correct prototype for host call functions in tests
All parameters for host calls declared in Colm source files are
converted to value_t by Compiler::writeHostCall:
https://github.com/adrian-thurston/colm/blob/fc61ecb3a22b89864916ec538eaf04840e7dd6b5/src/compiler.cc#L1109-L1117
This led to warnings when compiling with LTO. Correcting the
prototypes and inserting the appropriate casts makes everything
compile without warnings. The test suite still passes the same on
my x86_64 machine.
-rw-r--r-- | test/rlparse.d/actparams.cc | 16 | ||||
-rw-r--r-- | test/trans.d/actparams.cc | 16 |
2 files changed, 16 insertions, 16 deletions
diff --git a/test/rlparse.d/actparams.cc b/test/rlparse.d/actparams.cc index d9e9d6ff..dab00b61 100644 --- a/test/rlparse.d/actparams.cc +++ b/test/rlparse.d/actparams.cc @@ -14,18 +14,18 @@ using std::cout; using std::endl; extern "C" { - colm_value_t cc_action_params_find( struct colm_program *prg, tree_t **sp, str_t *_machine, str_t *_action ); - colm_value_t cc_action_params_insert( struct colm_program *prg, tree_t **sp, str_t *_machine, str_t *_action ); + colm_value_t cc_action_params_find( struct colm_program *prg, tree_t **sp, colm_value_t _machine, colm_value_t _action ); + colm_value_t cc_action_params_insert( struct colm_program *prg, tree_t **sp, colm_value_t _machine, colm_value_t _action ); } typedef set<string> Set; typedef map< string, Set > Map; static Map machineMap; -value_t cc_action_params_find( struct colm_program *prg, tree_t **sp, str_t *_machine, str_t *_action ) +value_t cc_action_params_find( struct colm_program *prg, tree_t **sp, value_t _machine, value_t _action ) { - string machine( _machine->value->data, _machine->value->length ); - string action( _action->value->data, _action->value->length ); + string machine( ( (str_t *) _machine )->value->data, ( (str_t *) _machine )->value->length ); + string action( ( (str_t *) _action )->value->data, ( (str_t *) _action )->value->length ); // cout << "cc_action_params_find " << machine << " " << action << " "; @@ -42,10 +42,10 @@ value_t cc_action_params_find( struct colm_program *prg, tree_t **sp, str_t *_ma return (value_t) res; } -value_t cc_action_params_insert( struct colm_program *prg, tree_t **sp, str_t *_machine, str_t *_action ) +value_t cc_action_params_insert( struct colm_program *prg, tree_t **sp, value_t _machine, value_t _action ) { - string machine( _machine->value->data, _machine->value->length ); - string action( _action->value->data, _action->value->length ); + string machine( ( (str_t *) _machine )->value->data, ( (str_t *) _machine )->value->length ); + string action( ( (str_t *) _action )->value->data, ( (str_t *) _action )->value->length ); // cout << "cc_action_params_insert " << machine << " " << action << endl; diff --git a/test/trans.d/actparams.cc b/test/trans.d/actparams.cc index d9e9d6ff..dab00b61 100644 --- a/test/trans.d/actparams.cc +++ b/test/trans.d/actparams.cc @@ -14,18 +14,18 @@ using std::cout; using std::endl; extern "C" { - colm_value_t cc_action_params_find( struct colm_program *prg, tree_t **sp, str_t *_machine, str_t *_action ); - colm_value_t cc_action_params_insert( struct colm_program *prg, tree_t **sp, str_t *_machine, str_t *_action ); + colm_value_t cc_action_params_find( struct colm_program *prg, tree_t **sp, colm_value_t _machine, colm_value_t _action ); + colm_value_t cc_action_params_insert( struct colm_program *prg, tree_t **sp, colm_value_t _machine, colm_value_t _action ); } typedef set<string> Set; typedef map< string, Set > Map; static Map machineMap; -value_t cc_action_params_find( struct colm_program *prg, tree_t **sp, str_t *_machine, str_t *_action ) +value_t cc_action_params_find( struct colm_program *prg, tree_t **sp, value_t _machine, value_t _action ) { - string machine( _machine->value->data, _machine->value->length ); - string action( _action->value->data, _action->value->length ); + string machine( ( (str_t *) _machine )->value->data, ( (str_t *) _machine )->value->length ); + string action( ( (str_t *) _action )->value->data, ( (str_t *) _action )->value->length ); // cout << "cc_action_params_find " << machine << " " << action << " "; @@ -42,10 +42,10 @@ value_t cc_action_params_find( struct colm_program *prg, tree_t **sp, str_t *_ma return (value_t) res; } -value_t cc_action_params_insert( struct colm_program *prg, tree_t **sp, str_t *_machine, str_t *_action ) +value_t cc_action_params_insert( struct colm_program *prg, tree_t **sp, value_t _machine, value_t _action ) { - string machine( _machine->value->data, _machine->value->length ); - string action( _action->value->data, _action->value->length ); + string machine( ( (str_t *) _machine )->value->data, ( (str_t *) _machine )->value->length ); + string action( ( (str_t *) _action )->value->data, ( (str_t *) _action )->value->length ); // cout << "cc_action_params_insert " << machine << " " << action << endl; |