It might contain something else than InputArgXxx (because of e.g. not
tracing from the start), but indeed the forwarded should be set to
None by the time we reach the backend (maybe we forget it too late?)

On Mon, Sep 14, 2015 at 7:31 PM, arigo <nore...@buildbot.pypy.org> wrote:
> Author: Armin Rigo <ar...@tunes.org>
> Branch:
> Changeset: r79638:cfe5d8e88d3d
> Date: 2015-09-14 19:31 +0200
> http://bitbucket.org/pypy/pypy/changeset/cfe5d8e88d3d/
>
> Log:    I *think* the inputargs should always contain InputArgXxx objects
>         that are not forwarded. (Hitting the case from test_ll_random)
>
> diff --git a/rpython/jit/backend/x86/regalloc.py 
> b/rpython/jit/backend/x86/regalloc.py
> --- a/rpython/jit/backend/x86/regalloc.py
> +++ b/rpython/jit/backend/x86/regalloc.py
> @@ -134,6 +134,8 @@
>          self.final_jump_op = None
>
>      def _prepare(self, inputargs, operations, allgcrefs):
> +        for box in inputargs:
> +            assert box.get_forwarded() is None
>          cpu = self.assembler.cpu
>          self.fm = X86FrameManager(cpu.get_baseofs_of_frame_field())
>          operations = cpu.gc_ll_descr.rewrite_assembler(cpu, operations,
> _______________________________________________
> pypy-commit mailing list
> pypy-com...@python.org
> https://mail.python.org/mailman/listinfo/pypy-commit
_______________________________________________
pypy-dev mailing list
pypy-dev@python.org
https://mail.python.org/mailman/listinfo/pypy-dev

Reply via email to