Blacklist 130-hprof for tracing configs.

Due to flakiness on art-tracing and art-interpreter-tracing.

Bug: 32383962
Test: test-art-host with ART_TEST_TRACE=true.
Change-Id: I7280c14ec7ce2bd4d5e5a599b4c42762baa07e13

Merge "ART: Blacklist test 908 for target"

ART: Blacklist test 908 for target

As usual, black-list an agent test for the target.

Bug: 31684633
Test: m test-art-host-run-test-908-gc-start-finish
Change-Id: I0c246f7af7942ae098dc1d6829b34562ec4222e5

Merge "ART: Add GC callbacks"

Merge "ART: Only scan tag table once during marking"

ART: Add GC callbacks

Add start and end of pause callbacks.

Fix setup and missing functions in test 907.

Bug: 31684633
Test: m test-art-host
Test: m test-art-host-run-test-908-gc-start-finish
Change-Id: I1d8872ef9cd6914de7fb033cd873f8d5cb48ea17

Merge "Remove unnecessary load class for new instance"

Merge "Move dex CFG dumping out of utils.cc"

ART: Only scan tag table once during marking

Remember an update and avoid multiple scans during the marking
phase.

Bug: 31385027
Test: m test-art-host
Test: m ART_USE_READ_BARRIER=true test-art-host
Change-Id: I82211938965fd32a6d6c2dbf58bf7be2fc688c46

Remove unnecessary load class for new instance

Remove the load class for new instance if the load class has only one
use and can not throw. Previously many were not removed due to
MarkInDexCache nulling out the environment of the HLoadClass and
causing CanMoveClinitCheck to fail.

Also keep track of initialized HLoadClass and always remove clinit
checks for these.

Added checker regression test.

Code size savings: ARM64 CC boot.oat: 47896936 -> 47642488 (-0.53%)
Savings from IsInitialized optimization: 65984 bytes

Performance unmeasured, probably faster due to removing unnecessary
work.

Test: test-art-host with CC baker

Bug: 29516974

Change-Id: I43358762ffb380ebe7e6518d0d440a5e1cc03b61

Move dex CFG dumping out of utils.cc

Move CFG dumping to dexdump, the only client.

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

Merge "Dex disassembly suppport for invoke-polymorphic."

Merge "Add support for proguard deobfuscation."

Merge "ART: Switch tagging table to a map"

Merge "Remove read barrier in UnstartedUnsafeCompareAndSwapLong."

ART: Switch tagging table to a map

Performance is critical. A map involves overhead for moving GC,
but has much faster lookup for the common case.

Make test 905 robust against unstable ordering.

Bug: 31385027
Test: m test-art-host
Test: m ART_USE_READ_BARRIER=true test-art-host
Change-Id: Ica3ff603fc78168759fccfe79c97860279ce9036

Merge "Reduce the number of dumps in 130-hprof."

Reduce the number of dumps in 130-hprof.

To avoid timeouts.

Test: test-art-host with ART_TEST_TRACE=true.
Change-Id: I849007b94341f4851c11a8896889f9e2ce851a86

Merge "Add missing shared library"

Add missing shared library

Host builds were compiling without -Wl,--no-undefined.  Add missing
shared library so that -Wl,--no-undefined can be added again.

Test: m -j host
Bug: 32305815
Change-Id: I39456a95c37b64b6ac65c0b3220d4816ce34e5c6

Merge "Don't dump loaded but unresolved classes in hprof"

Merge "Remove mutex dependency on art::Runtime"

Add support for proguard deobfuscation.

Test: m ahat-test, with obfuscation of test-dump.hprof added.
Bug: 25812772
Change-Id: I0f257432da570aa146e6bb5c549f014aecc0c326

Remove read barrier in UnstartedUnsafeCompareAndSwapLong.

art::interpreter::UnstartedRuntime::UnstartedUnsafeCompareAndSwapLong
manipulates a long field and does not need a read barrier
(read barriers are only required when loading an object
reference from the heap).

Test: ART_USE_READ_BARRIER=true make test-art-host
Bug: 12687968
Change-Id: Ia46590e649a7bb46ca4b3c90bab03fb3a836563d

Merge "Disable kBssEntry LoadString sharpening."

Merge "MIPS32: Implement HSelect"

Merge "MIPS32R6: Fix floating point compares in mterp."

Disable kBssEntry LoadString sharpening.

We seem to fail to visit the BSS GC roots sometimes.

Bug: 32124939
Test: m test-art-host
Change-Id: I4724ced649042cbeb170cac9e8da8c5db168c8fc

MIPS32R6: Fix floating point compares in mterp.

This fixes failures in the following tests:
003-omnibus-opcodes
082-inline-execute
083-compiler-regressions
107-int-math2
422-type-conversion
570-checker-select

Test: 'run-test --interpreter <tests above>'
      in QEMU for MIPS64R6/MIPS32R6

Change-Id: I11b4f1df7c212932c92d41fbc98c00465d03e850

Don't dump loaded but unresolved classes in hprof

If a class is loaded but not yet resolved, it will cause failed
DCHECK when reading static fields.

Bug: 32008990

Test: test/run-test --host 130

Change-Id: I05f549cb2b70c75b06ed23d3405373d15ad7707b

Merge "Improve recognition of select-based period induction."

MIPS32: Implement HSelect

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: I8a8127d8d29cb5df84ed6f4fd4478f8d889e5cb7

Improve recognition of select-based period induction.

Rationale:
Similar to the previous CL, this helps to eliminate more dead induction.
Now, CaffeineLogic, when compiled with dx (rather than jack) improves
by a 1.5 speedup (9000us -> 6000us).

Note:
We need to run the simplifier before induction analysis to trigger
the select simplification first. Although a bit of a compile-time hit,
it seems a good idea to run a simplifier here again anyway.

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

Merge "Test for the CC RegionSpace::WalkInternal crash."

Remove mutex dependency on art::Runtime

Breaks the cyclic dependency between mutex and the runtime.  This allows
the use of mutexes without instantiating a runtime.

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

Merge "Ensure that overrides work in presence of package-private methods."

Dex disassembly suppport for invoke-polymorphic.

Bug:  30550796
Test: art/test/dexdump/run-all-tests
Change-Id: I013ce2ebbcf9555e01170dc47fc38036c276b1b4

Merge "Disable 151-OpenFileLimit test."

Disable 151-OpenFileLimit test.

Until we can track down all the crashes it leads to.

Bug: 32302133
Test: mma -j32 ART_TEST_JIT=true ART_TEST_INTERPRETER=true test-art-host-run-test-151-OpenFileLimit
Change-Id: I515a5175b4cefe49a8e270d354879050b1f616f9

Merge changes I16f8b7ec,I075bbf55

* changes:
  jni: Support @FastNative methods that return objects
  Revert "jni: Disable FastNative path for methods returning objects"

Merge "Revert "Revert "Interpreter: Add support for direct handle invokes on methods."""

Merge "ARM: VIXL32: Pass initial ART tests with new code generator."

Merge "DexCache: Add support for updating MethodType references during GC."

Merge "Return error message if IndirectReferenceTable construction fails."

Revert "Revert "Interpreter: Add support for direct handle invokes on methods.""

This reverts commit cfa61ad52077df66a448b84c360b12bc6f0e3f51.

Test flakiness in 956-methodhandles is fixed by
d08e39b6f02368aaa668b5aae6b6077b3eb44d9c.

Test: make test-art-host

Change-Id: I56e02e9a5bbc2b992cf746a92fd95ea77d32456c

DexCache: Add support for updating MethodType references during GC.

Fixes flakiness in 956-methodhandles

Test: make test-art-host
bug: 30550796
Change-Id: Ia5359390d59e65fc6efc16cc55c11bc65029104d

Merge "Change pResolveString entrypoint to kSaveEverything."

Merge "Build: Avoid using ppid to generate temporary directory name."

Merge "ART: Harden dex-file verifier wrt/ LEB128"

Merge "Persistent bitmap for region space"

ART: Harden dex-file verifier wrt/ LEB128

Use out-of-bounds checks when decoding (U)LEB128.

Bug: 31252966
Test: m test-art-host
Change-Id: Id6dbbceb37c4d8d698e7685c61bf31912e3bf69e

Ensure that overrides work in presence of package-private methods.

It was possible that methods with the same signature & name of
package-private methods could fail to be correctly overridden causing
surprising behavior and DCHECK failures.

Bug: 32193118
Test: mma test-art-host
Test: ART_TEST_RUN_TEST_NDEBUG=true ART_TEST_RUN_TEST_NO_PREBUILD=true mma test-art-host-run-test-300-package-override
Change-Id: I8f53a830cd8d4210f60e9827e525c779a0696e04

Persistent bitmap for region space

Fix bug where region space was not safely walkable due to holes
in the unevac regions possibly having dangling class pointers.

No preformance change, RAM overhead 1.1-1.2% .Heap (non LOS).

Test: test-art-host
Test: https://android-review.googlesource.com/#/c/288907/

Bug: 31522820

Change-Id: Ic4f8b7175e117689cb1ce3e28b082cf63f1f7b5a

Merge "Move most of class linker to ObjPtr"

Merge "Fix bug in dynamic BCE. With regression test."

jni: Support @FastNative methods that return objects

Bug: 32088975
Change-Id: I16f8b7ec6b251812af60ab25f2153d9b72f37044

Revert "jni: Disable FastNative path for methods returning objects"

This reverts commit 6f029d9cfae142f318e3986331b04695f3ea524a.

Change-Id: I075bbf55a07565118b8fa529a397bb42490e6e51

Fix bug in dynamic BCE. With regression test.

Rationale:
We implicitly assumed anything on a fixed offset to a suitable
candidate could be picked for codegen. Although this is usually
true, the scanner found an exception (on types). So, it is best
to make the test explicit. Note: some type implications are TBD.

Bug:32193474

Test: test-art-host and offending APKs
Change-Id: I664a2caa97a7f231509f8832b940dd72da11a372

Move most of class linker to ObjPtr

Return values are still mirror pointer. Fix some failing asserts in
compiler driver and tests.

Bug: 31113334

Test: test-art-host

Change-Id: I4450bf9dfb2541749496b8388616e8aae8488919

Change pResolveString entrypoint to kSaveEverything.

Test: Run ART test suite including gcstress on host and Nexus 9.
Test: Run ART test suite including gcstress with baker CC on host and Nexus 9.
Bug: 20323084
Change-Id: I63c21a7d3be8ff7a5765b5003c85b5317635efe6

Merge "Delete unused blocked_register_pairs_ in code generators"

Merge "Remove mirror:: and ArtMethod deps in utils.{h,cc}"

Return error message if IndirectReferenceTable construction fails.

Previously if there was an error when constructing the
IndirectReferenceTable, the error message was lost. Now expose and
include the error message when throwing an exception related to
failures to construct the IndirectReferenceTable.

The error message is propagated through JVMEnvExt, JavaVMExt, and
Runtime::Init as well.

Bug: 32013594
Test: Added new 151-OpenFileLimit runtest.
Test: m test-art-host, m test-art-target

Change-Id: I3692f6928c9570358571bce634569d6f14cdeb05

Merge "Revert "Interpreter: Add support for direct handle invokes on methods.""

Build: Avoid using ppid to generate temporary directory name.

When TMPDIR is left unset, use the BSD checksum of
$(ANDROID_BUILD_TOP)-$(USER) instead of the PPID to construct a root
directory for all test runs. This workaround lets us run tests from
multiple repos at the same time, and will also avoid having to deal
with path length issues on the build bots.

This is a hacky workaround, but the only proper fix is stop using a
test runner built on top of make.

Bug: 32165422
Test: test-art-host

Change-Id: I61bc1eb3cf57d498f910573926d826a36020067a

Revert "Interpreter: Add support for direct handle invokes on methods."

This reverts commit e5cd2cd7d0f8e1332f25edfd2798d84fec871f10.

Reverting due to flakiness in 956-methodhandles. Flakiness will be
fixed by 67f0929710a4489e3603383e57c9d236920654aa but might take time
to submit.

Change-Id: Ifdda5d3caca62d16d21cefaab7742fb2e8ca6969

Merge "Remove libcore tests failures expectations for the concurrent collector."

Merge "Fix string compression, add tests."

Merge "Interpreter: Add support for direct handle invokes on methods."

Merge "Blacklist test that has been flaky for days."

Fix string compression, add tests.

Fix String.compareTo() for interpreter; memcmp() does not
return the required result (only the right sign).
Fix x86-64 stub where the assembler silently accepted and
generated bad code for out-of-range JECXZ.
Add extensive tests for String.equals(), String.compareTo()
and String.indexOf().

Bug: 31040547
Test: Run ART test suite including interpreter tests on host and Nexus 9.
Test: Ditto with string compression enabled.
Change-Id: I21b7a74da8a577c8fbaf8d9225f048550236d414

Merge "Release version 0.8 of ahat."

Blacklist test that has been flaky for days.

Change-Id: Ic3f4a10568b03f02c2173fd18e7642bb9ae4e2b3

Merge "ART: Remove IRT cruft"

Remove mirror:: and ArtMethod deps in utils.{h,cc}

The latest chapter in the ongoing saga of attempting to dump a DEX
file without having to start a whole runtime instance.  This episode
finds us removing references to ArtMethod/ArtField/mirror.

One aspect of this change that I would like to call out specfically
is that the utils versions of the "Pretty*" functions all were written
to accept nullptr as an argument.  I have split these functions up as
follows:
1) an instance method, such as PrettyClass that obviously requires
this != nullptr.
2) a static method, that behaves the same way as the util method, but
calls the instance method if p != nullptr.
This requires using a full class qualifier for the static methods,
which isn't exactly beautiful.  I have tried to remove as many cases
as possible where it was clear p != nullptr.

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

Delete unused blocked_register_pairs_ in code generators

Legacy code for compatibility with quick?

Test: test-art-host CC
Change-Id: I9de261daea67dfd9bd3df89826ba9d10f135e29e

Merge "Update the Begin() of LargeObjectMapSpace"

Remove libcore tests failures expectations for the concurrent collector.

Test: art/tools/run-libcore-tests.sh --mode=host --variant=X32 --debug (with ART_USE_READ_BARRIER=true)
Bug: 12687968
Change-Id: I470f11d0bc6e38011c5b8f3797f2c0e9ec1e56cc

Merge "Enable last value generation of periodic sequence."

Merge "Move art/native to ObjPtr"

Merge "More store/allocation elimination for singletons in case of loops"

Merge changes If0d5eeeb,Id927f2b5

* changes:
  ART: Fix and extend test 904
  ART: Better handle multiple event enables

Merge "ART: Expect a null self in SetEntrypointsInstrumented"

ART: Expect a null self in SetEntrypointsInstrumented

Follow-up to commit 157c77eb9a5d6a7fe6ebcc29afa7f1913a877c87.

Bug: 31684277
Test: m ART_TEST_GC_STRESS=true test-art-host
Change-Id: I9de848ea6a9484798b75a594f3e390ef27394743

Interpreter: Add support for direct handle invokes on methods.

Constructors are still not supported and will be implemented in a
follow up change.

Also adds a new run-test that compiles from source using an alpha
version of Jack.

Test: make test-art-host
bug: 30550796
Change-Id: I3212c032a5885854f691156c29f12613e5194667

Enable last value generation of periodic sequence.

Rationale:
This helps to eliminate more dead induction. For example,
CaffeineLogic when compiled with latest Jack improves with
a 1.3 speedup (2900us -> 2200us) due to eliminating first
loop (second loop can be removed also, but for a later
case). The currently benchmarks.dex has a different construct
for the periodics, however, still to be recognized.

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

Release version 0.8 of ahat.

Test: m ahat-test
Change-Id: I964f86fc66f901bff7ddeda81dd42a6e658fda9d

ART: Fix and extend test 904

Fix using the wrong thread when unregistering. The inner thread
should not use Thread.currentThread (copy-n-paste mistake).

Add test for registering in OnLoad.

Add test for keeping registered on exit.

Bug: 31684277
Bug: 31684920
Test: m test-art-host-run-test-904-object-allocation
Change-Id: If0d5eeeb864684ab42eeaf62698d1e639f2071d6

Merge changes I7061c450,Ie8f8f24b,Ia6cee2cb

* changes:
  ART: Do not free allocation listener in heap destructor
  ART: Fix allocation entrypoint instrumentation
  ART: Fix lock order issue in allocation tracking

Merge "Revert "Revert "Revert "Temporarily blacklist 955-method-handles-smali.""""

Merge "Fix handling of dex cache arrays for method types."

Update the Begin() of LargeObjectMapSpace

Root cause:
  LargeObjectMapSpace's begin_ is initialized by nullptr.
  so, it's always 0 after std::min

Solution:
  Consider the nullptr case when update begin_

Notes:
  Remove the nullptr check for end_ since it's not needed

Test: make test-art-host-gtest-large_object_space_test

Change-Id: I9918c30916e5d359f0ab23ec2bdb0fc9f093a85f

ART: Remove IRT cruft

Remove dead code for initial vs maximum table size.

Bug: 32125344
Test: m test-art-host
Change-Id: Ie5806da7f5f3238483da918deca5982f01764466

ART: Better handle multiple event enables

Only do extra work like allocation listener registration when the
global state changes.

Bug: 31684920
Test: m test-art-host
Change-Id: Id927f2b504e02f2d68bc6e4af2658a6017920d8a

ART: Do not free allocation listener in heap destructor

Do not deallocate on exit. Libraries are freed only after the heap
has already been destroyed.

Bug: 31684277
Bug: 31684920
Test: m test-art-host
Change-Id: I7061c4500a9ad314e2c00d7d722bb3372dff4810

ART: Fix allocation entrypoint instrumentation

Fix the unstarted-runtime case. Special-case the reset call to
the current thread, only.

Bug: 31684277
Test: m test-art-host
Change-Id: Ie8f8f24b2a6041344266a4e6b6bf8ddd9a0352a5

ART: Fix lock order issue in allocation tracking

(Un)instrumenting requires the instrumenting lock, which must be
acquired before the mutator lock. As the plugin can be called both
with and without the lock held, be careful. For simplicity, acquire
the lock (potentially) and immediately suspend.

Bug: 31684277
Test: m test-art-host
Change-Id: Ia6cee2cbe90f13f5543bdfea815d469b28a0f8f4

Move art/native to ObjPtr

Bug: 31113334

Test: test-art-host

Change-Id: I67eb89cf042c762c6dcd5eb8b008b9a28e9b3319

Merge "event interface going private"

Merge "Remove logging dependency on runtime"