Merge "ARM: VIXL32: Implement a disassembler."

Merge "MIPS64: Fix overlapping use of "in" & "out" registers."

Merge "Move ART_HOST_TEST_DIR to Android.common_test.mk"

Merge "MIPS32: Improve storing of constants in fields and array elements"

Merge "Revert "test: Remove icu cleanup hack""

Revert "test: Remove icu cleanup hack"

This reverts commit d4b294c14bd4e1654b276b931ee035b895b8e6cd.
Without this hack, the gCommonICUDataArray seems to
non-deterministically accumulate entries, and sometimes overflows,
causing test failures.

Bug: 31469648
Test: gdb /data/nativetest64/art/arm64/art_runtime_compiler_tests/reflection_test
Change-Id: Ie5a5b7b957de79758a227bfd2919b4cd0155cde8

Merge "Remove TypeLookupTable from DexFile."

Merge "Fix 2ND arch gtest testing."

Move ART_HOST_TEST_DIR to Android.common_test.mk

I'm improving Kati's re-run detection, and this $(shell echo $$PPID) now
now causes us to reparse all of the makefiles every time, since it
really does change on each execution. But if you aren't running the ART
tests, this doesn't actually get used.

So punt this over to only executing when you're including the test
runner -- it's not ideal, since you'll be reparsing the makefiles every
time you want to run the tests. Depending on the desired behavior, this
should be able to be implemented without depending on the PID of Kati,
or the test runner can be moved out of the build system, since most of
the builds here have moved to Soong now.

Bug: 30947985
Test: Compare build.ninja before and after this change, only change is
      the valgrind-test-art-target* targets.
Test: m -j test-art-host
Change-Id: Ibbc0f20e407bf70bce64d3428be5808d78bdaf91

Remove TypeLookupTable from DexFile.

One more step towards removing runtime dependencies from the DexFile
API.  This severs the ties to OatFile.  Work remains to move MemMap out
of DexFile.

Bug: 22322814
Change-Id: I29e7ad8fd292c7919ed2689dc754b958b88d6819
Test: test-art-host

Merge "Add a comment to dex2oat's parent pid logging"

Fix 2ND arch gtest testing.

Change-Id: I944feac324c49a95904f69d7b03b072e7150c134

Merge "Fix a deadlock in the CC collector."

Merge "Move art::Runtime::InitZygote logic from art into frameworks/base."

Merge "x86/x86-64: Simplify codegen using helper functions."

Merge "Fix build for VerifierDeps gtest on target"

ARM: VIXL32: Implement a disassembler.

Test: m test-art-host
Test: m test-art-target
Test: manual comparison with the previous oatdump output
Change-Id: I21ae26406200e455b8b036f53d585ea0b1bd11be

x86/x86-64: Simplify codegen using helper functions.

Simplify comparisons and array element address calculations.

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

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"

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"

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"

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

MIPS32: Improve storing of constants in fields and array elements

Test: booted MIPS32 in QEMU
Test: test-art-target-run-test-optimizing on CI20
Test: test-art-host-gtest

Change-Id: Ifcf8c1e215e3768711c391e8da6f663bba71f8d9

Fix a deadlock in the CC collector.

Fix a deadlock between CC GC disabling system weaks and thread attach.

See 31500969#2 for more details.

Bug: 31500969
Bug: 12687968
Test: test-art-host with CC. N9 libartd boot. Ritz EAAC.
Change-Id: Ic9a8bfb1c636643a03f4580b811fe890273576b6

Merge "Fix mac build"

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'

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

Merge "Fix run_java_fuzz_test.py imports"

Add a comment to dex2oat's parent pid logging

Adds a comment explaining why we are logging parent pid in dex2oat.

Test: mm -j
Change-Id: I6ed56bb9315a264e8f3d98777e616c4c9e544506

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."

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"

MIPS64: Fix overlapping use of "in" & "out" registers.

Test: Boot MIPS64 QEMU
Test: 568-checker-onebit

Change-Id: Ifc89fd8df674b1f3ffcdde52256a35b4a3562d45

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"

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.""""

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."

Merge "Move ArrayRef to runtime/base"

Merge "Fix dexlist_test"

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)"

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

Move art::Runtime::InitZygote logic from art into frameworks/base.

Test: m test-art
Test: m
Test: aosp_angler-eng boots
Change-Id: I119cdd74e5909137365ce723fca1a003ef3c5f95

Add missing OVERRIDE qualifiers in code generators.

Test: mmma art
Change-Id: I91d0a2dc23dc8d63a9bb3607eb1c1517eabaeb1f

Merge "Make ScopedAssertNoThreadSuspension no overhead for non-debug"

Merge "Update art to use hooks instead of customizer"

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

Merge "Separate annotations from dexfile reading."

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."

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."""

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."

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"

Merge "Add test dependency on dexlist"

Add test dependency on dexlist

dexlist_test depends on dexlist, add a new list of non-debuggable
executables for tests to depend on.

Bug: 31469510
Change-Id: I3e8f5798be523753ed8eda6cbbabe3251436877f

Fix mac build

-z muldefs doesn't exist on mac.  It is only necessary for dex2oats,
which doesn't build on mac, move it from the defaults to the dex2oats
targets.

Change-Id: I47f35d159fc0602f54412e12ef44798ca4afa4de

test: Remove icu cleanup hack

libjavacore now cleans up ICU itself properly in JNI_OnUnload,
removing the need for this hack in a test.

Bug: 31469648
Change-Id: Ic415edb20c3a64472f4dc4e892af01ad94cf2fce

Merge "Fix mac build"

Make ScopedAssertNoThreadSuspension no overhead for non-debug

Previously it required Thread::Current() which may not be free.
The plan is to add a lot more ScopedAssertNoThreadSuspension in
the codebase.

Also cleaned up callers.

Bug: 31458474
Change-Id: I5a1621a5435476504d22266cc01a9bf26aab7568

Merge "String Compression (interpreter only) bug fix"

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

Fix mac build

Change-Id: Ic3e74f96645384c0cec3aa5156f4ead8912f3be2

Update art to use hooks instead of customizer

Change-Id: I46de5ea6ffbc0e32a0b306b8fe82b66b986e33cd

Merge "Convert more of art to Android.bp"

Merge "Add more logging for checking in to-space for CC copy"

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

This reverts commit 9b5515ebb4502119c44e898b05c69312a45c5f49.

Breaks some gtests.

Change-Id: Ic9c9061b05219f19d5c7713ce0239e60c9d6e287