Merge "Shorten long test names"
am: 7bdc637f99

Change-Id: I58914e18b962a741e90e474a4e246998488bc3f9

Merge "Shorten long test names"

Shorten long test names

Long Android.run-test.mk targets cause testing framework to
misbehave. Issues occur in no-prebuild mode when --output-path
parameter generated by the make system is too long. When this
happens oat generation fails silently in oat_file_assistant.cc
during creation of empty vdex and oat files.

Creation fails because file names, which are as long as absolute path
to source dex file, are longer than 255 characters Linux file name
limit.

Because oat generation fails silently affected tests pass even though
they were effectively ran in interpreter mode, instead of intended
default mode.

This CL shortens test names to be at most 35 characters. This helps
make the tests more reliable.

Test: m test-art-run-test
Bug: 31597671
Change-Id: Ic68408ae0c98ffca0c64ca4f5535f4e683a296f0

Merge "Improve sharpening for read barrier image HLoadClass"
am: 2722620b24

Change-Id: I874f1652f6a717e2c945f6eaf070d4d1c396a514

Merge "Improve sharpening for read barrier image HLoadClass"

Merge "Fix mac build"
am: c520f85302

Change-Id: I5b79dd8a9d5035d8db240a9cab35de12a53367a1

Merge "Fix mac build"

Merge "Revert "Revert "test: Remove icu cleanup hack"""
am: 9aec85619f

Change-Id: I0b2257617f6984542ade989825c2a4c3fbbf64a0

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

Fix mac build

Move darwin disabling to art_test_defaults to disable mac build for
libarttest.

Test: mmma -j art on mac
Change-Id: I972245ddbacea274087d9231e4788bd9f2d6fe6b

Improve sharpening for read barrier image HLoadClass

Handle MarkInBoot image for LoadKind::kReferrersClass. This reduces
read barriers for the boot image if the referring class is in
the boot image.

Only really helps CC case, TODO avoid doing the work for non CC?

ARM64 CC baker boot.oat: 48006064 -> 47916736

Bug: 29516974

Test: test-art-host CC baker

Change-Id: Ibfa0cf0a3b888ad0e53c2d95a38e330b79e7443c

Merge "Ensure class is initialized when reading its fields from debugger."
am: 6a40fc9bbb

Change-Id: Iae28794d6ca5a59d699236ad0218960b3a2ca648

Merge "Ensure class is initialized when reading its fields from debugger."

Merge "Convert art runtests to Android.bp"
am: 904614f008

Change-Id: Ie9c606e11f31219b4ec86672509f3d103431b43c

Merge "Convert art runtests to Android.bp"

Convert art runtests to Android.bp

Build the art runtest libraries with Android.bp.  The host test
libraries move from out/host/linux-x86/lib[64] to
out/host/linux-x86/nativetest[64], and the device test libraires from
/data/artest/${TARGET_ARCH} to /data/nativetest[64]/art/${TARGET_ARCH},
which requires adding nativetest to the library search path, and fixing
some tests to support multiple entries in library search paths.

Test: m -j test-art-host
Test: m -j test-art-target
Change-Id: I2118a3292f22f275954ddd7cdd4e12287fe47808

Merge "Remove the `CanTriggerGC` side-effects on a few instructions."
am: ec88abde59

Change-Id: I67986e0771206a5ddf0742ec0f51653983d38242

Merge "Remove the `CanTriggerGC` side-effects on a few instructions."

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

This reverts commit 3c73829dbd9993b821171cdfe604d69a8330a20b.

Change-Id: Ib911f65456f522d0d8ac732aa3fa646217f126ca

Merge "Include `nodes.h` in `nodes_shared.h` to help editing tools."
am: 8cb366a7ba

Change-Id: I3b2e4505aa016787100980a912c8cd7d13b3ad9a

Merge "Include `nodes.h` in `nodes_shared.h` to help editing tools."

Merge "Minor induction variable analysis changes."
am: 28079d33e1

Change-Id: Ic7bfc3c6ad0b551f859b1efd71caa6d88eec2b36

Merge "Minor induction variable analysis changes."

Merge "ARM: VIXL32: Implement a disassembler."
am: 932e7d89ad

Change-Id: I1ca4882a776fb4a7e8fade6ac2bbc0b5773953bb

Merge "ARM: VIXL32: Implement a disassembler."

Include `nodes.h` in `nodes_shared.h` to help editing tools.

This include does not change anything to compilation, but it helps
editing tools (e.g. YouCompleteMe) to show proper error messages when
editing that file.

Test: mma -j40

Change-Id: I5ebd03040cfc90daa41faf85f273a86525207f19

Remove the `CanTriggerGC` side-effects on a few instructions.

The side-effect was specified for these instructions as they call
runtime. We now have a list of entrypoints that we know cannot trigger
GC. We can avoid requiring the side-effect for those.

Test: Run ART test suite on Nexus 5X and host.

Change-Id: I0e0e6a4d701ce6c75aff486cb0d1bc7fe2e8dda4

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

Change-Id: I24d120adc95b63d4fb6a746cade3fc14926d5261

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

Merge "Move ART_HOST_TEST_DIR to Android.common_test.mk"
am: 834582c4b0

Change-Id: I28dd222a6f43995609fb0489f494a136e0effab9

Merge "Move ART_HOST_TEST_DIR to Android.common_test.mk"

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

Change-Id: I47a2f35d0a589132f5ed42215cdfe58cda21b966

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

Merge "Revert "test: Remove icu cleanup hack""
am: 53c5337b04

Change-Id: Ic37957922c05b6b1dbe4a5cd3cfd4b5c53b7dbb0

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."
am: 6a4abc633f

Change-Id: Id26c8ab42a7282582937939f9d700523c30975bd

Merge "Remove TypeLookupTable from DexFile."

Merge "Fix 2ND arch gtest testing."
am: 8d0e6c5c18

Change-Id: Id04e0c3149b6280e5cfd67b609276ac297020072

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"
am: 47c22ff721

Change-Id: Id0ca435c27d91048f8071a5f6eee24b769d876c7

Minor induction variable analysis changes.

Rationale:
Few extra comments, added replacement of loop control functionality.
All in preparation of bigger loop optimizations CL.

Change-Id: Ie2243c3c8b57b2901654b05fb200e9d80a0cb641
Test: test-art

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."
am: be3a3ee02f

Change-Id: If75da5e6e72d7ba7556c800d31c7bfa830a7a211

Merge "Fix a deadlock in the CC collector."

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

Change-Id: Id2592a1cf90f82ac20bd383fd6cebdbd654a92e5

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

Change-Id: Ie6462837f4bc0fb8bc4185e2e5413e868a3f03bd

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

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

Change-Id: I132118d1b186fff07c7390ac85df5898e93a17f0

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

Change-Id: I9a404a88a3338f5d7c6df4066134816fe6f39b97

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

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

Ensure class is initialized when reading its fields from debugger.

Bug: 31016523
Test: mm run-jdwp-tests-ri & mm run-jdwp-tests-host
Change-Id: I6bccf8464de7ec1e0c1fdac862cbacf890abede6

Merge "Fix mac build"
am: 175a9be49d

Change-Id: I57ec257f1b84fffc9e201953981e2ac4a12e4fd0

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

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