JIT: Fix for lightweight assembler retry mechanism.

Removed a special path in the assembler that took in sideband
info on some instructions and used it to modify pc-relative load
addresses.  This mechanism (used in some older chaining cells) is
no longer needed, and was causing a problem for the new lightweight
assembler retry mechanism.

Change-Id: I1cca5b719bcdea0810831d889ebe2f935c61966f

JIT: Fix for 2898397 assert in JIT (chain cell offset)

My previous change to allow for lightweight assembler retries neglected
to restore a sanity check cookie when re-assembling without fully rebuilding
the trace.

Change-Id: Ief7eb76263e272b9cd79345f8344bd35926f5b10

More SMP fixes.

Convert some ANDROID_MEMBAR_FULL uses into equivalent atomic ops.  A
couple of "bool" had to convert to "int" since we don't have atomic
ops for bools.

Replaced a local implementation of atomic inc with a call to the
atomic inc function.

Change-Id: I948b8080d743552bde014d3a6e716ed2c30ebef8

am b0691476: Remove stale overwritefree flag.

Merge commit 'b0691476e24deae014dadbdbbceabc72d5a96ae6' into dalvik-dev

* commit 'b0691476e24deae014dadbdbbceabc72d5a96ae6':
  Remove stale overwritefree flag.

Remove stale overwritefree flag.

Change-Id: If14998128245245c908b2390acfe28b82339f8bf

am f98e0ed5: Merge "Add const qualifier to the dvmComputeStringHash signature."

Merge commit 'f98e0ed527d4b0b489bd714bb4e8606dde53973d' into dalvik-dev

* commit 'f98e0ed527d4b0b489bd714bb4e8606dde53973d':
  Add const qualifier to the dvmComputeStringHash signature.

Merge "Add const qualifier to the dvmComputeStringHash signature."

Add const qualifier to the dvmComputeStringHash signature.

Change-Id: Icc963f324ca738438c069e99f346826d7f301095

Tweaks to x86 mterp periodic checks [issue 2634642]

Update the x86 mterp main interpetation loop to match Arm and the portable
interpreter by checking for debugger/profiler after suspends.  Also,
enable ability to skip a load when checking for active debugger (both
Arm and x86).

Related cl: https://android-git.corp.google.com/g/#change,50229

Change-Id: Ic5d0f102b2250393eaf4405e4cafb7678b1a563e

am ce82d2d1: am c9153114: Merge remote branch \'goog/dalvik-dev\' into dalvik-dev-to-gingerbread

Merge commit 'ce82d2d1642e55ae62932003187cabc3f5db9e84' into dalvik-dev

* commit 'ce82d2d1642e55ae62932003187cabc3f5db9e84':

am c9153114: Merge remote branch \'goog/dalvik-dev\' into dalvik-dev-to-gingerbread

Merge commit 'c915311408e45b4687d230641382e025cf38805d'

* commit 'c915311408e45b4687d230641382e025cf38805d':
  Add a missing branch for inlined direct getters.
  Implement method inlining for getters/setters
  Switch -Xjnitrace over to using a custom JNI bridge.
  Change dvmHeapBitmapXorWalk to dvmHeapBitmapSweepWalk.
  Break apart the swapping of the mark and live bitmaps and zeroing of
  Fix a crash during VM shutdown.
  Update the atomics test for the new atomics API.
  Allow the VM to suspend while the heap worker initializes.

Merge "JIT: Support for shift field in disassembler [issue 2465170]" into dalvik-dev

JIT: Support for shift field in disassembler [issue 2465170]

Many Thumb2 instructions allow for an optional shift to be applied to
operand 2.  This cl enhances the disassembler to show a non-zero shift
in the dump output.

Change-Id: I6cbfec46cd45e3f2acc9553f880e80764a1eb65d

Merge "Move dvmFreeClassInnards responsibility." into dalvik-dev

JIT: Reworked the assembler to be smarter about short instruction forms

Previously, the JIT wasn't generating short-form compare and branch on
zero/not zero instructions for Thumb2.  The reason was that these only
allow a 1-byte displacement, and when they didn't reach the assembler would
abort the trace, split it in half and try again.  This change re-enables
cbz, cbnz generation and introduces a relatively lightweight retry
mechanism.

Also includes changes for Thumb2 to always generate large displacement
literal loads and conditional branches to minimize the number of retry
attempts.

Change-Id: Icf066836fad203f5c0fcbbb2ae8e1aa73d1cf816

Merge remote branch 'goog/dalvik-dev' into dalvik-dev-to-gingerbread

Bug fixes for JIT loop detection and formation

Specifically:
- Don't apply loop optimization if the basic induction variable is
  manipulated (ie excluding cases like "i+=0")
- Fix a case where variables reloaded with constants in the body are
  not considered as loop invariants

Bug: 2804188
Change-Id: Ia5ebb29bc6814b1be069e23794585f8313900b7d

am baf196a7: Fix for 2892472 jit spew: No free temp registers

Merge commit 'baf196a7b259704137cccf4058b78da15d4c3998' into dalvik-dev

* commit 'baf196a7b259704137cccf4058b78da15d4c3998':
  Fix for 2892472 jit spew: No free temp registers

Move dvmFreeClassInnards responsibility.

The GC has been responsible for ensuring that the "innards" of a
class object are freed before the ClassObject itself is discarded.
Since we don't currently unload classes, this situation can only arise
if something went wrong during the loading of a class (e.g. memory
alloc failure part way through), or another thread loaded the same
class at the same time and won the race.  It makes more sense to
have the cleanup occur at the point we decide that our class object
is not useful.

For the most part this was already being done, but there were a couple
of places where it wasn't.

This change
 - adds the missing dvmFreeClassInnards calls
 - changes the GC code to warn without taking action
 - enables the GC code only when assertions are enabled

The code in MarkSweep.c looks like it merits further rearranging, but
I'll leave that to Team GC.

Also, "dalvik_assert" is now only printed on the VM features line when
assertions are enabled.

For bug 2751668.

Change-Id: I90e4455e830766f94f921c7f23b44b423bce3321

Fix for 2892472 jit spew: No free temp registers

Neglected some register allocation hygene, which caused a
problem on armv5te builds.

Change-Id: I666b39b88822c4d3d3d7f0e84386aca2920bb9f1

Merge "New material for JNI Tips doc." into dalvik-dev

New material for JNI Tips doc.

Added three new "FAQ" sections, covering the questions that come up
most frequently on the android-ndk mailing list.

Augmented the "Unsupported Features" section with a list of items
that didn't used to work, and noted when they started working.

Various minor updates.

Change-Id: I9acb22e70f6dae1d9b5011da0eab08b6e739d5f9

Add a missing branch for inlined direct getters.

Change-Id: Ie11925b1856b32aeb6855601195e8a13d23c4e94

Implement method inlining for getters/setters

Changes include:
1) Force the trace that ends with an invoke instruction to include
   the next instruction if it is a move-result (because both need
   to be turned into no-ops if callee is inlined).
2) Interpreter entry point/trace builder changes so that return
   target won't automatically be considered as trace starting points
   (to avoid duplicate traces that include the move result
   instructions).
3) Codegen changes to handle getters/setters invoked from both
   monomorphic and polymorphic callsites.
4) Extend/fix self-verification to form identical trace regions and
   handle traces with inlined callees.
5) Apply touchups to the method based parsing - still not in use.

Change-Id: I116b934df01bf9ada6d5a25187510e352bccd13c

am 4dc622c5: Remove the dvmHeapSizeChanged no-op routine.

Merge commit '4dc622c5b4ce48b5f2f0a0f92f316dd557fc950f' into dalvik-dev

* commit '4dc622c5b4ce48b5f2f0a0f92f316dd557fc950f':
  Remove the dvmHeapSizeChanged no-op routine.

Remove the dvmHeapSizeChanged no-op routine.

Change-Id: I6deb4ea858610edee6e7aa44d49d91cae5a25404

am 6fa09f77: Fix the format conversions, again.

Merge commit '6fa09f77cda8be8505897881a74821886c5aa116' into dalvik-dev

* commit '6fa09f77cda8be8505897881a74821886c5aa116':
  Fix the format conversions, again.

Fix the format conversions, again.

Change-Id: I6e46d882bad1e3264caeffec6196ce6ed07299ef

am 4d529be0: Fix log message format conversions and remove uneeded newlines.

Merge commit '4d529be041b5a8c25647da505aed109d5f3f4e69' into dalvik-dev

* commit '4d529be041b5a8c25647da505aed109d5f3f4e69':
  Fix log message format conversions and remove uneeded newlines.

Fix log message format conversions and remove uneeded newlines.

Change-Id: Iae538c643d86522e8f0a52e6f01e411bbdea892b

am 5488c8b2: Merge "Change dvmHeapBitmapXorWalk to dvmHeapBitmapSweepWalk."

Merge commit '5488c8b2a6f4254cdfa4abbb655784fb5d632008' into dalvik-dev

* commit '5488c8b2a6f4254cdfa4abbb655784fb5d632008':
  Change dvmHeapBitmapXorWalk to dvmHeapBitmapSweepWalk.

Merge "Change dvmHeapBitmapXorWalk to dvmHeapBitmapSweepWalk."

am f3e71ffb: Merge "JIT: Fix for Issue 2881687 - fix volatiles and re-enable JIT"

Merge commit 'f3e71ffb84adfb0401e5b0d164e505ae3df22c01' into dalvik-dev

* commit 'f3e71ffb84adfb0401e5b0d164e505ae3df22c01':
  JIT: Fix for Issue 2881687 - fix volatiles and re-enable JIT

Merge "JIT: Fix for Issue 2881687 - fix volatiles and re-enable JIT"

Change dvmHeapBitmapXorWalk to dvmHeapBitmapSweepWalk.

Concurrent sweeping will require the callback to process objects which
are live and not marked, while the client code continues to allocate
and set bits. Since dvmHeapBitmapXorWalk was only used to sweep, it
seemed easiest to make it learn its true purpose.

Merge "Switch -Xjnitrace over to using a custom JNI bridge." into dalvik-dev

Switch -Xjnitrace over to using a custom JNI bridge.

This now works without needing to specify -Xint:portable, and no longer needs
to be manually enabled at compile-time.

If you're using -Xjnitrace, you'll pay to look up the real JNI bridge on every
invocation of a native method, but doing so leads to a really simple
implementation, and you're writing to the log on every invocation anyway, so
-Xjnitrace and performance don't go together anyway.

Bug: 2846017

Change-Id: I3108c4f35f27b72e8849c41077ae3e8768550e96

am 79a76f30: Remove the seldom used overwrite free code.

Merge commit '79a76f30a7fe513e08b82464b68e00d77b170f54' into dalvik-dev

* commit '79a76f30a7fe513e08b82464b68e00d77b170f54':
  Remove the seldom used overwrite free code.

Remove the seldom used overwrite free code.

This is being done in preparation of a change that will eliminate the
property handler from AndroidRuntime.cpp.

Change-Id: I05e43dab2e6e195048642e51d01986ed63472d21

am a20a9929: Move LIKELY and UNLIKELY to a public header file.

Merge commit 'a20a99298b45b4b6c7c9c528066a3e055e601535' into dalvik-dev

* commit 'a20a99298b45b4b6c7c9c528066a3e055e601535':
  Move LIKELY and UNLIKELY to a public header file.

Move LIKELY and UNLIKELY to a public header file.

Change-Id: I6c4429277f50ff34021bd2ef836834605323b9bd

am 577534a4: am 486e2d58: am 7f626db7: I got tired of rewriting something like this text into emails and bug comments.

Merge commit '577534a45611f2011cbfa5aa99590e8ef565250e' into dalvik-dev

* commit '577534a45611f2011cbfa5aa99590e8ef565250e':
  I got tired of rewriting something like this text into emails and bug comments.

am 486e2d58: am 7f626db7: I got tired of rewriting something like this text into emails and bug comments.

Merge commit '486e2d586ba964611e5d86742d0c8057268b7771'

* commit '486e2d586ba964611e5d86742d0c8057268b7771':
  I got tired of rewriting something like this text into emails and bug comments.

am 7f626db7: I got tired of rewriting something like this text into emails and bug comments.

Merge commit '7f626db7ae1408dfa7248de7f27fa65cae6fb0ce' into gingerbread-plus-aosp

* commit '7f626db7ae1408dfa7248de7f27fa65cae6fb0ce':
  I got tired of rewriting something like this text into emails and bug comments.

I got tired of rewriting something like this text into emails and bug comments.

Change-Id: Ib3bf57e0bac91567b6a331cd641c7589996e6a02

JIT: Fix for Issue 2881687 - fix volatiles and re-enable JIT

Change-Id: I2ccb4b46d92b729d5504ea9c1b771a5147096858

Change dvmHeapBitmapXorWalk to dvmHeapBitmapSweepWalk.

Concurrent sweeping will require the callback to process objects which
are live and not marked, while the client code continues to allocate
and set bits. Since dvmHeapBitmapXorWalk was only used to sweep, it
seemed easiest to make it learn its true purpose.

Change-Id: I44edd15e5a7447d029a7ae60c6cd9b101cb1e9d9

Break apart the swapping of the mark and live bitmaps and zeroing of
the mark bitmap.

This paves the way for concurrent sweep.

Change-Id: I93a95188fecfd69d1d1933391a332537649206fa

am 006346e8: Change the bitmap callback signature to have a void return value.

Merge commit '006346e8f138b4e4b16c7a94b049c1646b009b52' into dalvik-dev

* commit '006346e8f138b4e4b16c7a94b049c1646b009b52':
  Change the bitmap callback signature to have a void return value.

Change the bitmap callback signature to have a void return value.

Previously, a bitmap callback could abort the bitmap traversal by
returning false.  All of the bitmap callbacks used in the garbage
collector are hardwired to return true.  This change eliminates the
early termination facility and its attendant complexity.

Change-Id: I5490dd74b56b9700ec1b7e657637fae5eb4cc339

am 995b5ae8: Merge "Remove unused limits.h include and fix sys/mman.h include comment."

Merge commit '995b5ae8fc12df378dc4f228c86ab5f5bcb3a1a2' into dalvik-dev

* commit '995b5ae8fc12df378dc4f228c86ab5f5bcb3a1a2':
  Remove unused limits.h include and fix sys/mman.h include comment.

Merge "Remove unused limits.h include and fix sys/mman.h include comment."

Remove unused limits.h include and fix sys/mman.h include comment.

Change-Id: I00299ca598af03da27be29867b71588c3aa81e83

am a363035f: Remove an unused member from the mark context.

Merge commit 'a363035f3bbf56f6c4fe500dd85bc2f1330d7a20' into dalvik-dev

* commit 'a363035f3bbf56f6c4fe500dd85bc2f1330d7a20':
  Remove an unused member from the mark context.

Remove an unused member from the mark context.

Change-Id: Ic85061af92e507001ddfb61710e996362078ec9d

am 7a57d5b7: Re-enable the concurrent GC.

Merge commit '7a57d5b78f15da77f5b5cab14c8c347883b2a343' into dalvik-dev

* commit '7a57d5b78f15da77f5b5cab14c8c347883b2a343':
  Re-enable the concurrent GC.

Re-enable the concurrent GC.

Change-Id: I1ddee8491788bb60371cdbadf25b53c65e012e0e

am deac1a95: Merge "Allow the VM to suspend while the heap worker initializes."

Merge commit 'deac1a955f63be277342bc22c264c43aa1992f9c' into dalvik-dev

* commit 'deac1a955f63be277342bc22c264c43aa1992f9c':
  Allow the VM to suspend while the heap worker initializes.

Merge "Allow the VM to suspend while the heap worker initializes."

Allow the VM to suspend while the heap worker initializes.

The concurrent collector may initiate a collection, holding the heap
worker lock, while the heap worker thread starts up.  This causes the
thread suspension to wait indefinitely for the heap worker thread and
forces a VM abort.  The following stack trace is logged:

  #00 __futex_syscall3
  #01 pthread_mutex_lock<-_normal_lock
  #02 heapWorkerThreadStart<-dvmLockMutex
  #03 internalThreadStart
  #04 __thread_entry
  #05 pthread_create

This change puts the heap worker thread in to the VM wait state while
it acquires its locks during startup thereby avoiding the crash.  The
first dvmLockMutex substituted seems to have quelled the crashes at
startup.  The second may be unnecessary but seems benign.

am 35ff69be: JIT: temporarily disable the JIT and concurrent GC on SMP systems

Merge commit '35ff69be47f3f2bd33f2a2d8562eaa2ac3dc9f1d' into dalvik-dev

* commit '35ff69be47f3f2bd33f2a2d8562eaa2ac3dc9f1d':
  JIT: temporarily disable the JIT and concurrent GC on SMP systems

JIT: temporarily disable the JIT and concurrent GC on SMP systems

Change-Id: If2d489b2aca39cd0297453f311a5c3ce80f577d9

am e6899ef5: Merge remote branch \'goog/dalvik-dev\' into dalvik-dev-to-master

Merge commit 'e6899ef57ca0bea419992ab9dd83b8ca65854284' into dalvik-dev

* commit 'e6899ef57ca0bea419992ab9dd83b8ca65854284':

Fix a crash during VM shutdown.

The code to shutdown the GC daemon thread was not checking to see if
the GC daemon thread had been initialized.  This caused pthread_join
to crash waiting for an uninitialized thread object.

Change-Id: Iac338a054775aa024d74fbb4a5de35e12d95b862

Merge "Allow the VM to suspend while the heap worker initializes." into dalvik-dev

Update the atomics test for the new atomics API.

Change-Id: I2b675b4a8e4634304a7ef9b938657fccd740bd79

Merge remote branch 'goog/dalvik-dev' into dalvik-dev-to-master

Change-Id: I990ca89cfd36271813e5af4bcad479290bee2502

Use the card marks to find gray objects during a concurrent collection.

Presently, the garbage collector scans the mark bits looking for gray
objects.  As of this change, only objects spanning dirty cards will be
reexamined during re-marking.

As part of this change, re-marking of roots will push objects onto the
mark stack instead of setting their mark bits.  The number of gray
roots discovered during re-marking is small.  If this changes we can
dirty the cards instead and let re-scanning push the gray objects.

Change-Id: If270812821e070d09af344edb63dfede26d10410

Merge "Verify jmethodID when calling methods." into dalvik-dev

Remove an obsolete member reference.

Change-Id: I97394489304d86a840b4716b3e5d2f1e26745cfa

Merge "Separately report thread suspension and garbage collection times." into dalvik-dev

Separately report thread suspension and garbage collection times.

The GC elapsed time written to the log used to be the sum of the time
spent suspending threads and the time spent executing the rest of the
garbage collector code.  With this change we account for each of these
phases separately.  With the concurrent GC, thread suspension can
dominate the GC pause times so breaking this number out gives a clear
picture of the efficiency of the underlying GC algorithm.

Also, accounting for the concurrent garbage collector is more
accurate.  The start and end times did not subtract the amount of time
spent during the concurrent mark.  The clock is now stopped and
restarted during the concurrent phase of the collector.

Change-Id: Icf3f8c0e183166a62ca1bf32fc77f17e3b5aa963

Verify jmethodID when calling methods.

The JNI Call<type>Method, CallNonvirtual<type>Method, and
CallStatic<type>Method functions take an object (or class) and a
jmethodID as arguments.  For virtual calls, if the jmethodID is not
for a method in the provided object, the VM can crash.  (The most
common way to accomplish this is to pass a class object into a virtual
method call.)

For static method calls, the method defines the class, and we ignore
the jclass argument.  Still, we don't want badly-formed code floating
around, so we just log a warning.

This is part of CheckJNI.

Bug 2577881.

Change-Id: Ia19f0cb604b06a6bd4da8e3d6238d4e162b1480e

Merge commit '1daf86bdb630efa96147220019e1a97c853ed3d2' into mm

Merge "Remove some partial gc code." into dalvik-dev

Allow the VM to suspend while the heap worker initializes.

The concurrent collector may initiate a collection, holding the heap
worker lock, while the heap worker thread starts up.  This causes the
thread suspension to wait indefinitely for the heap worker thread and
forces a VM abort.  The following stack trace is logged:

  #00 __futex_syscall3
  #01 pthread_mutex_lock<-_normal_lock
  #02 heapWorkerThreadStart<-dvmLockMutex
  #03 internalThreadStart
  #04 __thread_entry
  #05 pthread_create

This change puts the heap worker thread in to the VM wait state while
it acquires its locks during startup thereby avoiding the crash.  The
first dvmLockMutex substituted seems to have quelled the crashes at
startup.  The second may be unnecessary but seems benign.

Change-Id: I208b559bcd01de6628f1b79ed6eef17f1536cbbc

Inflate excessively reacquired thin locks.

Change-Id: I78e436bb57722e23730b3c6b6f72705276bde44a

Remove some partial gc code.

This code conflicts with the concurrent collector.

Change-Id: I640d68943b72a967ff39989a445be87ce5ee781c

Merge "Drive the root verification by a root visitor." into dalvik-dev

Drive the root verification by a root visitor.

The change separates out the traversal of the roots from the
verification of references.  As of this change, the logic benhind
traversing is wholly owned by the visitor.

Change-Id: Idc07755436f5cd18ba010ec24d12b6e22000c56e

Merge "x86 interpreter write barrier support" into dalvik-dev

Merge "New string intern table without immortal bits." into dalvik-dev

New string intern table without immortal bits.

The old implementation of the intern table stole the lsb to denote
that a string was "immortal" meaning that it was loaded from a dex
file.  Before clients can use strings from this table they must be
aware of the bit and clear it before chasing the pointer.

The new intern table separates strings that are interned by a call to
String.intern and strings that are interned by the class loader.  This
adds a small cost to each intern call.  Fortunately, there is a
payback in the garbage collector as we can scan just the literal
strings during root marking and scan just the user interned strings
before sweeping to clear them.  Also we no longer have to special case
walking through reference-containing locations in the intern table.

Change-Id: I1192fdcc99e1bb2c606f74f54b3056ec60b6f39b

x86 interpreter write barrier support

Fix for 2837703.  Also added support for new Dalvik volatile ops.

Change-Id: Ic2b8d31afc8806077c5bf27ec90c643f7e47591e

Merge "JIT: Support for Dalvik volatiles (issue 2781881)" into dalvik-dev

JIT: Support for Dalvik volatiles (issue 2781881)

Also, on SMP systems generate memory barriers.

Change-Id: If64f7c98a8de426930b8f36ac77913e53b7b2d7a

In sweepBitmapCallback, index into ptrs, rather than increment.

The arm seems to only have data changes from this change, so should
not change performance.

Change-Id: I32e40ccc2724fccb5996cbafb9f3872785544ae3

Merge "Fixup a failing assert." into dalvik-dev

Fixup a failing assert.

YET AGAIN bit by max being inclusive. Only failed occasionally, and
failures quickly restart and roll off the log. I wasn't watching
closely enough when I tested, I expect.

Change-Id: I2c8e9c0b49ffd9b575e8fa4c882bdae9c64e8304

Merge "Be less misleading when something goes wrong early." into dalvik-dev

Be less misleading when something goes wrong early.

In the recent icu4c breakage, we misinterpreted the "thrown during ...<clinit>"
to mean that <clinit> was the method that threw. This patch makes the wording
less easy to misinterpret.

The other thing that was confusing was the lack of a stack trace. It turns out
that dalvikvm tried to output a stack trace when the thread died, but it was
using LOGI. We'd have seen that on the device (which logs everything), but it
was filtered out by the host build's liblog. (Our ANDROID_LOG_TAGS was set to
"*:w".)

Change-Id: Ib46b3c7addc75aed103a3eb9fa010234f4444fbe

Merge "When aliasing a bitmap, use smallest available limit." into dalvik-dev

When aliasing a bitmap, use smallest available limit.

dvmHeapSourceGetObjectBitmaps was using each Heap's allocation limit
to set the max for the bitmap. This causes the HeapBitmap iterators to
do unnecessary work, over parts of the HeapBitmap which could be known
to be all zeros.

Making the max for a HeapBitmap also take into account the liveBits's
max will inform the HeapBitmap iterators that these regions are all
zeros. This will only change the calculation for the active allocation
Heap; but when there are two Heaps, the Zygote Heap is expected to be
pretty well packed, with not a lot of extra words of zeroes in the
HeapBitmap.

Also fixes a defect in aliasBitmap's calculation of bitsLen.
See http://b/issue?id=2857152

Change-Id: Iacb6bc400318702d760a774c6ca5eab67b8bdfd3

Merge "Port the improved 087 test back to test 086." into dalvik-dev

Merge "Add class checks in AllocObject / NewObject." into dalvik-dev

Merge "Remove #ifdef 0 code in removeCollectedObject." into dalvik-dev

Add class checks in AllocObject / NewObject.

The JNI spec says the functions throw InstantiationException if you
pass in an abstract or interface class, and that you "must not" try to
allocate an array class with this mechanism.  These are now checked.
We also forbid creation of a Class object with these calls, since
that should only be done by DefineClass.

Also, added appropriate "is class initialized" code for the A/V
variants of NewObject, which were strangely lacking them.

Change-Id: I8dde7a812265cfb16b1765695c652e98c9de0840

Remove #ifdef 0 code in removeCollectedObject.

Change-Id: I095b5bdccde621da88ac9118123d82583b0230b6

Port the improved 087 test back to test 086.

Change-Id: Id4dffe23fb60c4e703a0c4de0fab1a0876b2756a

Merge "Add a regression test. dvmFreeClassInnards had a defect where it would SIGSEGV when a linking error tried to clean up, and then GC also tried to clean up when freeing a ClassObject." into dalvik-dev