Apply the VM-to-State transition to the JIT code #1646

Merged
merged 1 commit into from Apr 2, 2012

Projects

None yet

2 participants

@ryoqun
Member
ryoqun commented Apr 2, 2012

Since the following commit, many C++ functions are changed to take a pointer to
a State, not a pointer to a VM. But the LLVM code hasn't catch up the change.

35a49f3 Introduce State class as STATE

This is very confusing. Just update it.

The reason Rubinius doesn't currently break in this situation is that the LLVM
IR type system is completely independent of the actual C++ type system.

@ryoqun ryoqun Apply the VM-to-State transition to the JIT code
Since the following commit, many C++ functions are changed to take a pointer to
a State, not a pointer to a VM. But the LLVM code hasn't catch up the change.

  35a49f3 Introduce State class as STATE

This is very confusing. Just update it.

The reason Rubinius doesn't currently break in this situation is that the LLVM
IR type system is completely independent of the actual C++ type system.
4bf2ad4
@dbussink dbussink merged commit 668375c into rubinius:master Apr 2, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment