Error running JVML-JIT #14

Activity

1. SquidDev

   added

   bug

   on Oct 9, 2016
2. SquidDev commented on Oct 9, 2016

   

   SquidDev

   on Oct 9, 2016

   Last edited by SquidDev

   OwnerAuthor

   More actions

   A minimal reproduction:

   for i=1, 1 do
       local ca
       for v = 1, 1 do
           if v then ca = v end
       end
   
       assert(ca);
       (function() return ca end)()
   end
3. SquidDev commented on Oct 13, 2016

   

   SquidDev

   on Oct 13, 2016

   Last edited by SquidDev

   OwnerAuthor

   More actions

   It appears that the upvalue is being opened on the for loop, resulting in it being cleared each iteration. We either need to defer opening the variable or move opening to before the loop.

   Currently

   local ca
   for v = 1, 1 do
       -- Upvalue created here (and after the last iteration)
       if v then ca = v end
   end
   
   assert(ca);
   (function() return ca end)()

   Ideally

   local ca -- Upvalue created here
   for v = 1, 1 do
       if v then ca = v end
   end
   
   assert(ca);
   
   -- Or here
   (function() return ca end)()