Wrong C-code for Modelica.Math.FastFourierTransform.Examples.RealFFT1

[martin.otter@…]

In MSL trunk, I have changed Modelica.Math.FastFourierTransform.Examples.RealFFT1 so that the model translates in OpenModelica. However, the generated C-code is wrong. Error message from OpenModelica:

Modelica.Math.FastFourierTransform.Examples.RealFFT1_functions.c: In function 'omc_Modelica_Math_FastFourierTransform_Internal_rawRealFFT':
Modelica.Math.FastFourierTransform.Examples.RealFFT1_functions.c:104: error: '_work' undeclared (first use in this function)
Modelica.Math.FastFourierTransform.Examples.RealFFT1_functions.c:104: error: (Each undeclared identifier is reported only once
Modelica.Math.FastFourierTransform.Examples.RealFFT1_functions.c:104: error: for each function it appears in.)

The function calls an external C-function with a work array:

function rawRealFFT 
  input Real  u[:] ;
  output Integer info ;
  output Real amplitudes[div(size(u,1),2)+1];
  output Real phases[    div(size(u,1),2)+1];
protected 
  Real work[3*size(u,1) + 2*(div(size(u,1),2)+1)];
  external "C" info = ModelicaFFT_kiss_fftr(u, size(u,1), work, size(work,1), amplitudes, phases)

and the protected "work" seems to be "somehow" not included in the C-code (a similar structure appears for Modelica.Math.Matrices.* when LAPACK functions are called.

[Martin Sjölund]

Milestone pushed to 1.9.5

[Martin Sjölund]

Milestone renamed

[Martin Sjölund]

Ticket retargeted after milestone closed

[Martin Sjölund]

Milestone moved to 1.12.0 due to 1.11.0 already being released.

[Francesco Casella]

The milestone of this ticket has been reassigned to "Future".

If you think the issue is still valid and relevant for you, please select milestone 1.13.0 for back-end, code generation and run-time issues, or 2.0.0 for front-end issues.

If you are aware that the problem is no longer present, please select the milestone corresponding to the version of OMC you used to check that, and set the status to "worksforme".

In both cases, a short informative comment would be welcome.

[Per Östlund]

The code generation issue seems to have been fixed since the model now simulates. There's still some issue with not getting the correct result, but I guess that's covered by #5855.