Merge "Fix build for VerifierDeps gtest on target"
am: 4613f533a8

Change-Id: I9a404a88a3338f5d7c6df4066134816fe6f39b97

Merge "Fix build for VerifierDeps gtest on target"

Fix build for VerifierDeps gtest on target

Forgot to specify the target for the .jar file.

Bug: 30937355
Change-Id: Id44bdeb91474d8acfdf9e9144da9a4c9f4c6cf55

Merge "Implement VerifierDeps encoding/decoding"
am: bbefcb3c8f

Change-Id: I803f79f06583505fee868cfab210847b5b3d7127

Merge "Implement VerifierDeps encoding/decoding"

Merge "Add a phony target for golem."
am: 31eb450500

Change-Id: I98cb017cecb25826698e92e732ab8b9ad76702da

Merge "Add a phony target for golem."

Add a phony target for golem.

To cut down on the size of the build.

Change-Id: Ic7dce1f8b73cf234c3e0163a0c03c7e19d9b02e1

Merge "Collect verifier dependencies"
am: 031f3dd847

Change-Id: I5315199a9832d293d37b180d978bdd1531e2ea9b

Merge "Collect verifier dependencies"

Implement VerifierDeps encoding/decoding

This patch implements serialization and deserialization of the data
recorded by VerifierDeps.

Test: m test-art-host-gtest-verifier_deps_test
Bug: 30937355
Change-Id: I19320b8e70d5c5128653d09a5cdb5b6f677a2f2d

Collect verifier dependencies

MethodVerifier tests whether a DEX method is valid w.r.t. the classes
in class path. Since the APK does not change across OTA updates, it
is not necessary to analyze the bytecode again with MethodVerifier,
as long as its dependencies on the class path (which may have changed)
are satisfied.

This patch introduces VerifierDeps, a class path dependency collector,
and adds hooks into MethodVerifier where classes/methods/fields are
resolved and where assignability of types is tested.

Test: m test-art-host-gtest-verifier_deps_test
Bug: 30937355
Change-Id: Iee0b321d772a5c7d1cb471aaa6e13918310b7e2f

Merge "test: Remove icu cleanup hack"
am: 0764877e21

Change-Id: Iad7d8809f1fc3ba5d46368fd36f08577dbf8997a

Merge "test: Remove icu cleanup hack"

Merge "Fix 32-bit device tests"
am: 2c92bf2340

Change-Id: I3ecc880064de5a71237761445dcb2bafa41f9c11

Merge "Fix HOST_PREFER_32_BITS tests"
am: 6615375c90

Change-Id: I81d5706719d1b758d6fae72a45379f94e19d144b

Merge "Fix 32-bit device tests"

Merge "Fix HOST_PREFER_32_BITS tests"

Fix 32-bit device tests

32-bit tests are installed to /data/nativetest, not /data/nativetest32.

Test: m -j test-art-target-gtest32
Change-Id: I2736d6783dd34550dbfbbe54ffa724cd2b886d8b

Fix HOST_PREFER_32_BITS tests

When HOST_PREFER_32_BITS is set, HOST_ARCH is still x86_64 but art tests
are only built for x86.  Use ART_HOST_ARCH, which will be set to x86,
to find the list of host tests.

Test: m -j HOST_PREFER_32_BITS=true test-art-host-gtest32
Change-Id: I11a9e1b5111c3c06139b44e364fdcd04bb8db4c6

Merge "Fix mac build"
am: 175a9be49d

Change-Id: I57ec257f1b84fffc9e201953981e2ac4a12e4fd0

Merge "Fix mac build"

Merge "Avoid using memcpy for object header in ConcurrentCopying::Copy"
am: c8651471ab

Change-Id: I72b035ec5714413040c3cea6e13fac6ca0876641

Merge "Avoid using memcpy for object header in ConcurrentCopying::Copy"

Fix mac build

I accidentally dropped the hunks that disabled the mac build when
rebasing.

Change-Id: I412325f71c4bfde7a357b50daeb966e6db32c388

Avoid using memcpy for object header in ConcurrentCopying::Copy

Memcpy can do per byte copy, this occasionally caused tearing in
the lockword when read from the source resulting in incorrect
values.

Test: test-art-host CC baker

Bug: 31423258
Change-Id: I13f856835125a3aa331530fb1f16db45d1955c8a

Merge changes from topic 'art_tests'
am: 7d26164308

Change-Id: I64cfdc5ea42dd9f439129c106fbc814f38eb192b

Convert art gtests to Android.bp
am: 6e95dd56e4

Change-Id: I2cb2e32e4ce446a4cbb6776f0a8b81d5b67d0b7d

Convert libart-gtest to Android.bp
am: 6b22aa5106

Change-Id: Ib3f8198e9a6c84527cd50353a52be7bdbea398bd

Merge changes from topic 'art_tests'

* changes:
  Convert art gtests to Android.bp
  Convert libart-gtest to Android.bp

Merge "Fix run_java_fuzz_test.py imports"
am: 669cfcd570

Change-Id: I257360044e58e5cbddcfd54e8038cb1b71d77c09

Merge "Fix run_java_fuzz_test.py imports"

Merge "Minor edits in javafuzz"
am: 2de29f1626

Change-Id: I55e9702655cbfbf51d6b7b55dfd83186c5e49c8a

Merge "Minor edits in javafuzz"

Fix run_java_fuzz_test.py imports

Make imports works independently of directory the tool is called
from.

Test: run_java_fuzz_test.py

Change-Id: I28cd875948e3e5e58358ecfd6f41e63b808172c9

Convert art gtests to Android.bp

This splits the compilation and running of the art gtests into two
separate locations.  The tests are now compiled in multiple Android.bp
modules in each directory.  art.go collects the installed locations of
each test and exports it as make variables.  art/build/Android.gtest.mk
converts the list into the rules to run the tests.

This has a few changes in behavior:
  - The rules to build tests are now always defined, and will build as
    part of mmma art or make checkbuild.
  - Host tests are no longer installed into out/host/linux-x86/bin, they
    are in out/host/linux-x86/nativetest[64]/<module name>/<test name>
  - Target tests are now in
    /data/nativetest[64]/art/<arch>/<module name>/<test name>

Test: mmma -j art
Test: m -j test-art-host
Test: m -j test-art-target
Change-Id: Iabcd99d43890e6b693688422b07a283c3226a496

Convert libart-gtest to Android.bp

Change-Id: Ibea17e3e002e6ad3f82158cd3c69d087fa9b15c9

Merge "Fix google-explicit-constructor warnings in art."
am: 6b4d988feb

Change-Id: I7b077106056073983a5ceb975d483ed72d83350f

Merge "Fix google-explicit-constructor warnings in art."

Minor edits in javafuzz

Rationale:
Missing newline in run script. Updated README with caps for args.

Change-Id: Ic8a46ab5812049401c00d7d37119bfc4034fe2d6
Tests: run_java_fuzz_test.py

Merge "Fix moving GC bugs in entrypoints"
am: 5a7c611d7e

Change-Id: I1abf194d7779b4cad3b05d82d6eae57236ea755d

Merge "Fix moving GC bugs in entrypoints"

Merge "Added ability to generate last-value of linear induction. Also added utility to update fetches in induction nodes."
am: 514e346dd7

Change-Id: I689c8d493d9ae741480d09ac25609541404c25e1

Merge "Added ability to generate last-value of linear induction. Also added utility to update fetches in induction nodes."

Merge "Integrate bisection search with javafuzz"
am: eb1aa7da78

Change-Id: Ie0324c9fff3a6d8c43439106b711984713f452db

Merge "Integrate bisection search with javafuzz"

Merge "ARM: Try to emit branches early to save memory."
am: fad6f4dbf9

Change-Id: Iae74bb30115039631fc44830f8c73dda32ccc37a

Merge "ARM: Try to emit branches early to save memory."

Fix google-explicit-constructor warnings in art.

* Add explicit keyword to conversion constructors,
  or NOLINT for implicit converters.
Bug: 28341362
Test: build with WITH_TIDY=1

Change-Id: I1e1ee2661812944904fedadeff97b620506db47d

Added ability to generate last-value of linear induction.
Also added utility to update fetches in induction nodes.

Rationale:
This is a first step towards the larger CL that introduces
a new loop optimization framework in the optimizing compiler
(see https://android-review.googlesource.com/#/c/271392/3).

Change-Id: Ibecd674c8146d9665340e68718c498555646129a
Tests: induction_var_range_test

ARM: Try to emit branches early to save memory.

Try to emit unexpanded branches when binding a Label to
reduce the memory needed for Fixup dependencies. We use
a simple heuristic to limit the time we spend trying
but this heuristic is enough to resolve all branches in
the emitted String.equals() intrinsic.

Test: Run ART test suite on host and Nexus 6.
Test: Manually check memory usage for a certain apk.
Bug: 28603175
Change-Id: I362e07c2c1285ff609cf8df0a00b6b420729b9ce

Merge "Revert "Revert "Re-apply "Cleanup run-test and Makefile around boot image and PIC.""""
am: fce42f486d

Change-Id: I32862cd80b29bb0c5d61a428ad1ea20d23087f74

Merge "Revert "Revert "Re-apply "Cleanup run-test and Makefile around boot image and PIC.""""

Merge "MIPS: Enable the ComparisonsLong test in the code generator test."
am: 6f41ec68d3

Change-Id: Idd9b1947a8b0dda85b612cd6f307389fd0b6bd9f

Merge "MIPS: Enable the ComparisonsLong test in the code generator test."

Revert "Revert "Re-apply "Cleanup run-test and Makefile around boot image and PIC."""

This reverts commit 8522758c5ef6050e355613630ff81c2b49373b44.

Change-Id: Ie18417bd3d528673e3da39d92f43f20b853facda

Merge "Add missing OVERRIDE qualifiers in code generators."
am: 37c38930c5

Change-Id: I7aadcee62daa5827a9c590914c0245828c825b59

Merge "Add missing OVERRIDE qualifiers in code generators."

Merge "Move ArrayRef to runtime/base"
am: 755bebaa19

Change-Id: I43651567025ccde9ef38df319af34c415abf4dc0

Merge "Move ArrayRef to runtime/base"

Merge "Fix dexlist_test"
am: 602700c305

Change-Id: I4a9c89693096d80889868f745d93612ef12a14d9

Merge "Fix dexlist_test"

Merge "Log PPID when running dex2oat"
am: dfcfc5458a

Change-Id: Ica56371926a32cea3784797d44196a965ee33c23

Merge "Log PPID when running dex2oat"

Fix dexlist_test

xbin is deprecated, dexlist installs to /system/bin now.

Bug: 31469510
Test: m test-art-target-gtest-dexlist_test
Change-Id: I352ae9eaaf2f9e4ce6682c5022c138b07d29ccca

Fix moving GC bugs in entrypoints

In FindMethodFromCode, there was missing handles for objects used
after a suspend point (ResolveType).

For check_jni, JniMethodEndWithReferenceHandleResult did not
handlarize the return value when calling CheckReferenceResult.

For CheckReferenceResult, the argument was not in a handle.

Bug: 31458474
Change-Id: I06617fc63f1fb63d7c13ad4245223a8c54fa4b98

Log PPID when running dex2oat

Log parent's pid in dex2oat. Logging happens only in verbose
compiler mode.

This can be used to find out which process executed a dex2oat
instance.

Test: m test-art-host
Change-Id: I49a139ed39a2454649f21275d6be860cbab96f55

MIPS: Enable the ComparisonsLong test in the code generator test.

Test: test-art-host-gtest-codegen_test
Test: test-art-target-gtest-codegen_test (MIPS32R2 & R6, MIPS64)

Change-Id: Ieae0fdb2ed30f262baac0eb7c6b658341c511a47

Merge "Make LinearizeGraph() public (and move it to nodes files)"
am: 32772cbdbc

Change-Id: Idb70fd87a2049a02f88a7b66de9e40025d9349ff

Merge "jni: Update VIXL macro assembler to have vmovdrr/vmovsr support"
am: 8dae910bb8

Change-Id: Ib0f877b0be99b23dc71259369ad8ffd939dd18c0

Merge "Make LinearizeGraph() public (and move it to nodes files)"

Merge "jni: Update VIXL macro assembler to have vmovdrr/vmovsr support"

Integrate bisection search with javafuzz

This CL makes javafuzz call bisection search on failing tests.

Three switches were added to bisection_search, --logfile which can be
used to provide custom logfile destination, --never-clean which
disables automatic cleanup of bisection directory and --timeout
which allows user to specify maximum time in seconds to wait for
a single test run.

ITestEnv subclasses were updated to integrate with javafuzz.

run_java_fuzz_test.py is now reusing code from bisection_search
module. It also better matches python style guidelines.

Change-Id: Ie41653b045469f2ceb352fd35fb4099842bb5bc3

Make LinearizeGraph() public (and move it to nodes files)

Rationale:
It is strange that HLinearOrderIterator is defined (and visible)
in nodes.h, but clients have no way to build this order. This CL
makes the building available at the usual place.

Change-Id: Ib66f2edf6dfc8edd6b429bd4bea3ac7e37440b28
Tests: m test-art

jni: Update VIXL macro assembler to have vmovdrr/vmovsr support

This is sometimes used by @CriticalNative JNI compiler
to avoid spilling the user native function pointer to the stack.

(Also fixes the failing target jni_compiler_test32)

Test: make -j test-art-target-gtest-jni_compiler_test32
Bug: 31469040
Change-Id: I04a5ec20272fe89e613959a37da1cb472c4ebdf9

Move ArrayRef to runtime/base

Will be used in upcoming CLs regarding VDEX and VerifierDeps.

Test: m test-art-host
Change-Id: I68e611a4a52246c2bdf45eab7c61f3212908afd4

Add missing OVERRIDE qualifiers in code generators.

Test: mmma art
Change-Id: I91d0a2dc23dc8d63a9bb3607eb1c1517eabaeb1f

Merge "Make ScopedAssertNoThreadSuspension no overhead for non-debug"
am: 26ead4975e

Change-Id: I9c109f784cf101efe627cf495c72e7a2604421ed

Merge "Make ScopedAssertNoThreadSuspension no overhead for non-debug"

Merge "Update art to use hooks instead of customizer"
am: 79f6d70618

Change-Id: Id90182aa3018757095b4634fb901dceeb331e23b

Merge "Update art to use hooks instead of customizer"

Merge "ARM/ARM64: Fix throwing entrypoints to call runtime handlers."
am: 52a8da639e

Change-Id: Icfd04eee73ee323215cc10d690c9bbba27973ed4

Merge "ARM/ARM64: Fix throwing entrypoints to call runtime handlers."

Merge "Separate annotations from dexfile reading."
am: 088efb4cf1

Change-Id: I7f1b5511ad0199a550f6dcd92f5ff695657252a3

Merge "Separate annotations from dexfile reading."

Merge "MIPS32: Implement table-based packed switch"
am: 8850c73572

Change-Id: I7583811454efd24862c8cab792855452afe98d82

Separate annotations from dexfile reading.

Bug: 22322814
Change-Id: I867d66da407dd80394a10d19903ebbc1ec3986ff
Test: test-art

Merge "MIPS32: Implement table-based packed switch"

ARM/ARM64: Fix throwing entrypoints to call runtime handlers.

Test: Run ART test suite including gcstress on Nexus 9.
Bug: 31468464
Change-Id: I2521675125d243be3168a2e87b70185791322b42

Merge "Remove ThrowNoSuchMethod entrypoint."
am: 58735fbfbb

Change-Id: I0a957e50944dbd690c359c4023673b455359c6ea

Merge "Remove ThrowNoSuchMethod entrypoint."

Remove ThrowNoSuchMethod entrypoint.

Run ART test suite on host and Nexus 6.
Bug: 31464666
Change-Id: I5aa737726031adae0b132f759cf802a93d581a7f

Merge "Revert "Revert "Use implicit null checks inside try blocks."""
am: e305eaf2c0

Change-Id: Ifd058e63aa920ec9769152c37198c0aabceb8c24

Merge "Revert "Revert "Use implicit null checks inside try blocks."""

Revert "Revert "Use implicit null checks inside try blocks.""

Fix implicit checks in try blocks to emit stack maps.
Fix arm64 null expection from signal entrypoint to call
the runtime handler instead or simply jumping there.

On Nexus 9, AOSP ToT, the boot.oat size reduction is
  prebuilt multi-part boot image:
    - 32-bit boot.oat: -448KiB (-1.3%)
    - 64-bit boot.oat: -528KiB (-1.2%)
  on-device built single boot image:
    - 32-bit boot.oat: -448KiB (-1.4%)
    - 64-bit boot.oat: -528KiB (-1.3%)
Note that the oat files no longer contain dex files which
have been moved to vdex, so the percentages are not directly
comparable with the those reported in the original commit.

Test: Run ART test suite including gc-stress on host and Nexus 9.
Bug: 30212852
Bug: 31468464

This reverts commit 0719b5b9b458cb3eb9f0823f0dacdfe1a71214dd.

Change-Id: If8a9da8c11adf2aad203e93b6684ce16ed776285

Merge "Make dex2oat art_cc_binary instead of just cc_binary."
am: 5cfaafbda5

Change-Id: Ib418ba66d3fabea5b852ced1f19ead54199996f9

Merge "Make dex2oat art_cc_binary instead of just cc_binary."

Make dex2oat art_cc_binary instead of just cc_binary.

This to get customized properties like CUSTOM_TARGET_LINKER.

Change-Id: Ifa25c52faa40fa6da40b614f664ab24ac1b7356a

MIPS32: Implement table-based packed switch

Test: booted MIPS32R2 in QEMU
Test: test-art-target-run-test-optimizing (MIPS32R2) on CI20
Test: booted MIPS64 (with 2nd arch MIPS32R6) in QEMU
Test: test-art-target-run-test-optimizing (MIPS32R6) in QEMU
Test: test-art-host-gtest

Change-Id: I2e1a65ff1ba9406b84351ba7998f853b1ce4aef9

Merge "Fix mac build"
am: b180b893b5

Change-Id: I6f42ae22e2ceb31612decca689296a44154e5d8a