Merge "Make class redefinition work with native methods on stack."

Merge "ART: Update unstarted runtime for changed libcore"

Merge "Revert "Revert "ART: Add JIT + events test to run-test 912"""

Make class redefinition work with native methods on stack.

Previously we would crash if we tried to redefine a class with native
methods if it had any native frames on a stack. This happened when we
tried to create an obsolete method for the native method.

Since native methods cannot really become obsolete in the same way
that regular java ones can we simply do not make any obsolete methods
for these native methods.

Test: mma -j40 test-art-host
Change-Id: Ib677f73faae52627a478aa7cfe870c1f2c4fb503

Merge "Fix ownership of DexFile in profman"

ART: Update unstarted runtime for changed libcore

Update float conversion to current libcore implementation. Add
tests.

Bug: 34956610
Test: m test-art-host-gtest-unstarted_runtime_test
Change-Id: Ib62c64011e2d5b980fb8fab81f6c343065ce946c

Fix ownership of DexFile in profman

Don't leak the DexFiles we open in profman.

Bug: 34929204
Test: make test-art-host
Change-Id: Ife29556117c4dd84dd3e970901a7fdf458e5ad98

Merge "A bit of oatdump clean up"

Merge "Fix dex cache resolved types and class table mismatch."

Fix dex cache resolved types and class table mismatch.

Record class table in ClassLinker::DexCacheData and use
it in DexCache.setResolvedType() to store the type also
in the initiating loader's class table if the dex file
has been registered.

Also throw InternalError when trying to register the
same DexFile with multiple class loaders. (Different
DexFile instances referencing the same file are OK.)

Test: 155-java-set-resolved-type
Test: m test-art-host
Bug: 30627598
Bug: 34193123
Bug: 34839984
Change-Id: Ia48acb300337c45880ea1459d2d32789546d67f4

Revert "Revert "ART: Add JIT + events test to run-test 912""

This reverts commit f833c5c551ec87f8d240b97142fd27b689613ccf.

Add NoSuchElementException to preparation.

Bug: 31684920
Test: m ART_TEST_JIT=true test-art-host-run-test-912-classes
Change-Id: Ied7c273194e77236e1df67a92c4a0086abcbcd46

A bit of oatdump clean up

Fix typo, remove unused byte category.

Test: test-art-host-gtest-oatdump_test

Change-Id: Ia6f58d5dd17930fbea7fd9cb05edc014c190f080

Merge "Update oatdump to have inline info breakdown"

Merge "Un-disable redefinition tests on JIT."

Merge "Keep failing tests' infomation without --verbose as well"

Un-disable redefinition tests on JIT.

We no longer have any problems with racing JIT threads.

Test: ART_TEST_JIT=true mma -j40 test-art-host
Test: ./test/testrunner/testrunner.py --host --jit -j40
Change-Id: Ie88d0a891fbdd906641c0f7561406390db208b09

Update oatdump to have inline info breakdown

Add a breakdown of where bytes are used for inline infos.

Bug: 34621054

Test: test-art-host-gtest-oat_dump_test

Change-Id: I5e44d17d631ff418a1c5baba166b1456211e9a33

Merge "Clean up code info table layout"

Merge "Revert -Wunreachable-code workaround."

Merge changes I1fb50fb6,I8b3fa817

* changes:
  Ensure that both stderr and stdout are captured
  Make sure that we join paths correctly.

Keep failing tests' infomation without --verbose as well

Test: Ran the testrunner with a failing test.
Change-Id: I4fd51c689c6c484f8ff141a3e20ff8fab4b11142

Revert -Wunreachable-code workaround.

Bug: http://b/31104323
Test: Built/ran ART successfully on sailfish.

This reverts commit 813b9c47d8952b3f02a6032d79d66433c981d030, which
added a workaround for Clang mistriggering -Wunreachable-code.

Change-Id: I46636c9b4f574a870c1f95b6160d05f2a75f8ab8

Ensure that both stderr and stdout are captured

We were not capturing the stdout FD for the run-test process. This was
leading to blank lines being printed during the running of some tests.

Test: ./test/testrunner/testrunner.py --host -j40
Change-Id: I1fb50fb6ae2af9c439f126c9fe47a3e24ad84e10

Merge "Undo class_linker.cc workaround for out of registers bug."

Make sure that we join paths correctly.

We were simply concatenating a pair of paths together but this can
cause problems if the second path is not relative. This could occur if
the OUT_DIR environment variable has been set.

Test: ./test/testrunner/testrunner.py --host -j40
Change-Id: I8b3fa817cdb114e39f9cf33a0c2e73e1a53f7e99

Clean up code info table layout

Previously:
Table layout was computed multiple places like stack_map_stream,
and getters. This made it difficult to add new stack map tables and
made the code hard to understand.

This change makes the table layout specified all inside of the code
info. Updating the layout only requires changing ComputeTableOffsets.

Changed the stack map inline info offset to be an index, so that it is
not require the inline infos are directly after the dex register table.

Oat file size for a large app: 94459576 -> 93882040 (-0.61%)

Updated oatdump and fixed a bug that was incorrectly computing the
register mask bytes.

Bug: 34621054

Test: test-art-host

Change-Id: I3a7f141e09d5a18bce2bc6c9439835244a22016e

Merge "ART Test Runner"

Merge "Revert "ART: Add JIT + events test to run-test 912""

Revert "ART: Add JIT + events test to run-test 912"

Still flakey
Bug: 31684920

This reverts commit 55e09d45d6ddedc88e50fbf290efbd39cae8380f.

Change-Id: I1a9dc303a1f9c4520e15edf584d24cb8e6bc4169

Undo class_linker.cc workaround for out of registers bug.

Bug: http://b/30419309
Test: Built x86_64/x86 targets successfully with latest toolchain.

This change re-enables optimization for a function in class_linker.cc
that previously exposed an internal compiler error in Clang. Now that
the bug has been fixed, we can remove the workaround.

Change-Id: If6520c17630b6f67ce016966c0777b675be1b852

Merge "ART: Make run-test 931 more defensive wrt/ shutdown"

Merge "ART: Add JIT + events test to run-test 912"

Merge "Verify redefined classes"

ART: Make run-test 931 more defensive wrt/ shutdown

Make the main thread loop a bit longer waiting for the agent
thread to die, so we don't unload under its feet.

Bug: 34701734
Test: m test-art-host-run-test-931-agent-thread
Change-Id: I554d72249c366b17c644f6c17c2b59fda2ab810e

ART: Add JIT + events test to run-test 912

Add a test for loading by the JIT thread.

This reverts commit fab0b1947db6812c91827d051f6fe5b21ccb494c.

Bug: 31684920
Test: m ART_TEST_JIT=true test-art-host-run-test-912-classes
Change-Id: I577cc3269ed437b605b4ebf6bac7262e0459af76

Merge "Add test if dex file fails to open during layout."

Merge "Fix issues due to updated toolchain."

Merge "Remove Mutex dependency from MemMap"

Merge "Revert to `-fstack-protector-strong` in read barrier config."

Verify redefined classes

Test: mma -j40 test-art-host

Change-Id: Ia0b5a0934ed895e37b67c9ae2e819648086280c8

Merge "ARM: VIXL32: Remove erroneous DCHECK."

ART Test Runner

The testrunner runs the ART run-tests by simply invoking the script.
It fetches the list of eligible tests from art/test directory, and list of
disabled tests from art/test/knownfailures.json. It runs the tests by
invoking art/test/run-test script and checks the exit code to decide if
the test passed or failed.

Before invoking the script, first build all the tests dependencies by
building 'test-art-host-run-test-dependencies' for host tests,
'test-art-target-run-test-dependencies' for target tests, and
'test-art-run-test-dependencies' for building dependencies for both.
There are various options to invoke the script which are:
-t: Either the test name as in art/test or the test name including the variant
    information. Eg, "-t 001-HelloWorld",
    "-t test-art-host-run-test-debug-prebuild-optimizing-relocate-ntrace-cms-checkjni-picimage-npictest-ndebuggable-001-HelloWorld32"
-j: Number of thread workers to be used. Eg - "-j64"
--dry-run: Instead of running the test name, just print its name.
--verbose
-b To build the dependencies before running the test.

To specify any specific variants for the test, use --<<variant-name>>.
For eg, for compiler type as optimizing, use --optimizing.

In the end, the script will print the failed and skipped tests if any.

New targets - test-art-host-run-test-dependencies and
test-art-target-run-test-dependencies have been added. The targets build
dependencies for host and target respectively.

Test: No tests
Change-Id: Ia4f13ee3444c2c733342c046ef1582517459fc9b

Merge "ARM: VIXL32: Fix "Align allocation entrypoints.."."

Merge "Revert "Revert "Redesign implementation of RegisterNativeAllocation."""

Merge "Force stack dump to diagnose empty checkpoint timeout (2)."

Add test if dex file fails to open during layout.

Bug: 34971122
Test: mm test-art-host-gtest-oat_test
Change-Id: I2cdf080783aa24193026683d98caa0d98cb7d055

Force stack dump to diagnose empty checkpoint timeout (2).

Bug: 33006388
Bug: 12687968
Test: test-art-host
Test: Thread dumping in a simulated empty checkpoint timeout.

Change-Id: I06641396b8f3d7a1a98366a01807aab2e6f31bd5

Merge "ART: Add marking to thread peer gathering in TI"

Merge "ART: Fix systrace monitor logging deadlock"

Merge "Add support for checking method and field invariants for redefinition."

ART: Add marking to thread peer gathering in TI

When CC is marking, we may get stale references due to a missing
mark.

Bug: 34760612
Test: ART_TEST_JIT=true ART_TEST_GC_STRESS=true test-art-host-run-test-924-threads
Test: ART_TEST_JIT=true ART_TEST_GC_STRESS=true test-art-host-run-test-925-threadgroups
Change-Id: I1becfc188b59a3c99cc7eea07c63abaaf108fd15

Merge "Revert "Revert "build: Generate asm_support_gen.h from the build, validate up-to-date"""

Add support for checking method and field invariants for redefinition.

Test: mma -j40 test-art-host
Change-Id: I62c5d967522c2b3f455253a299c889fd5be19232

ART: Fix systrace monitor logging deadlock

Move the method logging earlier to avoid a deadlock due to re-acquiring
the mutator lock.

Bug: 34990215
Test: m
Test: Test on device
Change-Id: Id8069b133d1408a659afadd7e265d5027ac55a39

Merge "Remove obsolete restriction with allocations and dex cache."

Remove Mutex dependency from MemMap

Use std::mutex to remove the dependency between MemMap and Mutex/Thread,
which depend upon Runtime.  Next step towards making dexdump2 build and
run on Windows.

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

Revert "Revert "build: Generate asm_support_gen.h from the build, validate up-to-date""

This reverts commit 592505c340e82091a6e13e2dff3d8589255df0bb.

Fix HOST_PREFER_32_BIT=true causing buildbot
to fail with the new asm_support genrule.

Test: SOONG_ALLOW_MISSING_DEPENDENCIES=true HOST_PREFER_32_BIT=true\
      make -j32  build-art-host-tests
Original-Change-Id: I1f0f94914d328c396906583d0732e281c076e69f
Change-Id: Ie08a11fdb9486b697d6cef4cec41b23ff120b205

ARM: VIXL32: Fix "Align allocation entrypoints..".

Add missing changes for VIXL backend to
https://android-review.googlesource.com/#/c/330124/.

Test: m test-art-host
Test: m test-art-target
Change-Id: Ic8383a7a426cbe8501e12838e9728fc3f359f03d

Merge "Enable thread annotations for std::mutex"

Remove obsolete restriction with allocations and dex cache.

We used to rely on being in the same dex file for doing allocations,
but not anymore.

Test: ART_TEST_JIT=true test-art-host
Change-Id: Iabc27469dadea6bfe39054e4be305e620818ac99

Merge "Fix test after inlining optimization."

Fix test after inlining optimization.

Runtime.getRuntime() can now be inlined, but the test was
assuming it could not, therefore instructions would not get
licm'ed.

test: 482-checker-loop-back-edge-use
Change-Id: I19ea20c555f699673556225ecf9d945c5a91afa5

Merge "Tests for libcore change d2aa1365d25911076cb174a04b1d5152f0ff80e3"

Revert "Revert "Redesign implementation of RegisterNativeAllocation.""

This reverts commit 36bdbd2bf2ee36662f700b63474b546a7abecfa3.

Bug: 29156652
Bug: 32576211
Test: 004-NativeAllocations in a loop with high machine load.
Change-Id: I4470222c66aef4e0daa7612c84177b6c35bd28a9

Merge "Revert "Revert "Inline across dex files for JIT."""

Tests for libcore change d2aa1365d25911076cb174a04b1d5152f0ff80e3

Test: make test-art-host
Bug: 31028374
Change-Id: Ic6e459d068f8ff2f7bd958bca9e7be433170381b

Merge "Clean up art::mirror::Class::SetSuperClass."

Merge "Code refactoring around sharpening HLoadClass."

Revert "Revert "Inline across dex files for JIT.""

bug:30933338

This reverts commit d16da8bd8106452eea82408748dc6b3fd64bcb80.

Change-Id: I6a30354d6d00442cb1a542af063c7769865e369d

Merge "Clean up transaction"

Merge changes Idcdfe515,I888be2bd

* changes:
  Revert "ART Test Runner"
  Revert "Fix issues in ART testrunner"

Merge "Revert "Fix a typo in ART testrunner""

Revert "ART Test Runner"

This reverts commit b5a77b91f3b52ff7de5bb3be2fb8c420f6968cb6.

Change-Id: Idcdfe5157624ebfe29a0f9079265368ea184d0f9

Revert "Fix issues in ART testrunner"

This reverts commit e2f7cedae1aaa4d100bcee558ab5e7abf5ef8b5f.

Change-Id: I888be2bd1205e3eac3923695cb3f24c4e858e868

Revert "Fix a typo in ART testrunner"

This reverts commit 542d5b88748d5ae8ef143b2b744321660a0cb92b.

Change-Id: I67d61087b277da0d110237d600478b4255806d23

Merge "Fix a typo in ART testrunner"

Fix a typo in ART testrunner

Test: Runner is working with ART_TEST_ANDROID_ROOT set.
Change-Id: If2a23002bdff9a10f28b5b1b730d879440c3bde6

Merge "Fix issues in ART testrunner"

Fix issues in ART testrunner

Some missed tests added to knownfailure.json file.

Change-Id: Idb7eec4a5080401d5bf00b14c5486293f5c7acf5

Merge "ART Test Runner"

ART Test Runner

The testrunner runs the ART run-tests by simply invoking the script.
It fetches the list of eligible tests from art/test directory, and list of
disabled tests from art/test/knownfailures.json. It runs the tests by
invoking art/test/run-test script and parses it output to check if the test
passed or failed.

Before invoking the script, first build all the tests dependencies by
building 'test-art-host-run-test-dependencies' for host tests,
'test-art-target-run-test-dependencies' for target tests, and
'test-art-run-test-dependencies' for building dependencies for both.
There are various options to invoke the script which are:
-t: Either the test name as in art/test or the test name including the variant
    information. Eg, "-t 001-HelloWorld",
    "-t test-art-host-run-test-debug-prebuild-optimizing-relocate-ntrace-cms-checkjni-picimage-npictest-ndebuggable-001-HelloWorld32"
-j: Number of thread workers to be used. Eg - "-j64"
--dry-run: Instead of running the test name, just print its name.
--verbose
-b To build the dependencies before running the test.

To specify any specific variants for the test, use --<<variant-name>>.
For eg, for compiler type as optimizing, use --optimizing.

In the end, the script will print the failed and skipped tests if any.

Changes in the Android.run-test.mk

1) Test targets like
"test-art-host-run-test-debug-prebuild-jit-relocate-ntrace-cms-checkjni-picimage-npictest-ndebuggable-055-enum-performance64"
2) New targets - test-art-host-run-test-dependencies and
test-art-target-run-test-dependencies have been added. The targets build
dependencies for host and target respectively.
3) Old targets like test-art-host-run-test-optimizing, test-art-host-run-test
build the dependencies using the above targets and then invokes the testrunner
to run the tests.

Test: No tests
Change-Id: I1823aa1cbfe3637a3c6ae8eb5843709fa77cee09

Merge "Revert "build: Generate asm_support_gen.h from the build, validate up-to-date""

Revert "build: Generate asm_support_gen.h from the build, validate up-to-date"

Breaks the build bot:

error: art/tools/cpp-define-generator/Android.bp:41:1: dependency "cpp-define-generator-data" of "cpp-define-generator-asm-support" missing variant "arch:linux_x86_64"

This reverts commit 2449a154c457eaa2c6a30e11184bb7381903cfd5.

Change-Id: I05ae3fa3c56a10238d48c52faa10b8c9c82d9d24

Merge "Revert "Revert "ART: Give JIT thread pool workers a peer"""

Merge "build: Generate asm_support_gen.h from the build, validate up-to-date"

Clean up transaction

Disallow copy constructors, use move constructors, and fix
formatting. Aims to prevent future bugs.

Test: test-art-host

Bug: 34963782

Change-Id: Iaf626e94b14a1fd1b6b64fbd434f5de678e53dc2

Enable thread annotations for std::mutex

Turn on c++ library support for clang thread safety annotations.  This
will allow using std::mutex and related classes directly for
synchronization.

Bug: 22322814
Test: build
Change-Id: Ib8eda18a1f4f0d7a6f99c4d1b1438293b8e22de6

Merge "Don't use ModifyDebugDisallowReadBarrier in transaction mode."

Merge "ART: Refactor verify_object.h"

Don't use ModifyDebugDisallowReadBarrier in transaction mode.

Bug: 34963782
Test: test-art-host
Change-Id: I356eaf9010f1c137bed703326dcb7f641ce7fcfe

Revert "Revert "ART: Give JIT thread pool workers a peer""

This reverts commit 9dfb707ba2f8c2ff67d42c26e3214f5d7142b6d3.

Accept a live Java thread for the JIT, and adjust the tests
accordingly.

Bug: 31684920
Test: m ART_TEST_JIT=true ART_TEST_INTERPRETER=true test-art-host
Change-Id: I92cbae1eaae05711b9069335cf1a5f7eb58b9fd8

Merge "Fix root visiting in transaction.cc"

Fix root visiting in transaction.cc

Using a non reference auto makes a copy of the pair, this resulted in
the GC root never being updated if it moved.

Bug: 34981452

Test: untested

Change-Id: I3ea5d567e7ac016332c27058f9fdb77561c022c1

Merge "Add nullptr check for dex layout when opening the dex file fails."

Add nullptr check for dex layout when opening the dex file fails.

Bug: 34971122
Test: mm test-art-host
Change-Id: Id486d6ecbcbc85bcb8dd0ccd24df45a1c2c07f43

ART: Refactor verify_object.h

Move the actual VerifyObject check into a new cc file, as we
commonly don't enable the check at all. This allows to cut the
-inl include from almost all current users.

This also exposes missing -inl includes. Also fix up some of our old
mess where .h defined functions require -inl.h defined functions.

Test: m

Change-Id: I3dd821bbe2015564a29bf1ed9be00f7a7276ad61

Clean up art::mirror::Class::SetSuperClass.

- Move its definition from runtime/mirror/class.h to
  runtime/mirror/class-inl.h.
- Prevent debug code from running in non-debug mode.

Test: test-art-host
Change-Id: I981cb9aa5c59c75c6a1c4d2145185cf8b147e2db

Merge "ART: Remove unused asm definition"

Code refactoring around sharpening HLoadClass.

Even if the class is not accessible through the dex cache, we
can access it by other means (eg boot class, jit table). So rewrite
static field access instruction builder to not bail out if a class
cannot be accessed through the dex cache.

bug:34966607

test: test-art-host test-art-target
Change-Id: I88e4e09951a002b480eb8f271726b56f981291bd

ART: Remove unused asm definition

CL 331207 added a jit constant to be used in assembly, but ended
up not using it.  Removed here.

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