Re: [pypy-dev] [pypy-commit] pypy default: I *think* the inputargs should always contain InputArgXxx objects that

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 <noreply@buildbot.pypy.org> wrote:

Hi Maciej, On Tue, Sep 15, 2015 at 9:11 AM, Maciej Fijalkowski <fijall@gmail.com> wrote:
No, I think it's only test_random.py that reuses operations from the main loop as fail_args when compiling a bridge, and these operations sometimes already have a get_forwarded() from the main loop's rewrite step. It should not occur outside this test. It took me a while to figure out what was wrong, though :-) A bientôt, Armin.

On Tue, Sep 15, 2015 at 10:49 AM, Armin Rigo <arigo@tunes.org> wrote:
Yes, forwarded has this problem that if you forget to clean it, there is a bit of trouble hard to debug :/ not sure what is the solution

Hi Fijal, On Tue, Sep 15, 2015 at 11:25 AM, Maciej Fijalkowski <fijall@gmail.com> wrote:
Yes, forwarded has this problem that if you forget to clean it, there is a bit of trouble hard to debug :/ not sure what is the solution
I think it's enough (or will become enough after one or two more such issues) to check in the next step that all operations and inputargs that you receive are not forwarded. That's what I did in this checkin. A bientôt, Armin.

Hi Maciej, On Tue, Sep 15, 2015 at 9:11 AM, Maciej Fijalkowski <fijall@gmail.com> wrote:
No, I think it's only test_random.py that reuses operations from the main loop as fail_args when compiling a bridge, and these operations sometimes already have a get_forwarded() from the main loop's rewrite step. It should not occur outside this test. It took me a while to figure out what was wrong, though :-) A bientôt, Armin.

On Tue, Sep 15, 2015 at 10:49 AM, Armin Rigo <arigo@tunes.org> wrote:
Yes, forwarded has this problem that if you forget to clean it, there is a bit of trouble hard to debug :/ not sure what is the solution

Hi Fijal, On Tue, Sep 15, 2015 at 11:25 AM, Maciej Fijalkowski <fijall@gmail.com> wrote:
Yes, forwarded has this problem that if you forget to clean it, there is a bit of trouble hard to debug :/ not sure what is the solution
I think it's enough (or will become enough after one or two more such issues) to check in the next step that all operations and inputargs that you receive are not forwarded. That's what I did in this checkin. A bientôt, Armin.
participants (2)
-
Armin Rigo
-
Maciej Fijalkowski