[Octave-bug-tracker] [bug #64914] Bytecode VM crash with Symbolic package operation

[Petter]

Follow-up Comment #2, bug #64914 (project octave):

I can recreate the issue and made a patch addressing it.

Essentially the problem is that the vm pushed cell elements by row to
the stack before inserting them into a cell row, which didn't work
very well for big cells, leading to stack overflow.

There are some:

a = {a{:} b{:}};

in the python_copy_vars_to.m in the package, which led to the failure,
since the resulting cs-lists were pushed to the stack element by element.

The patch reworks how cells are created so that the row element
arguments are put straight into the cell after being evaluated.


pkg load symbolic
__vm_enable__ (1);
tic
val = sym (ones(89));
toc
whos
tic
val = sym (ones(89));
toc
whos


Runs fine with the patch.

(file #55349)

    _______________________________________________________

Additional Item Attachment:

File name: 32725.patch                    Size:18 KB
    <https://file.savannah.gnu.org/file/32725.patch?file_id=55349>



    _______________________________________________________

Reply to this item at:

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

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