wrong blt, becasue inlining of functions fail for more then one output

[Vitalij Ruge]

package inlineTest
  model A
    function f2
      input Real x1;
      input Real x2;
      output Real y1;
      output Real y2;
    algorithm
      y1 := cos(x1 + x2);
      y2 := sin(x1);
      annotation(Inline = true);
    end f2;

    Real x, z, y;
  equation
    (x, z) = f2(x, y);
    z = sin(time * x);
  end A;

  model B
    Real x, z, y;
  equation
    x = cos(x + y);
    z = sin(x);
    z = sin(time * x);
  end B;
end inlineTest;

In model B I try to inline f2.
The result for A: x,y,z are part of a loop.
The result for B: x,z are part of a loop.

[Lennart Ochel]

What flags do I need to reproduce the wrong BLT?

[Vitalij Ruge]

In Inline.mo

(crefs,{cr},stmts,repl) = getFunctionInputsOutputBody

we have the limit for the outputs {cr} only one element

[Vitalij Ruge]

Replying to lochel:

What flags do I need to reproduce the wrong BLT?

see description

[Vitalij Ruge]

would it be possible to fixe/extend the frondend modul Inline.inlineCall and forceInlineCall for this case?

[Lennart Ochel]

Well, I cannot see a wrong BLT. It is just that the equation system is different due to inlining (by hand in the second case).
So is this just about the limitation of the inlining module? Or is the BLT really wrong?

[Martin Sjölund]

f2 should give a warning: Inline=true was given, but the function has more than a single assignment statement to a single variable and cannot be inlined. Or someone needs to rewrite the module to handle this.

[Vitalij Ruge]

Replying to sjoelund.se:

Or someone needs to rewrite the module to handle this.

done in ​251

[Martin Sjölund]

Milestone renamed

[Martin Sjölund]

Milestone renamed