Ticket #983 (closed bug: wontfix)

Opened 5 years ago

Last modified 5 years ago

JIT code incorrectly access registers

Reported by: bacek Owned by:
Priority: normal Milestone:
Component: core Version: trunk
Severity: medium Keywords:
Cc: Language:
Patch status: Platform:

Description

Hello.

In r41023 (re-enabled in r41040) I've put strict asserts to access Context registers. make testj failing in t/op/calling.t (#52) and t/pmc/sub.t (#36).

-- Bacek

Change History

  Changed 5 years ago by jkeenan

  • component changed from none to core
  • summary changed from [bug] JIT code incorrectly access registers to JIT code incorrectly access registers

follow-up: ↓ 3   Changed 5 years ago by bacek

  • status changed from new to closed
  • resolution set to wontfix

Current JIT is deprecated.

in reply to: ↑ 2   Changed 5 years ago by jkeenan

  • status changed from closed to reopened
  • resolution wontfix deleted

Replying to bacek:

Current JIT is deprecated.

Even if the current JIT is deprecated, we can't do a release with these two tests failing under make testj. If the tests should not pass under make testj from this point going forward, then, at the very least, they need to be SKIPped. But we should have some understanding or explanation as to why the changes in r41023 caused these failures.

Ticket must be reopened.

Thank you very much.

kid51

  Changed 5 years ago by bacek

Root problem - JIT-ed code access non-existed registers. Asserts from r41023 just exposed it. So, SKIPing those tests for JITed version is fine for release purpose.

  Changed 5 years ago by bacek

  • status changed from reopened to closed
  • resolution set to wontfix

Tests were marked with "todo" at r41258. Closing ticket as "wontfix" because root issue not going to be resolved.

Note: See TracTickets for help on using tickets.