Also make pc a VALUE instead of ptr to VALUE

Need to read the index

* Remove redundant  statements

* Remove .clone() since A64 implements Copy

* Remove .clone() since InsnId implements Copy

.

* Dereference since *const rb_call_data implements Copy

* Remove unnecessary return statement

* Remove unnecessary braces

* Use .is_empty() over length checks

* Remove unnecessary conversion handling

Since i32 can always fit into i64 (the inner type in Opnd::Imm), the conversion is infallibile.

* Use slice notation in lieu of Vec

https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg

* Simplify match statement

We want to return representatives for the stack and locals.

* Remove immediately deferenced borrow operators

source: https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

* Remove unnecessary cast as i64

Since Const::CInt64 wraps i64, it is unnecessary to cast to i64.

* Remove unnecessary borrow operators

* Beautify assign operation syntax

* Use .is_null() convenience method

* Omit .into() call from u32 to u32

* Use more descriptive std::ptr::null<VALUE>()

In lieu of casting a literal as a type, opt to use the convenience type.

* Use sized integer literal

* Simplify pattern matching to explicit check

* Add --zjit-profile-interval option

* Fix min to max

* Avoid rewriting instructions for --zjit-call-threshold=1

* Rename the option to --zjit-num-profiles

* Disable ZJIT profiling at call-threshold

* Stop referencing ZJIT instructions in codegen

In case the callee writes to the caller's locals, we need to be able to
immediately side-exit because our frame's invariants might have been
invalidated.

If in the course of optimization we rewrite the Send, we can choose to
remove the PatchPoint too---but it's not an error to keep it around.

A new optimization pass. Also:
- Printing for `Insn::CCall`
- Wrap `ID` and add convenience method for printing, replacing calls to rb_id2name()

We don't want to only guard on the class

We need to keep the Snapshot instruction alive.

Helps with readability and threads type info that's there around for
longer. Amend test so `GuardType` doesn't DCE'ed.

* Implement JIT-to-JIT calls

* Use a closer dummy address for Arm64

* Revert an obsoleted change

* Revert a few more obsoleted changes

* Fix outdated comments

* Explain PosMarkers for CCall

* s/JIT code/machine code/

* Get rid of ParallelMov

* Show failing test

* Add second test case

* Add empty NewArray setup

* Update opt_tests and fix NewArray instantiation

* Add code generation for NewArray

* Add NewArray ordering test

This makes the output a little nicer when doing --zjit-dump-hir=all