Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#4022 closed defect (fixed)

Code generation for DAE mode broken

Reported by: Francesco Casella Owned by: Willi Braun
Priority: critical Milestone:
Component: Run-time Version:
Keywords: Cc:

Description (last modified by Francesco Casella)

I just ran the test of the ScalableTestSuite using the DAE mode, see ​https://test.openmodelica.org/libraries/ScalableTestSuite_Experimental/BuildModelRecursive.html.

Apparently, the generated C code is broken, as most models generate errors of this kind

ScalableTestSuite.Elementary.SimpleODE.Models.CascadedFirstOrder_16dae.c:47:91:
error: expected ')'

The flags are exactly the same as with the previous successful test, run on 26 July, ​https://test.openmodelica.org/libraries/history/ScalableTestSuite_Experimental/ScalableTestSuite_Experimental-2016-07-26.html.

Change History (5)

comment:1 by Francesco Casella, 8 years ago

Description: modified (diff)

comment:2 by Henning Kiel, 8 years ago

A better error description could be:

In file included from ScalableTestSuite.Elementary.SimpleODE.Models.CascadedFirstOrder_16dae.c:34:
./ScalableTestSuite.Elementary.SimpleODE.Models.CascadedFirstOrder_16dae.h:3:13: warning: ISO C99 requires whitespace after the macro name [-Wc99-extensions]
#define data->localData[0]->realVars[0] /* $DAEres[1] #UNKNOWN_VARKIND */ data->simulationInfo->daeModeData->residualVars[0]

comment:3 by Willi Braun, 8 years ago

Status: new → accepted

comment:4 by Willi Braun, 8 years ago

Resolution: → fixed
Status: accepted → closed

fixed in 9f51354.

comment:5 by Martin Sjölund, 7 years ago

Milestone: 1.10.0

Milestone deleted

Note: See TracTickets for help on using tickets.