Merge "Fix dex2dex logic"

Fix dex2dex logic

There was a bug in the dex2dex logic where would only dex2dex if
IsCandidateForCompilation was true or method verifier had check casts.

The bug was caused by not adding verification results, which caused
verification_results_->GetVerifiedMethod to not have verified methods
in most cases for interpreter-only and verify-none compilation.

Caffeinemark interpret-only before:
Sieve score = 1184
Loop score = 630
Logic score = 3519
String score = 1532
Float score = 882
Method score = 686
Overall score = 1159

After:
Sieve score = 2820
Loop score = 1942
Logic score = 4694
String score = 1679
Float score = 2288
Method score = 1294
Overall score = 2244

Bug: 17950037

Change-Id: I535ab8784b434b0a8e3f66e3ab4936d10a1b04bf

Merge "Test uses libart.so, so can't be run NDEBUG."

Test uses libart.so, so can't be run NDEBUG.

Change-Id: I22c7391e07a9d06a8ce9a4d23a21ba9f60bcfe64

Merge "Fix tests now that the SSA builder is pruning more phis."

Fix tests now that the SSA builder is pruning more phis.

Change-Id: I503400e7343322e31ad393d51367b404d6ed7bd8

Merge "ART: Relax test 703"

Merge "Make the SSA builder honor the debuggable flag."

Merge "Add way to print JIT related info at shutdown"

Merge "Use quick compiler to generate odex for oat file assistant tests."

Use quick compiler to generate odex for oat file assistant tests.

This is needed to generate non-PIC odex files, because optimizing
compiler always generates PIC.

Change-Id: Ic77a0faef6b36f13c61607327640e3378a7ee75e

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

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

This reverts commit f45599d8f92748c5c2d2aec515948932c52444eb.

Change-Id: I33c6c5bf1d2643bdf8ce13708f376d01599c2a98

Make the SSA builder honor the debuggable flag.

This requires to properly type phis that are only
used by environments, and discard phis with incomptable types.
The code generators do not handle these conflicting types. In
the process, ensure a phi has a type that does not depend
on the order of the inputs (for example (char, short) -> short),
and set int for int-like types. We can refine this later.

Change-Id: I60ab601d6d00b1cbf18623ee4ff1795aa28f84a1

ART: Relax test 703

NaNs are a set of numbers. Do not expect one specific one as the
result of a NaN computation.

Change-Id: Ib98d61ea1fe8c2dd80027d1a23b8a77bfb248a27

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"

Add way to print JIT related info at shutdown

Added a runtime option called -XX:DumpJITInfoOnShutdown with prints
various interesting JIT statistics during the shutdown of the runtime
if enabled.

Example of running EvaluateAndApplyChanges with the option:
I art     : Code cache size=427KB data cache size=84KB num methods=1598
I art     : Start Dumping histograms for 1598 iterations for JIT timings
I art     : Compiling: Sum: 727.850ms 99% C.I. 135us-4402us Avg: 455.475us Max: 15567us
I art     : Verifying: Sum: 191.074ms 99% C.I. 19us-1450.500us Avg: 119.570us Max: 5142us
I art     : TrimMaps:  Sum: 62.053ms 99% C.I. 13us-199.374us Avg: 38.831us Max: 10058us
I art     : MakeExecutable:  Sum: 11.153ms 99% C.I. 4us-47us Avg: 6.979us Max: 47us
I art     : Initializing:  Sum: 7.991ms 99% C.I. 1us-49.781us Avg: 5us Max: 4852us
I art     : Done Dumping histograms

Bug: 17950037
Change-Id: I23538b24f947e5f6934f55579b8baa9d806eb3d5

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>