bugGNU Octave - Bugs: bug #64703, VM machine has issues to cope with...

 
 

bug #64703: VM machine has issues to cope with functions containing both inline as well as nested functions

Submitter:  None
Submitted:  Thu 21 Sep 2023 05:54:09 AM UTC
   
 
Category:  None Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Fixed Assigned to:  None
Originator Name:  Originator Email:  -email is unavailable-
Open/Closed:  * Closed Release:  * dev
Operating System:  * GNU/Linux Fixed Release:  None
Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 01 Oct 2023 04:40:32 PM UTC, comment #3: 

No complaints from CI. Closing as fixed.

Arun Giridhar <arungiridhar>
Group Member
Sat 23 Sep 2023 03:24:44 PM UTC, comment #2: 
Arun Giridhar <arungiridhar>
Group Member
Sat 23 Sep 2023 12:14:50 AM UTC, comment #1: 


VM Remove forgotten feuture block for nested functions (bug #64703)

Also allow compiling inline functions.

* libinterp/parse-tree/pt-bytecode-walk.cc: Remove check for nested, inline
* test/compile/bytecode_nested.m: Update tests


Please see attached patch.

(file #55159)

Petter <petter>
Thu 21 Sep 2023 05:54:09 AM UTC, original submission:  

The VM machine has issues to cope with functions containing both inline as well as nested functions.


test1.m
------------------------------------------

function retval = test1(x)
  retval = zeros(2,1);

  f1 = @(x) 3 .* x;
  retval(1) = f1(x);

  function [ ret ] = f2(x)
    ret = 4 .* x;
  endfunction

  retval(2) = f2(x);
endfunction
-------------------------------------------


_enable_vm_eval_ (0); test1(1)
ans =

   3
   4


_enable_vm_eval_ (1); test1(1)
warning: Auto-compilation of anonymous function failed with message Inlined or scoped functions are not supported by the VM yet
warning: called from
    test1 at line 3 column 10

ans =

   3
   4

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #55159:  octave_32321.patch added by petter (2KiB - text/plain)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by arungiridhar (Posted a comment)
  • -email is unavailable- added by petter (Updated the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only project members can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2023-10-01 arungiridhar StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2023-09-23 arungiridhar StatusNone Ready For Test
    2023-09-23 petter Attached File- Added octave_32321.patch, #55159

    Back to the top

    Powered by Savane 3.12