[pypy-dev] [pypy-commit] pypy unroll-if-alt: a) Renamed @unroll_if to @look_inside_iff at carl's request

[Alex Gaynor]

On Wed, Aug 3, 2011 at 1:06 AM, Antonio Cuni <anto.cuni at gmail.com> wrote:

> On 03/08/11 02:16, alex_gaynor wrote:
>
>> Author: Alex Gaynor<alex.gaynor at gmail.com>
>> Branch: unroll-if-alt
>> Changeset: r46219:524483d98407
>> Date: 2011-08-02 17:17 -0700
>> http://bitbucket.org/pypy/**pypy/changeset/524483d98407/<http://bitbucket.org/pypy/pypy/changeset/524483d98407/>
>>
>> Log:    a) Renamed @unroll_if to @look_inside_iff at carl's request b)
>>        Unroll Argument.unpack if one should. c) Leave a comment about a
>>        small level mess with keyword argumengs and the decorator
>>
>> diff --git a/pypy/interpreter/argument.py b/pypy/interpreter/argument.py
>> --- a/pypy/interpreter/argument.py
>> +++ b/pypy/interpreter/argument.py
>> @@ -125,6 +125,7 @@
>>
>>      ###  Manipulation  ###
>>
>> +    @jit.look_inside_iff(lambda self: not self._dont_jit)
>>      def unpack(self): # slowish
>>          "Return a ([w1,w2...], {'kw':w3...}) pair."
>>          kwds_w = {}
>> @@ -245,6 +246,8 @@
>>
>
> I think that this should have a test_pypy_c test.
>
> ciao,
> Anto
> ______________________________**_________________
> pypy-dev mailing list
> pypy-dev at python.org
> http://mail.python.org/**mailman/listinfo/pypy-dev<http://mail.python.org/mailman/listinfo/pypy-dev>
>

Probably, or I can revert it.  Currently in pyjitpl a call wipes all heap
caches in the frontend, so I wanted to remove that call so that something
would be properly constant early enough, unfortunately I forgot creating a
dict is a call of it's own, so it doesn't do anything useful ATM.

Alex

-- 
"I disapprove of what you say, but I will defend to the death your right to
say it." -- Evelyn Beatrice Hall (summarizing Voltaire)
"The people's good is the highest law." -- Cicero
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.python.org/pipermail/pypy-dev/attachments/20110803/0e7a3db0/attachment.html>