…0 rc1"

This fixed the issue but @subbuss rightly points out this scope should have
been marked as BINDING_HAS_ESCAPED and thus this pass should not have even
ran.

This reverts commit 467e2f7.

This replaces the previous fix.  The previous fix was wrong because the
code in question contained a closure and we were not marking the parent
method as scope as BINDING_HAS_ESCAPED.  As a result, OptimizeDelegationPass
was executing and mucking things up.  The previous fix corrected the pass
to work with nested closures but it would create new problems if that closure
did something which needed to conume the parents binding.

The new fix is just a simple oversight in a few of our instructions.  ZSuper,
UnresolvesSuper, and Match are calls but they were not calling their
superclasses computeScopeFlags.  This is turn made the issue at hand not notice
that the method contained a closure so it never marked BINDING_HAS_ESCAPED.

This fix should potentially fix other weird errors where we did not set common
call IRScope flags.

In MRI, rb_read_internal always just does a blind read on the
descriptor, since kernel-level interruption of that blocking call
is safe. On JDK, interruption of an NIO channel read generally
closes that channel, because there's too much state between Java
and the native side to ensure has been cleaned up correctly. Our
normal behavior when dealing with NIO channels is then to always
use select before blocking reads, so we can safely interrupt when
there's no data available. However, this is not appropriate for
native descriptors, like the FIFO in this case, because all ENXIO
channels appear to be selectable (so we attempt it) and the
additional select does not match MRI semantics here.

My change is to modify our version of rb_read_internal to never
select when using a native descriptor. ENXIO should not damage the
descriptor when interrupted, and we should still be able to
interrupt it properly.

Note also that we generally use ENXIO native channels for the
following cases, so only these cases should see any changed
behavior:

* stdio
* some pipes
* most files

This reverts commit 04db9d8.

variableNames is updated when IRScope gets set into StaticScope,
and is used for other sizing and definition logic, so make it the
sole place we get variable counts from when sizing scopes.

…e argument.

…sk staticScope for it (closer proximity to ic than ss from interpreterengine's perspective)

We have largely committed to JRuby 9k being Java 7+, and to
simplify the process of binding jitted method bodies we are using
MethodHandle exclusively. Because of this, and because MHs work
anywhere reflection works, we are removing the bytecode-generate
CompiledMethod and the Reflected*Method.

There's also a bit of peripheral cleanup for code that is defunct
with these classes removed.

…0.0.rc1

needs more testing and fixes
This reverts commit 71f95f7.

fixes #3095

Sponsored by Lookout Inc.

Conflicts:
	lib/ruby/shared/rubygems/defaults/jruby.rb

…ties.

* Integer.class would be for the case when we need to allow null.

Take two.  The first one broke array packing.

…t classloader

…gument in the :string_character_byte_index primitive.

…ke as input and return as output, zero-offset indices.

ByteLists with 'begin' values > 0 are an implementation detail and should be handled internally only.

…be consistent with the MatchData calculations.

* Fixes remaining specs.