Add basic image writer support for app images

Needed to handle references from app image -> boot image.

Generate app images for tests to enable some testing.

Bug: 22858531

Change-Id: I1af98b6c4dfcb3a147fb5b0dea64aa4946c7ce57

Merge "Delay emitting CFI PC adjustments until after Thumb2/Mips fixup."

Delay emitting CFI PC adjustments until after Thumb2/Mips fixup.

On Mips also take into account out-of-order CFI data emitted
from EmitBranches().

Change-Id: I03b0b0b4c2b1ea31a02699ef5fa1c55aa42c23c3

Merge "Fix test in case we run the ART_TEST_TRACE tests."

Fix test in case we run the ART_TEST_TRACE tests.

Change-Id: Ic998a29f2d5c573f34def798eb4f2fe5cf30e8a5

Merge "JDWP: fix java.lang.String creation"

Merge "Fix TODO on instrumentation and add some more DCHECKs."

Merge "Fix mips32oc ArraySet null constant assignment"

Fix TODO on instrumentation and add some more DCHECKs.

bug:25343683
bug:25438583

Change-Id: I232deb1b6761466b514c687ce304f61928755cdc

Fix mips32oc ArraySet null constant assignment

Fix assigning a null constant to an array index. Previously this
would unnecessarily invoke pAputObject.

This commit fixes previous test failure: 521-checker-array-set-null.

Change-Id: I02ff5296fa3a48b021c0dfd9bb10628ba6e5d5e5

Merge "lambda: Minor capture-variable/liberate-variable clean-up after post-merge reviews."

Merge "Annotate References with their referent."

Annotate References with their referent.

Also, don't include annotations in an object's link. That way we can
distinguish between the object link and any links in the annotations.

Change-Id: I3fef3f5b2c343699f527cad9efa7c93a4b35a3a4

Merge "Fix broken handling of instrumentation lists."

Merge "Make things generally quieter."

Make things generally quieter.

* Give run-test a --quiet flag that causes it to only print on errors.
* Give cpplint a --quiet flag that causes it to not print anything
  when there are no errors.
* Add a ART_TEST_QUIET flag to build/Android.common_test.mk which
  makes run-test targets much quieter when true (the default). With
  this flag only failures will be printed.
* Make build/Android.cpplint.mk pass the new cpplint --quiet flag so
  that only failures will be printed.

Before:
    [ 96% 5715/5906] build test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-461-get-reference-vreg32
    test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-448-multiple-returns32 RUNNING
    /usr/local/google/buildbot/src/googleplex-android/master-art-host/art/test/448-multiple-returns: building...
    /usr/local/google/buildbot/src/googleplex-android/master-art-host/art/test/448-multiple-returns: running...
    /usr/local/google/buildbot/src/googleplex-android/master-art-host/art/test/448-multiple-returns: succeeded!
    test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-448-multiple-returns32 PASSED

After:
    [ 96% 5715/5906] build test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-461-get-reference-vreg32

Change-Id: Idf6fce7f48a619f83254b48861dbd7f8eb4ebdbf

Merge remote-tracking branch 'goog/stage-aosp-master' into HEAD

Merge "Limit summary string lengths to 200 characters."

Fix broken handling of instrumentation lists.

- We cannot copy before iterating, as entries might be deleted.
- We cannot remove entries in the list, as mutators could be
  currently iterating over it.

Solution in this change is to never remove list entries, but
put null when a listener is removed. When adding a listener, we
will either put it where there is a null slot, or at the end
of the list if there is no null slot.

Change-Id: Id94582fd971cd56bcb445caff64270d21987f700

JDWP: fix java.lang.String creation

Bug: 25439464
Change-Id: I56f11ed942585e8110dbbba1178cf11ec76e032f

Merge "Use --oat-fd instead of --oat-file when calling dex2oat."
am: ea834bbed4

* commit 'ea834bbed4f39d23cb2aafb1aa5cf73f71bcf21a':
  Use --oat-fd instead of --oat-file when calling dex2oat.

Merge "Use --oat-fd instead of --oat-file when calling dex2oat."

Merge "Fix deadlock with the JIT code cache."
am: df6dc42ba2

* commit 'df6dc42ba2ca0fa43ba970ba2e60977422105f7e':
  Fix deadlock with the JIT code cache.

Merge "Fix deadlock with the JIT code cache."

Fix deadlock with the JIT code cache.

Also remove hack done for ThreadStress.

Change-Id: Ie25c3bca08d9f2b8919706fa3fc26c5ab213f4a3

Merge "Minor improvements for the CC collector."
am: 4bbe7807f3

* commit '4bbe7807f313bb8e59131812c31bf31513094f8f':
  Minor improvements for the CC collector.

Merge "Minor improvements for the CC collector."

lambda: Minor capture-variable/liberate-variable clean-up after post-merge reviews.

Change-Id: I64f867d4ed5a5efcac138097f38efe4bb7f1281d

Merge "Fix minor problems with interface verification and default methods."
am: 3482c27127

* commit '3482c271278b30ca7a5c6206b7f80332616f6a84':
  Fix minor problems with interface verification and default methods.

Merge "Fix minor problems with interface verification and default methods."

Merge "Revert "ART: Update DCE to work with try/catch""
am: 86a8e247e3

* commit '86a8e247e3cd873c03bb79f79bf88a1b4c4e0597':
  Revert "ART: Update DCE to work with try/catch"

Merge "Revert "ART: Update DCE to work with try/catch""

Revert "ART: Update DCE to work with try/catch"

This reverts commit ce52901e2c8377fc1c331ae0faf7fbcb46b9da97.

Change-Id: I6b3a1f2a3dc036030b089b0df2005ecefa66b949

Minor improvements for the CC collector.

- Split Mark() and inline its first part.
- Make sure some other routines are inlined.
- Add some UNLIKELY's.
- Use VisitConcurrentRoots().

Ritz EAAC GC time decreased from 28.9 -> 27.6s (-4.5%) on N5.

Bug: 12687968

Change-Id: I7bd13f162e7daa2a5853000fb22c5fefc318994f

Merge "ART: Update DCE to work with try/catch"
am: 808e752491

* commit '808e7524918da5f7f41d2ceaa58dbb381e5708ff':
  ART: Update DCE to work with try/catch

Merge "ART: Update DCE to work with try/catch"

ART: Update DCE to work with try/catch

Dead block elimination was previously disabled because it needed
to be updated. With this patch, try/catch blocks can be removed
as a result of a dead if/switch branch.

Change-Id: I3261060bf24fd5fe7bb0f989247f0ef62ec5fd7b

Merge "Add application image dex2oat options"
am: 49560fcce5

* commit '49560fcce5b9a07108f7cc75a3fb0fa265e5fa0e':
  Add application image dex2oat options

Merge "Add application image dex2oat options"

Add application image dex2oat options

Bug: 22858531
Change-Id: Ia6a2a832b20b5bb70672521cd06f679335efbd92

Merge "Fix tests flakiness with jit when using Proxy classes."
am: e00197d605

* commit 'e00197d60579838974c02853fd3371ac761922ae':
  Fix tests flakiness with jit when using Proxy classes.

Merge "Fix tests flakiness with jit when using Proxy classes."

Fix tests flakiness with jit when using Proxy classes.

We cannot copy the entry point between ArtMethod when
the entry point has been JITted. We put the interpreter
bridge instead.

bug:25334878

Change-Id: I65a50cc1f10a5a152733807f8c85fb3ed81c5829

Merge "Fix stalled comment from last commit."
am: 78ef3fee3f

* commit '78ef3fee3fcf0b2cfb68b3e943cd144d389313fd':
  Fix stalled comment from last commit.

Merge "Fix stalled comment from last commit."

Fix stalled comment from last commit.

Change-Id: Ic54821c80a8c6ab76d44e2a81a6f821b5741dc53

Merge "Fix inlining and lse bugs with unresolved access."
am: 816b2b3f70

* commit '816b2b3f704d61ba608ad2c3513d4697e7fb5dfe':
  Fix inlining and lse bugs with unresolved access.

Merge "Fix inlining and lse bugs with unresolved access."

Fix inlining and lse bugs with unresolved access.

bug:25414532

Change-Id: I48b6660754774ea3e8a62a74175b1aa3728e0151

Merge "Remove extra space at end of line in output."
am: 43c6d31c43

* commit '43c6d31c43c05d97511bb9658964fd3d2f32b915':
  Remove extra space at end of line in output.

Merge "Remove extra space at end of line in output."

Merge "Remove package on which we cannot revert."
am: f0b308bdba

* commit 'f0b308bdba546081d64d1e593395f8957b3a2e89':
  Remove package on which we cannot revert.

Merge "Remove package on which we cannot revert."

Remove package on which we cannot revert.

Change-Id: I863176f91747e418bc3365b724970d3354a75971

Remove extra space at end of line in output.

The done dumping histograms message in DumpHistogram had an extra
space before the \n, which this CL removes.

Change-Id: I9db3c185cb39fec6cbe78a60699357448bc6e3b0

Merge "ART: Enable inlining under try/catch"
am: 3e0597facd

* commit '3e0597facd506073fcf91db2d81ae2ffdce6b701':
  ART: Enable inlining under try/catch

Merge "ART: Enable inlining under try/catch"

ART: Enable inlining under try/catch

This patch updates the inliner to set try/catch information
when inlining into a method with try/catch. It does not yet
allow inlining of methods with try/catch because that will
require generating catch stack maps with inline info.

Change-Id: I7d57e1454e7da537d75c5c7eda60b22f3a30fa60

Merge "ART: Arm32 packed-switch jump tables"
am: c74e69f831

* commit 'c74e69f831608964f89ac7a641fbcc09156eefaa':
  ART: Arm32 packed-switch jump tables

Merge "ART: Arm32 packed-switch jump tables"

ART: Arm32 packed-switch jump tables

Add jump table support to the thumb2 assembler. Jump tables are
a collection of labels for the case targets, and an anchor label
denoting the position of the jump.

Use the jump table support to implement packed-switch support for
arm32.

Add tests for BindTrackedLabel and JumpTable to the thumb2 assembler
test.

Bug: 24092914
Change-Id: I5c84f193dfebf9e07f48678efc8bd151bb1410dd

Merge "Merge test 539 into test 538-checker-embed-constants."
am: 564420bc5b

* commit '564420bc5b37572d45067fd71ed64d818ef3030d':
  Merge test 539 into test 538-checker-embed-constants.

Merge "Merge test 539 into test 538-checker-embed-constants."

Merge "X86: Add support for ucomis[sd] reg/memory form"
am: 34e38c43d4

* commit '34e38c43d4728fb1d3a011f8c142bff94927f6fb':
  X86: Add support for ucomis[sd] reg/memory form

Merge "X86: Add support for ucomis[sd] reg/memory form"

Merge "Keep list of dex files for oat file in CompilerDriver."
am: 469e9c4515

* commit '469e9c4515cb68509f02def2fe6c0465ffb55628':
  Keep list of dex files for oat file in CompilerDriver.

Merge "Keep list of dex files for oat file in CompilerDriver."

Keep list of dex files for oat file in CompilerDriver.

Use this list to improve invoke-static/-direct dispatch for
intra-oat calls.

Also fix a latent ArmBaseRelativePatcher::ReserveSpaceEnd()
bug exposed by a buggy early version of this CL: when we
have unresolved patches at the end of all code, we need to
emit a final thunk. Though the OatWriter will try to patch
the unresolved call to a trampoline at the beginning of the
oat file, that trampoline may be too far and the relative
patcher doesn't know about it anyway, so it needs to assume
that a thunk is needed.

This reduces the overall size of oat files present in dalvik
cache on Nexus 9 after first boot by over 1MiB, AOSP ToT,
aosp_flounder-userdebug build.

Change-Id: I98604b70cb17377eed057c1c23971865cf344e43

X86: Add support for ucomis[sd] reg/memory form

Allow an Address as the second operand.

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

Merge "A generic jni method can be instrumented."
am: 289c4e3a10

* commit '289c4e3a10acb6d0250dc25dd42465ca4d08c762':
  A generic jni method can be instrumented.

Merge "A generic jni method can be instrumented."

Merge "Add a few instructions to setup-buildbot-devices.sh"
am: 257df4e79a

* commit '257df4e79a7fc53406fd39f1417611d6e1cac316':
  Add a few instructions to setup-buildbot-devices.sh

A generic jni method can be instrumented.

Fixes tracing 119-noimage-patchoat test.

Change-Id: Ifc9709b714dbad6e087d327fdde7a008e5e731d3

Merge "Add a few instructions to setup-buildbot-devices.sh"

Merge "Fix compile-time error."
am: a5f295fcdb

* commit 'a5f295fcdb21c0a8e06382485250dcb4250ac442':
  Fix compile-time error.

Merge "Fix compile-time error."

Fix compile-time error.

Change-Id: I457db9d43f1cc0f908542b0fe5f13cbc4beb1f1b

Merge "Refactor code so that JIT can parse compiler options."
am: f31f9739e6

* commit 'f31f9739e6cb06298604f5fb723db2ab9a8f2962':
  Refactor code so that JIT can parse compiler options.

Merge "Refactor code so that JIT can parse compiler options."

Add a few instructions to setup-buildbot-devices.sh

- Uptime to know if the device restarted
- Kill stalled test dalvikvm processes.

Change-Id: I22e01ccc2d2c75a252e9fcee577e05b203e976c0

Refactor code so that JIT can parse compiler options.

Currently only the CompilerOptions class. We should also
do it for the CompilerDriver options.

This will fix the flakiness seen on jdwp testing, as the
debuggable flag was not passed to the compiler.

Change-Id: I4218dd5928c9f4fe2d6191ab11e5598e7cf84bcf

Merge "MIPS32: Create infrastructure to support intrinsics"
am: 70d95b4cab

* commit '70d95b4cabcabff7eda37e9e8348744891008e65':
  MIPS32: Create infrastructure to support intrinsics

Merge "MIPS32: Create infrastructure to support intrinsics"

MIPS32: Create infrastructure to support intrinsics

Add an empty intrinisics implementation for MIPS32.

Change-Id: I3f46ade1c548e6453f094ce200e66232348ef467

Merge "Made run-test 960-964 support being run with jack compiler."
am: 56c8c9ec60

* commit '56c8c9ec60a1a887bd39302fa8e2a422cb86fc80':
  Made run-test 960-964 support being run with jack compiler.

Merge "Made run-test 960-964 support being run with jack compiler."

Made run-test 960-964 support being run with jack compiler.

They will explicitly check that the jack compiler being used is
capable of compiling them before they do however (just to make
test-art work better).

Change-Id: I6b30c50d4159c2924886886d161479cbe869f4b0

Merge "Revert "ART: Enable more passes under try/catch""
am: d13f2b3554

* commit 'd13f2b35546d421f570c64654ad8479caf1165d6':
  Revert "ART: Enable more passes under try/catch"

Merge "Revert "ART: Enable more passes under try/catch""

Revert "ART: Enable more passes under try/catch"

BCE does not set TryCatchInformation when creating new blocks.
Will be fixed with DynamicBCE CL.

This reverts commit 39fabd6bb6fcf7a712b370d3b6fd0ada83e2e5d8.

Change-Id: I76ae707ac132bb1a4a9f64f4916ffcd786ef730c

Merge "Change a few places to use string version of GetTypeId"
am: 8cb0bc28b1

* commit '8cb0bc28b1161dad0d2c2c1e75318b7e504faa46':
  Change a few places to use string version of GetTypeId

Merge "Change a few places to use string version of GetTypeId"

Change a few places to use string version of GetTypeId

GetTypeId is about 20% faster than GetStringId + integer GetTypeID
since it does less binary searches.

Change-Id: I876c4ac89ab206acca217b2287b0197ef2e408c2

Merge "Move loop invariant utility to more general place."
am: c65d1f3d26

* commit 'c65d1f3d26a47e2fce3c3f8872e7942fec23f9b9':
  Move loop invariant utility to more general place.

Merge "Move loop invariant utility to more general place."

Merge "ART: Enable more passes under try/catch"
am: 5ad27592e2

* commit '5ad27592e2317b4c7932885eff702099f7cbe0f1':
  ART: Enable more passes under try/catch

Merge "ART: Enable more passes under try/catch"

Move loop invariant utility to more general place.

Change-Id: I15ebfbf9684f0fcce9e63d078ff8dc1381fd1ca3

Merge "Add comment for RunCheckpoint and clean up ThreadList a bit"
am: e22d85249e

* commit 'e22d85249e6e3985edf5e17a9c28ec26e0e6df3e':
  Add comment for RunCheckpoint and clean up ThreadList a bit

Merge "Add comment for RunCheckpoint and clean up ThreadList a bit"