Changeset 4f671af in OpenModelica

Timestamp:

2016-04-04T14:06:30+02:00 (8 years ago)

Author:

hudson <openmodelica@…>

Branches:

Added-citation-metadata, maintenance/v1.14, maintenance/v1.15, maintenance/v1.16, maintenance/v1.17, maintenance/v1.18, maintenance/v1.19, maintenance/v1.20, maintenance/v1.21, maintenance/v1.22, maintenance/v1.23, master, omlib-staging

Children:

536387b

Parents:

b4f223e

git-author:

Martin Sjölund <martin.sjolund@…> (04/04/16 14:06:30)

git-committer:

hudson <openmodelica@…> (04/04/16 14:06:30)

Message:

Do not use listReverse in append_reverse

The only difference is listReverse may optimize the case listLength==1.

File:

• Compiler/Util/List.mo (modified) (1 diff)

Compiler/Util/List.mo

@@ -407 +407 @@
   output list<T> outList=inList2;
 algorithm
-  if listEmpty(outList) then
-    outList := listReverse(inList1);
-    return;
-  end if;
+  // Do not optimize the case listEmpty(inList2) and listLength(inList1)==1
+  // since we use listReverseInPlace together with this function.
+  // An alternative would be to keep both (and rename this append_reverse_always_copy)
   for e in inList1 loop
     outList := e::outList;