Compile libdex as C++.

The major-looking code changes were all just to unravel some gotos.

Change-Id: I86f98a48b160f357ce93c87446bad5d705d5f05b

Add a class flag CLASS_ISCLASS.

This flag is only turned on for the unique class Class, and it is
meant to make it possible to quickly test objects for "classiness" as
well as trivially figure out if one is looking at the class Class
itself.

Bonus: Made a few places that directly set class->accessFlags use
the prescribed macro for doing same.

Change-Id: I89fce708c407b597f28ce610b778123f5dcaecf6

Merge "Move the reflection code into C++." into dalvik-dev

am e7147462: Fix debugger performance regression

* commit 'e71474621b968a9bc649d067155d31213f5764a9':
  Fix debugger performance regression

Move the reflection code into C++.

Smart pointers would make dvmGenerateProxyClass cleaner.  This code should
be revisited once those interfaces come into existence.

Change-Id: If25abb87e22d658ad88f997324c35703b23f623c

Fix debugger performance regression

http://b/issue?id=4282910

The interpreter was reporting every instruction as a method entry point,
causing significant performance degradation in the debugger.

Change-Id: If6fb092d6ade7fc3d74be3c7f806bf2d6923a063

Convert the internal and in-line natives to C++.

Change-Id: I2ece682bc3b4d3b55ab27c60fd84a0b3243d7ca6

Merge "Add additional asserts and combine declarations with definitions." into dalvik-dev

Add additional asserts and combine declarations with definitions.

Change-Id: Ibdc37f3d7eb6385da1059d5525727b090626ec4c

am 0161e158: Merge "Add missing JUMBO opcodes to volatile rewriting"

* commit '0161e1582b13173cc5a125cf504fd7714c9c82b2':
  Add missing JUMBO opcodes to volatile rewriting

Merge "Add missing JUMBO opcodes to volatile rewriting"

am 5867bea1: [JIT] Close code cache race window

* commit '5867bea193779ff3009738513bae9bc84f4a34aa':
  [JIT] Close code cache race window

Add missing JUMBO opcodes to volatile rewriting

The boolean/byte/char/short variants of iget/iput/sget/sput were not
being correctly converted to _VOLATILE opcodes for SMP builds.

Change-Id: Ic81d75f9c927325d54a98465c146b6d43ea204f4

[JIT] Close code cache race window

See http://b/issue?id=4271784 for details.

Three fixes:
   1.  Verify the code cache version hasn't changed between completion
       of the translation and registering it in JitTable
   2.  When code cache full detected during translating a trace, mark
       the "discard" flag on the work order.
   3.  [The actual cause of the bug] When doing a code cache flush,
       traverse the thread least and cancel any trace selections in
       progress.

Change-Id: Ifea70416d7d91637fb742fc8de11044a89358caa

Dx block dumper no longer fails on abstract and native methods.

Change-Id: I3bad07a1621c0ef4bc495816b2438cab9401a828

am 8d6291cc: Merge "[JIT] Fix volatile test"

* commit '8d6291cce6ff3c9a3010f3725563983847370df7':
  [JIT] Fix volatile test

Merge "[JIT] Fix volatile test"

[JIT] Fix volatile test

On SMP systems, Dalvik opcodes referencing volatile fields will be
rewritten to their _VOLATILE variant.  On non-SMP systems, though,
this rewriting is not done.  The JIT, however, needs to know about
volatility for all systems in order to avoid performing unsafe
optimizations.  This change fixes the JIT's volatility test to be
either _VOLATILE opcode or the volatile flag in the field access bits
depending on SMP type.

Change-Id: I2edde58dc25f22cba88f62c5f1a2d125473309e6

am 2336a395: Merge "Add jniThrowExceptionFmt."

* commit '2336a3953305b2fe3e59b850d2e5804c86e818dd':
  Add jniThrowExceptionFmt.

Merge "Add jniThrowExceptionFmt."

Add jniThrowExceptionFmt.

I still think you should be doing checks and throwing exceptions in Java,
and your native code should just crash if you lie to it, but this will
ease the translation of frameworks/base code over to using JNI more correctly.

Change-Id: I9f2512e2349452b82360b375911c814ab00db23b

Merge "Fix for loops and eliminate some other undesirable idioms." into dalvik-dev

Fix for loops and eliminate some other undesirable idioms.

Previously, we were using a non-standard C dialect which did not permit
the nicer variable scoping in loops.  Now that we are using C++ in the
collector we can guarantee that feature is available.  This change also
rearranges some of the surrounding code to take advantage of the more
flexible scoping rules.

Change-Id: I9be35794cc12bcbc0d5299fe387d4bc406481075

am f1b53179: Merge "Fix method profiling [issue 4260697]"

* commit 'f1b531793e65fda22818d8dca8b17f94f618ad64':
  Fix method profiling [issue 4260697]

Merge "Fix method profiling [issue 4260697]"

am 80d118ae: Revert "[JIT] Fix volatile test" - breaks build

* commit '80d118aefbba31b7658299bf19f5ca72473390cd':
  Revert "[JIT] Fix volatile test" - breaks build

Merge "Updated dx to emit rsub instructions whenever it can." into dalvik-dev

Revert "[JIT] Fix volatile test" - breaks build

This reverts commit 11fb99d598ebe640719743a0d3bd7ed091e5be03.

Fix method profiling [issue 4260697]

In the new interpreter control mechanism, method tracing invoke
events are generated at the call site, whereas debug entry events
happen prior to the interpretation of the first instruction of a
method.  Because we were failing to check for active profiling
on entry to a new interpreter instance, we would miss the profile
event for the first method interpreted.  We subsequently (and correctly)
recorded the return event, and ended up with a mismatch causing traceview
to complain and die.

Change-Id: Ibd93dfcd16c2f1fc62f6a86626a902eaf5d4b160

Updated dx to emit rsub instructions whenever it can.

Change-Id: I148c03fdc19dde8aece448e288d3c963490c9d0f

am 11fb99d5: [JIT] Fix volatile test

* commit '11fb99d598ebe640719743a0d3bd7ed091e5be03':
  [JIT] Fix volatile test

[JIT] Fix volatile test

On SMP systems, Dalvik opcodes referencing volatile fields will be
rewritten to their _VOLATILE variant.  On non-SMP systems, though,
this rewriting is not done.  The JIT, however, needs to know about
volatility for all systems in order to avoid performing unsafe
optimizations.  This change fixes the JIT's volatility test to be
either _VOLATILE opcode or the volatile flag in the field access bits
depending on SMP type.

Change-Id: If485875c5abbf0147c3ef4b6d557faa89ed85426

am 9688de1e: Bug fix for rsubs being improperly generated.

* commit '9688de1ebc68f4f865a827709298390a72dff698':
  Bug fix for rsubs being improperly generated.

Bug fix for rsubs being improperly generated.

This will miss some opportunities to generate rsubs, and has been marked
by a TODO note where code needs to be added.

Cherry-picked frmo dalvik-dev.

Change-Id: Iac272f16b78ab6d985b5aabef6bd360d36a5c7ce

Merge "Bug fix for rsubs being improperly generated." into dalvik-dev

Compile the garbage collector and heap profiler as C++.

Change-Id: I25d8fa821987a3dd6d7109d07fd42dbf2fe0e589

Bug fix for rsubs being improperly generated.

This will miss some opportunities to generate rsubs, and has been marked
by a TODO note where code needs to be added.

Change-Id: I6dd2c7a5bb82294ee5f88ba41191e473133ed847

am 2f5e3cb2: Merge "Scale the card count by bits per byte to derive a bit vector length."

* commit '2f5e3cb20ed3fa00e474fb4813f2c1f46d04afd4':
  Scale the card count by bits per byte to derive a bit vector length.

Merge "Scale the card count by bits per byte to derive a bit vector length."

Scale the card count by bits per byte to derive a bit vector length.

Change-Id: If3177d924193cd665f12758ce83f3be4cc5a66c8

am feeceefb: Merge "Improve JNI indirect ref diagnostics"

* commit 'feeceefbbe44a53e3d02812a126f93d3ef8ec37b':
  Improve JNI indirect ref diagnostics

Merge "Improve JNI indirect ref diagnostics"

Improve JNI indirect ref diagnostics

Improved CheckJNI problem detection and failure reporting.  Also,
the indirect reference table data structure now responds to invalid
requests with a magic bogus object instead of NULL, so that failures
are immediate and obvious.

The extended checks in the indirect ref implementation are still on,
mostly so that bad code fails in an obvious way even when CheckJNI
is not enabled.

This change also includes a hack to allow weak globals to work.
Returning non-NULL for invalid refs turned a rarely-fails-weirdly
issue into an always-fails-loudly, which is good, but we need to
paper over it until the WGR fix is in place.  This will need to be
removed later.

Also, reduced some log levels.

Bug 4184721

Change-Id: Iaec75c71dbc85934366be2e717329b635d0fa49e

am afe002d1: Update the interpretation of --target-api.

* commit 'afe002d1474d7c2ef2ac80de32f21f1949af2a41':
  Update the interpretation of --target-api.

Update the interpretation of --target-api.

Change-Id: I4ab158384d09470d84097d944ae1ddadeb2ad045

Merge "Fix some stale docs references." into dalvik-dev

Fix some stale docs references.

Change-Id: I3b58dfd05a63671cf315698247d88e67528e118d

am a4a19037: Fix x86 fast interpreter

* commit 'a4a19037f75e0e3c85fb5f3c7fa31732f8475719':
  Fix x86 fast interpreter

Fix x86 fast interpreter

Was passing wrong value to dvmResolveField() in OP_IGET_WIDE_JUMBO

Change-Id: I7773fd0932d7d032fce30dc57b33ef6a8fbc7afb

Normalize loop exit condition before checking counted loops

The loop exit condition for a simple one like "for (i=0; i<10; i++)"
can be done as
    - i<10
    - 10>i
    - !(i>=10)
    - !(10<=i)

This change normalizes the loop exit condition as if the exit path
is always on the taken direction of the branch. Doing so enables the
discovery of more counted loops without complicating the detecton logic.

Change-Id: I06f428334d4f1ca00d229fe76c282c6f161e8912

Merge "Stop logging a couple of messages" into dalvik-dev

Stop logging a couple of messages

These used to be uncommon, now they're not.

Change-Id: Ibab0c94b183cf969836277e6bc29b0ef3eba3a86

Merge "Fix JNI GetStaticFieldID" into dalvik-dev

am 8b94be1f: [JIT] Rely on the opcode only to determine volatile

* commit '8b94be1f80b135e7a5c1f542861f3e80c4c52823':
  [JIT] Rely on the opcode only to determine volatile

Fix JNI GetStaticFieldID

GetStaticFieldID wasn't searching into the superclasses.

Bug 4205860

Change-Id: Ide89b61ad8ed42c10c160968f5cce745d9f0eefe

[JIT] Rely on the opcode only to determine volatile

In the past, it was possible to have a volatile field with a
non-volatile opcode.  This is no longer the case, so this change
eliminates the volatile field flag check.

Change-Id: I1cface4e813144634b2f90732c76b0a16f08c304

am 1d41040a: Merge "Do not emit the write barrier when storing to primitive volatile fields."

* commit '1d41040ad8175f89f1c19a4dfbaf6ae47baf7947':
  Do not emit the write barrier when storing to primitive volatile fields.

am c26cd1ae: Merge "Fix sizing of the modunion table on small heaps"

* commit 'c26cd1ae75aa947b452ebe208b1ceaa06f292378':
  Fix sizing of the modunion table on small heaps

Merge "Do not emit the write barrier when storing to primitive volatile fields."

Merge "Fix sizing of the modunion table on small heaps"

am 552cacb8: Include strings.h for the ffs prototype.

* commit '552cacb8aef667fec06196e6d33d2d81177a61fb':
  Include strings.h for the ffs prototype.

Do not emit the write barrier when storing to primitive volatile fields.

Change-Id: I79cdac62baa40582bba160a04cbd4c8b2c9151a5

Fix sizing of the modunion table on small heaps

Change-Id: I8696fb2f6d408049ec39609db34238edee171f79

Include strings.h for the ffs prototype.

Change-Id: I70a4dc5c8d9015cfc069611b7eba0860bb37a8d7

Merge "Size the jni local references table to avoid resizing at startup." into dalvik-dev

Merge "Do not emit the write barrier when storing to primitive volatile fields." into dalvik-dev

Size the jni local references table to avoid resizing at startup.

Change-Id: Ief382b9acb0fa6ce1e0371ecbb7d86f7059102d8

Do not emit the write barrier when storing to primitive volatile fields.

Change-Id: I7700523250d8f63f0b9aa153d0f5f42fdbf3d64f

Use x86 variant in dalvik for TARGET_ARCH_VARIANT=x86-atom

Change-Id: I1f7968a5bf8e24f3a3feb5f5a40ee2c21294a0c3

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

Change the tread state before acquiring the code cache lock.

Fix a deadlock situation.
Bug: 4192964

Change-Id: I27f869d90d58f67e675a65444ebed6fdf2a5f518

Merge "[JIT] Clear inCodeCacheFlag on extended singlestep" into dalvik-dev

[JIT] Clear inCodeCacheFlag on extended singlestep

The inJitCodeCache flag was not being reset if an extended
singlestep prevented a return to an existing translation.

Change-Id: I84f9e4053e3dc713a5ff994df9ecd2ab54e7fe56

Merge "Generate code for loops formed with the new builder" into dalvik-dev

Merge "Delegate the queuing of cleared references to managed code." into dalvik-dev

Generate code for loops formed with the new builder

Adapt the existing counted loop analysis and range/null check
elimination code to work with the new loop building heuristics.
Cleaned up the old ad-hoc loop builder.

Suspend polling is enabled by default for loops. The backward chaining
cell will be used in self-verification and profiling mode.

If the loop includes accesses to resolved fields/classes, abort code
generation for now and revert to the basic acyclic trace. Added
tests/090-loop-formation to make sure the JIT won't choke on such
instructions.

Change-Id: Idbc57df0a745be3b692f68c1acb6d4861c537f75

Merge "Add missing JUMBO ops [fixed]" into dalvik-dev

Add missing JUMBO ops [fixed]

Neglected to git-add the most recent version in Change I0c0b6d17.

Change-Id: I5f9e630f652edcf70ab893ade6f559056ed31f8f

Merge "Improve diagnostics when non-String passed as a jstring." into dalvik-dev

Improve diagnostics when non-String passed as a jstring.

For my particular bug, the cause was obvious by inspection of the calling
code, but this might be useful in harder cases.

Before:
    W(17868) JNI WARNING: jstring 0x3600001 points to non-string object (Check_GetStringUTFChars)  (dalvikvm)

Change-Id: I883a99936b8c2cd79c5a691f7c335245ff50b34f
After:
    W(19680) JNI WARNING: jstring 0x3600001 points to object of type Llibcore/io/Posix; (Check_GetStringUTFChars)  (dalvikvm)

Merge "Add missing JUMBO ops to JIT" into dalvik-dev

Add missing JUMBO ops to JIT

We were missing a dozen or so of the new jumbo ops in the JIT's
code generator.  Interestingly, our compiler error recovery mechanism was
good enough that we didn't notice (we die on an assert build,
but silently recover and continue on a production build).

Change-Id: I0c0b6d1704c47e81b39c7dcf7d1172dbdcd29856

Add missing jumbo opcodes to the self verification exclusion list

Change-Id: If38ce20b2ac30488118259e3b3bdfbc9f068e5c3

Delegate the queuing of cleared references to managed code.

Previously, the garbage collector would pass each cleared reference to
the heap worker thread for queuing.  The heap worker thread would then
perform a callback into managed code for each cleared reference which
assigned the reference to its reference queue.

With this change, the garbage collector instead links together all of
the cleared references and calls back into managed code exactly once
to hand off the references for processing.  This change makes the heap
worker thread and its data structures obsolete.

Change-Id: I28e02638f0877a7fd2ac96b9c3f5597a38541ebb

Merge "Update a test to handle a different anonymous class name." into dalvik-dev

Update a test to handle a different anonymous class name.

Change-Id: Ie39f76b9aa7478c39e10165eba122e3aa2834461

Merge "Fix interpreter debug attach" into dalvik-dev

Fix interpreter debug attach

Fix a few miscellaneous bugs from the interpreter restructuring that were
causing a segfault on debugger attach.

Added a sanity checking routine for debugging.

Fixed a problem in which the JIT's threshold and on/off switch
wouldn't get initialized properly on thread creation.

Renamed dvmCompilerStateRefresh() to dvmCompilerUpdateGlobalState() to
better reflect its function.

Change-Id: I5b8af1ce2175e3c6f53cda19dd8e052a5f355587

am 3a2e2a73: Merge "SamplingProfilerIntegration and SamplingProfiler improvements (1/3)"

* commit '3a2e2a73b687ba68d879e49b465778c4d589cff1':
  SamplingProfilerIntegration and SamplingProfiler improvements (1/3)

Merge "SamplingProfilerIntegration and SamplingProfiler improvements (1/3)"

SamplingProfilerIntegration and SamplingProfiler improvements (1/3)

Summary:

frameworks/base
- fix profiling to collect data beyond the first snapshot
- avoid many small files, accumulate data over process lifetime

libcore:
- add support for VM specific sampling, trying to cut down overhead
- added support for converting snapshot files to text format
- fixed race in profiler when stopping

dalvik
- added VMStack.setThreadStackTrace interface for filling a stack
  trace into an existing StackTraceElement[]

Details:

frameworks/base

    Changed snapshots from text to binary hprof format (bumping version to 3)
    Changed from one file per snapshot to one file per process lifetime.
    Restart profiling after snapshot.

core/java/com/android/internal/os/SamplingProfilerIntegration.java

    Add quick test in maybeSnapshot to avoid doing work when the
    SamplingProfilerIntegration is disabled. Make maybeSnapshot
    private. Remove unneeded memory allocation in handleLowMemory.

core/java/android/app/ActivityThread.java

libcore

    Added ThreadSampler interface. This allows VM specific thread
    sampling optimizations. The portable version continues to use
    Thread.getStackTrace().

dalvik/src/main/java/dalvik/system/profiler/ThreadSampler.java
dalvik/src/main/java/dalvik/system/profiler/PortableThreadSampler.java
dalvik/src/main/java/dalvik/system/profiler/SamplingProfiler.java

    Add VMStack.setThreadStackTrace and use in new DalvikThreadSampler
    to avoid allocating a full stack trace when only a limited depth
    is desired.

dalvik/src/main/java/dalvik/system/profiler/DalvikThreadSampler.java
dalvik/src/main/java/dalvik/system/VMStack.java

    Refactored BinaryHprof.readMagic out of BinaryHprofReader so it
    can be used by HprofBinaryToAscii converter to probing file
    types. Added magic number constant to be shared between readMagic
    and BinaryHprofWriter.

dalvik/src/main/java/dalvik/system/profiler/BinaryHprof.java
dalvik/src/main/java/dalvik/system/profiler/BinaryHprofReader.java
dalvik/src/main/java/dalvik/system/profiler/BinaryHprofWriter.java
dalvik/src/main/java/dalvik/system/profiler/HprofBinaryToAscii.java

    Removed unneeded HprofWriter interface. Changed to simpler static
    interface to write HprofData to binary and text formats.

dalvik/src/main/java/dalvik/system/profiler/HprofWriter.java
dalvik/src/main/java/dalvik/system/profiler/AsciiHprofWriter.java
dalvik/src/main/java/dalvik/system/profiler/BinaryHprofWriter.java
dalvik/src/test/java/dalvik/system/profiler/SamplingProfilerTest.java

    Added support for reading snapshot files created by
    SamplingProfilerIntegration by stripping the text header to allow
    easier conversion to the text format.

dalvik/src/main/java/dalvik/system/profiler/HprofBinaryToAscii.java

    Fixed race between Sampler and
    SamplingProfiler.stop. SamplingProfiler.stop previously simply
    called the Sampler's TimerTask.cancel method, but this does not
    wait for a currently running Sampler to finish. The TimerTask
    documentation says the only reliable way to do this is to have the
    run() cancel itself, so that is what is now done, with new code to
    ensure that SamplingProfiler.stop does not return until the
    Sampler has been terminated.

dalvik/src/main/java/dalvik/system/profiler/SamplingProfiler.java

dalvik

   Refactored VMStack_getThreadStackTrace to create helper getTraceBuf
   used to implement new VMStack_setThreadStackTrace. The new version
   interface fills an existing StackTraceElement[], avoid allocating
   unnecessary StackTraceElements.

vm/native/dalvik_system_VMStack.c

   Refactor dvmGetStackTraceRaw to create dvmSetStackTraceRaw which
   fills in an existing, potentially smaller, StackTraceElement[].

vm/Exception.c
vm/Exception.h

   Change stack depths to be size_t to avoid signed/unsigned comparison warnings.

vm/Ddm.c
vm/Exception.c
vm/Exception.h

Change-Id: Ie88a3f9e069f6984d790ee61c6f7ac5ed47d3388

Merge "Improve diagnostics when RegisterNatives fails." into dalvik-dev

Improve diagnostics when RegisterNatives fails.

This assumes that people spell the method name correctly, but make mistakes
in the signature (which seems to be true in practice).

Before:
    W( 7757) ERROR: Unable to find decl for native Llibcore/io/Posix;.sendfile:(Ljava/io/FileDescriptor;Ljava/io/FileDescriptor;Llibcore/util/MutableInt;J)J  (dalvikvm)
    E( 7757) RegisterNatives failed for 'libcore/io/Posix', aborting  (JNIHelp)

After:
    E(23966) ERROR: couldn't find native method  (dalvikvm)
    E(23966) Requested: Llibcore/io/Posix;.sendfile:(Ljava/io/FileDescriptor;Ljava/io/FileDescriptor;Llibcore/util/MutableLong;J)Z  (dalvikvm)
    E(23966) Candidate: Llibcore/io/Posix;.sendfile:(Ljava/io/FileDescriptor;Ljava/io/FileDescriptor;Llibcore/util/MutableLong;J)J  (dalvikvm)
    E(23966) RegisterNatives failed for 'libcore/io/Posix', aborting  (JNIHelp)

Change-Id: I434aee679bd7c2760404b7790b996f7f86ed9b44

Init classes used by DDMS

Not all call paths ensure class initialization.

Change-Id: Ic12ac5c59f3ffa2d7c04e84ea37264f65a7630be

Merge "Add safepoint callback for gc" into dalvik-dev

Add safepoint callback for gc

This change adds a safepoint callback registration mechanism.
On a per-thread (or all-thread) basis, you pass in a function to be
called at the next safe point by the target thread.  That if that
function returns 0, it will be automatically disarmed.  If not,
the callback will stay in effect and the function will be called
on all subsequent safe points.

GC is the expected customer of this feature.

Change-Id: Icd3b93128b1fd547e142d047a12df7ae8ee646e3

Remove unused security manager code.

Change-Id: Iecc602b8235eb3feeac31e198bdaf61c65dd9fbf
http://b/4087759

Merge "Support reflective lookup of single fields." into dalvik-dev

Merge "Prep DDMS stuff in InitRefs" into dalvik-dev

Prep DDMS stuff in InitRefs

Find some methods and field offsets.

Also removed a duplicate entry.

Change-Id: I4c23af6cfe6e7d4f5e8767c5a9a30d3b441ad425