You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@headius@enebo This bug opens up the potential for cleanup that I had been considering for a while. I had introduced the scope / module variable stuff long back so that we can exploit static information where available. However, as part of removing IRScope references for JIT, that basic intention has been a bit obscured.
In any case, will take a look at this later tonight and fix up the mixing initialization.
The bug is fixed, but let us chat later about cleaning up the CurrentScope/CurrentModule uses. I have a feeling get rid of the temporary variables and simplify some of that old code.
The bug is fixed, but let us chat later about cleaning up the
CurrentScope/CurrentModule uses. I have a feeling get rid of the temporary
variables and simplify some of that old code.
—
Reply to this email directly or view it on GitHub #2284 (comment).
Reproducible with this simple snippet:
The IR built for the lookup of C is:
Notice the %v_1 use without a corresponding definition for it.
This is the source of the two travis-core failures seen in https://travis-ci.org/travis-ci/travis-core/jobs/42825525
The text was updated successfully, but these errors were encountered: