1. 23 Sep, 2015 1 commit
  2. 09 Sep, 2015 2 commits
    • Kevin Modzelewski's avatar
      Merge pull request #905 from kmod/change_numdefaults · 54a9559e
      Kevin Modzelewski authored
      Allow changing the number of default arguments
      54a9559e
    • Kevin Modzelewski's avatar
      Allow changing the number of default arguments · df2808da
      Kevin Modzelewski authored
      We already supported changing the values, but not the number
      of them.  The main trickiness here is
      - We had been assuming that the number of defaults was immutable,
        so I had to find the places that we used it and add invalidation.
      - We assumed that all functions based on the same source function would
        have the same number of defaults.
      
      For the first one, I found all the places that looked at the defaults array,
      which should hopefully be all the places that need invalidation.
      
      One tricky part is that we will embed the num_defaults data into code produced
      by the LLVM tier, and we currently don't have any mechanism for invalidating
      those functions.  This commit side-steps around that since the only functions that
      we can inline are the builtins, and those you aren't allowed to change the defaults
      anyway.  So I added a "can_change_defaults" flag.
      
      For the second part, I moved "num_defaults" from the CLFunction (our "code" object)
      to the BoxedFunction (our "function" object), and then changed the users to pull
      it from there.
      df2808da
  3. 08 Sep, 2015 11 commits
  4. 07 Sep, 2015 2 commits
  5. 04 Sep, 2015 24 commits