ART: Refactor unstarted runtime

Refactor and clean up unstarted runtime.

Bug: 19542228
Change-Id: Ib3e4b3517e06e8242d4fed32ca59419fef553a47

Merge "[MIPS64] Fix GenInlinedPeek and GenInlinedPoke functions."

[MIPS64] Fix GenInlinedPeek and GenInlinedPoke functions.

Change-Id: I4994bb2f9075cb3b6488e5a5b88dbed17b80595a

Merge "Turn on logging for oat file assistant tests."

Turn on logging for oat file assistant tests.

Temporary to debug test failure.

Change-Id: Idbfcdbdae5e0aa3544b01f1a1ab0297a81d736d5

Merge "Remove ThrowLocation."

Remove ThrowLocation.

Note that this is a cleanup change, and has no functionality change.
The ThrowLocation had no use anymore.

Change-Id: I3d2126af1dc673cec3a0453ff3d56a172663a5f6

Merge "Create OatFileAssistant class for assisting with oat files."

Merge "JDWP: allocate DebugInvokeReq only when requested"

Merge "Trim arenas for JIT"

Create OatFileAssistant class for assisting with oat files.

The oat file assistant is used for determining whether dex2oat or
patchoat is needed, for running dex2oat or patchoat as needed to make
an oat file up to date, and to load dex files associated with a given
dex location.

The introduction of the OatFileAssistant class is meant to clean up and
consolidate code related to the management of oat files that was
duplicated and spread across dalvik_system_DexFile.cc and
class_linker.cc.

Bug: 11301553
Change-Id: I0c16027b9bae4570c2c50faa9c14f581c0cbafb8

Trim arenas for JIT

Moved arena pool into the runtime. Added arena trimming to arena
pool. When called, this madvises the used memory.

Changed the JIT compiler to trim arenas after compilation. Changed
the arena mmap name to dalvik-LinearAlloc.

Native PSS before:
  80353 kB: Native
  80775 kB: Native
  78116 kB: Native
After:
  73357 kB: Native
  70181 kB: Native
  70306 kB: Native

Bug: 17950037
Bug: 17643507
Bug: 19264997

Change-Id: I63e7a898fd6e909c2c677fa57b5917a7b1398930

Merge "ART: Fix init-failure graph tool"

ART: Fix init-failure graph tool

Fix missing entries when the failed class doesn't appear in the
stack trace.

Add coloring for loadLibrary and getProperty.

Change-Id: I35df917ac8b1ada1f6eb2e418fa6ff9ca6b17048

Merge "Use the current Java stack instead of the ThrowLocation."

Use the current Java stack instead of the ThrowLocation.

The ThrowLocation contains information we can actually retrieve
from the current stack.

Change-Id: I7e5b2f519f95830c457cb31ea10a581aa9c973ee

JDWP: allocate DebugInvokeReq only when requested

Only allocates thread-local DebugInvokeReq when the debugger requests
a thread to invoke a method. The JDWP thread allocates that structure
then attaches it to the target thread. When the thread is resumed, it
executes the method. Once the invocation completes, the thread
detaches the DebugInvokeReq, signals the JDWP thread then suspends.
Finally, the JDWP thread wakes up, prepares the reply with the invoke
result (or exception) and deallocates the DebugInvokeReq.

Also ensures GC safety for object returned by the invoke. We add the
object to the JDWP object registry right after the invoke. We now
reference that object with a JDWP ObjectID instead of an Object* in
the DebugInvokeReq struct. This prevent from accessing a stale
reference if the GC runs and moves the Object*.

This CL includes the following changes:
- Move former DebugInvokeReq::ready flag to
  Thread::tls_32bit_sized_values::ready_for_debug_invoke. It's needed
  to know whether a thread has been suspended by an event, thus ready
  to invoke a method from the debugger.
- Remove DebugInvokeReq::invoke_needed: we now test if we attached a
  DebugInvokeReq* to the thread.
- Rename misleading FinishMethod function to RequestMethod.

Bug: 19142632
Bug: 18166750
Change-Id: I351fb4eb94bfe69fcafb544d21d55ff35a033000

Merge "Fix run-test with JIT enabled"

Fix run-test with JIT enabled

Uses -Xusejit:true instead of -Xjit.

Bug: 17950037
Change-Id: I8280245e3e2aeced5927ade1632b7a110b040136

Merge "Revert "Revert "More Makefile fun for --debuggable."""

Revert "Revert "More Makefile fun for --debuggable.""

This reverts commit 9728a930fdf717cca60b48d9c2b715d0eed497fd.

Change-Id: Ieca964a793e58f11931f78f50be47a4a9f0dc21f

Merge "Revert "More Makefile fun for --debuggable.""

Merge "Revert "Fix braino in Makefile.""

Revert "Fix braino in Makefile."

This reverts commit 64657d2d840e5d4e6b4b71d5df6ee0fcc4a8afab.

Change-Id: I5b778027538a4247cf51eff338220328e741074f

Revert "More Makefile fun for --debuggable."

This reverts commit bd648cd3aef1ce82aaaa85924b1178a7f499cc29.

Change-Id: I9be508730dc5c25b9d08f914c5628e88bc4aa15b

Merge "Fix braino in Makefile."

Fix braino in Makefile.

They are such a joy to work with.

Change-Id: I1a982c97fb9e2e9f34f921e7f6b481988a57f858

Merge "Compute the right catch location for the debugger."

Compute the right catch location for the debugger.

Also remove tls ThrowLocation, it is not needed anymore.

Change-Id: I78fddf09ce968ca475e39c17fa76d699c589c8d9

Merge "More Makefile fun for --debuggable."

Merge "ART: Allow arraycopy with int[] in unstarted runtime"

ART: Allow arraycopy with int[] in unstarted runtime

Overlooked in the condition, the actual code was already there.

Bug: 19542228
Change-Id: I30caf77b345cb9bdb1f54c28bf4d42852c9a3298

Merge "Only allocate collectors if we may use them"

Only allocate collectors if we may use them

I believe most of the savings come from not allocating the concurrent
copying 8MB MarkQueue.

Before AOSP hh native PSS:
    72653 kB: Native
    72998 kB: Native
    72882 kB: Native

After AOSP hh native PSS:
    64823 kB: Native
    65986 kB: Native
    64219 kB: Native

Bug: 17643507
Change-Id: Ic5a8e753beca36142c1bf36be6311051f7c78e47

Merge "Initial support for quick compiler on MIPS64r6."

Merge "Fix failing hprof DCHECK"

Initial support for quick compiler on MIPS64r6.

Change-Id: I6f43027b84e4a98ea320cddb972d9cf39bf7c4f8

Fix failing hprof DCHECK

Benign.

Bug: 19627999
Change-Id: I20f3d36ffa1efaf23b40b6290c042df81eca2fea

Merge "Fix byte and char Get32"

Fix byte and char Get32

8 bit and 16 bit fields aren't necessarily 32 bit aligned due to
field packing. This was causing SIGBUS for hprof dumping some apps.

Bug: 19627999

(cherry picked from commit 2ff8da6a4d44bcf6a72dca1b5294a72297c5ce26)

Change-Id: If549412fc4d62c55bb6a46f11658fc60b822a27e

Merge "Do not dump memory usage if driver was never created"

More Makefile fun for --debuggable.

Change-Id: Icd4a4cf53907bf04b0d45211e264c9985045b55c

Do not dump memory usage if driver was never created

Bug: https://code.google.com/p/android/issues/detail?id=158593
Change-Id: I8084f9ac823910aa11f854f3e764240741fbf0d5

Merge "Revert "[optimizing] Enable x86 long support.""

Revert "[optimizing] Enable x86 long support."

Few libcore failures.

This reverts commit b4ba354cf8d22b261205494875cc014f18587b50.

Change-Id: I4a28d853e730dff9b69aec9555505803cf2fcd63

Merge "Opt compiler: Clean the use of `virtual` and `OVERRIDE`."

Opt compiler: Clean the use of `virtual` and `OVERRIDE`.

Change-Id: I806ec522b979334cee8f344fc95e8660c019160a

Merge "Extend compiler/optimizing/stack_map_test.cc."

Merge "[optimizing] Enable x86 long support."

[optimizing] Enable x86 long support.

Change-Id: I9006972a65a1f191c45691104a960366747f9d16

Merge "ART: Enable the use of relative addresses in the arm64 disassembler."

ART: Enable the use of relative addresses in the arm64 disassembler.

Also, only keep register aliases for the link register 'lr' and the
thread register 'tr' in the arm64 disassembler. Other aliases are not
very important, and this way we don't have to provide aliases
specialised for Quick or Optimizing.

Change-Id: Ie7a04910f0c587710a0cf2648203d7e89eab5d1f

Merge "Follow up 137982"

Merge "ART: Add option to abort dex2oat on hard failure"

Merge "ART: Fix Cmdline Parser test"

ART: Fix Cmdline Parser test

Follow-up to https://android-review.googlesource.com/138593.

Bug: 17950037
Change-Id: Ie8b796262989bf8374f5cecff15a1f2dad2cb3c1

ART: Add option to abort dex2oat on hard failure

Add an option that aborts dex2oat when a hard verifier failure occurs.

Bug: 19606409
Change-Id: I53195284e22fe6207274101e85745af763c06271

Merge "Rename -Xjit / -Xnojit options to -Xusejit:<boolean>"

Rename -Xjit / -Xnojit options to -Xusejit:<boolean>

Bug: 17950037
Change-Id: I9182820240c8bbc2fa4767f6361ab3b7f1083a31

Merge "Fix some incorrect IsCompiler instead of IsAotCompiler"

Fix some incorrect IsCompiler instead of IsAotCompiler

Fixes jdwp related issues (DDMS) when JIT is enabled.

Bug: 19623297
Change-Id: I36139c70a97b529135febcf01b227e7ab9affacc

Merge "ART: AddVectorReduce should store result in memory carefully"

Follow up 137982

Updates comment explaining why we sort types by name in run-test
031-class-attributes.

Bug: 19499481
Change-Id: I1ee425a24dffd55d134effc6857e2c0ca368422a

Merge "ART: Fix test breakage"

ART: Fix test breakage

Recent commit changed the direction of iteration over safepoints in
the register allocator but contained a bug that skipped some of them
at the boundaries of interval siblings. This patch fixes the bug.

Change-Id: Ia7d4892536b5198e01c9bc3034f448227794ff72

Extend compiler/optimizing/stack_map_test.cc.

Change-Id: I6f8c3186683210a5162cbf4f15361a4c494495c9

Merge "ART: Optimize iteration of safepoints"

ART: Optimize iteration of safepoints

The LiveInterval::Covers method is optimized for multiple calls with
non-decreasing positions. This patch reverts the order of iteration
over safepoints in RegisterAllocator::ConnectSiblings to capitalize
on this effect.

Change-Id: Ieb70eb9d5c0a06ee79379aab6c87cb3290c15bf7

Merge "[optimizing] Use callee-save registers for x86"

[optimizing] Use callee-save registers for x86

Add ESI, EDI, EBP to available registers for non-baseline mode. Ensure
that they aren't used when byte addressible registers are needed.

Change-Id: Ie7130d4084c2ae9cfcd1e47c26eb3e5dcac1ebd6
Signed-off-by: Mark Mendell <mark.p.mendell@intel.com>

Merge "ART: Fix compile-time-init fail"

ART: Fix compile-time-init fail

It is possible that a class that isn't itself marked as an image
class and would be skipped for compile-time-initialization by the
compiler driver is initialized when initializing a second class
that is an image class.

If the second class only depended on the first class during
initialization, e.g., to read field values, the compiler driver
may discard the class object (not reachable from image classes),
but record the oat class status as initialized (as it has been).

In that case we miss the initialization when we load the class at
runtime.

Ensure that all initialized classes with a class initializer are
considered image classes and retained.

Bug: 19323020
Change-Id: I4e537f328d9a4ea23ed5ff7166d532b8855f7acd

Merge "ART: Report zip opening error"

ART: Report zip opening error

Do not swallow the error message from ZipArchive::OpenFromFd.

Bug: 19574093
Change-Id: I14ea771e63b7bc3e8051012a841e66b9e894aa3a

Merge "Include what we use."

Include what we use.

Newest libc++ update fixes some of the header cruft.

Change-Id: Ic5019ffcb272274ac3b21627218d65a887074458

Merge "ART: Do not produce CFI when not asked for"

ART: Do not produce CFI when not asked for

Insignificant time savings on the host, but also reduces native
allocation size.

Change-Id: Iea3d335e5375a0076306059d094e5b994e24b9e6

Merge "[optimizing] Improve x86/x86_64 bound check code"

Merge "ART: Add debuggable compiler flag"

Merge "ART: Fix missing handles"

Merge "Fix bce gtest."

ART: Add debuggable compiler flag

Add a flag to compiler options that shows debuggability.

Change-Id: Id17ec72babe2ee88713a0d274eff86508de30666

ART: Fix missing handles

Follow-up to https://android-review.googlesource.com/137010.

Change-Id: Ie97bd01f3cd6eeef9ae38fd189b933b905832d52

[optimizing] Improve x86/x86_64 bound check code

Don't force a constant index into a register just to compare to the
array size.  Allow a constant, and compare the constant to the size.

Change-Id: I1c5732fbd42e63f7eac5c6219a19e9c431c22664
Signed-off-by: Mark Mendell <mark.p.mendell@intel.com>

Fix bce gtest.

Change-Id: I8b974b68bbebef704fa3c8a073f2920f8a78d1fa

Merge "Make BCE a no-op if there is no array access."

Merge "Update run-test 031-class-attributes"

Update run-test 031-class-attributes

Ensure types are printed in alphabetical order.

Bug: 19499481
Change-Id: Ia14d2529f8031bc4c2bfb8911fdebebf97638dc9

Merge "Change how we report exceptions to the debugger."

Merge "Fix arm64 build."

Fix arm64 build.

Change-Id: Ib6babc1c6e8f2e78badc93cfcf89950e53f71bbb

Merge "Switch from memalign to posix_memalign for Mac build"

Switch from memalign to posix_memalign for Mac build

Change-Id: Id320015ea3999605954f53b5e266f53e968c7065

ART: AddVectorReduce should store result in memory carefully

When generating AddVectorReduce, in some cirsumstances we
add value reduced from vector directly to memory. We must
ensure that local LIR optimizations are aware about it.

Change-Id: I8fe19939f67dcd184b08f63026b0da18007d34b8

Make BCE a no-op if there is no array access.

Change-Id: I8456182808c1dbaa0c0ae1b8c2e94bb17baf5f29

Merge "Reserve bits in the lock word for read barriers."

Reserve bits in the lock word for read barriers.

This prepares for the CC collector to use the standard object header
model by storing the read barrier state in the lock word.

Bug: 19355854
Bug: 12687968
Change-Id: Ia7585662dd2cebf0479a3e74f734afe5059fb70f

Merge "Wait for threads to finish unregistering"

Wait for threads to finish unregistering

There was a race where Thread::join would return before the thread was
unregistered. This caused a problem with Daemons.stop since the thread
list could get deleted before the daemon thread was removed from list_.
This caused occasional "Request to unregister unattached thread"
errors and warnings.

The fix is to wait until threads finish registering before destroying
the thread list. The only threads which can be unregistering at this
point are the daemons we stopped earlier during the runtime shutdown
process. The issue is that thread join finishes before we remove the
thread from the thread list.

Also some cleanup.

Bug: 18713034
Change-Id: I8921122fe8462643a6b814b5f00632481e3831fb