[Octave-bug-tracker] [bug #56752] Performance slowdown from version 3.2.4 through to current dev branch

[anonymous]

Follow-up Comment #42, bug #56752 (project octave):

@Rik, I am not sure whether other uses of NoAlias should also be changed.

All the instances that you found appear to be of the form 


octave_value_list retval;
NoAlias<Array<T>> r;
do_stuff ();
retval (0) = r;
return retval;


I am not sure in this case 

1) whether retval (0) = r; requires a copy
2) whether avoiding use of NoAlias would allow 
   the compiler to avoid the copy

(My guess is the copy does not happen in this case or is at most a shallow
copy)

On the other hand I see the implementation of NoAlias itself can be simplified
(as suggested by the FIXME comment) as we now require c++11.

Patch to simplify NoAlias is attached.

(file #47491)
    _______________________________________________________

Additional Item Attachment:

File name: open_0qtoU59a.txt              Size:1 KB
    <https://savannah.gnu.org/file/open_0qtoU59a.txt?file_id=47491>



    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?56752>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/