Merge "ART: Refactor class-linker methods"

Merge "ART: Initialize field in IndirectReferenceTable"

Merge "Fix race with LOS Begin() and End()"

Merge "Enable agent attaching during live phase"

Enable agent attaching during live phase

This is the ART part, the plumbing from VMDebug_attachAgent() to
actually loading the agent into the runtime.

Test: m test-art-host

Bug: 31682382

Change-Id: I3ccc67aa050c1f78278882128983686ed44ddec2

Merge "Revert "ART: Temporarily use global references for GetLoadedClasses""

ART: Initialize field in IndirectReferenceTable

Initialize last known previous segment state.

Bug: 32125344
Test: m test-art-host
Test: m valgrind-test-art-host
Change-Id: I894f467a9f1dae8de55c1e113f0b00a127b13f33

Fix race with LOS Begin() and End()

There was a race for the first large object allocation that cause
callers of Begin() and End() to see a null End() and non-null
Begin(). The fix is to hold the lock and get both Begin() and End().

Bug: 32387879

Test: test-art-host CC

Change-Id: I6173bf3a55d3ba017ffa5b5e9f566025c65b7555

ART: Refactor class-linker methods

Hide the LookupClass with hash version. Clients should not have to
know about that performance detail.

Hide FindClassInPathClassLoader. This is an implementation detail.

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

Merge "Clear (madvise) card table for CC"

Revert "ART: Temporarily use global references for GetLoadedClasses"

This reverts commit d2d0353de9e5c10f2a4a9ff2102d2b777c134a37.

The IndirectReferenceTable now allows resizing for local references.

Bug: 31684578
Bug: 32125344
Test: m test-art-host
Change-Id: I47dab19eb7ae66fc7d55d202a13a87f508c35e03

Merge "Remove workaround for repo."

Merge "Let dexdump prompt filename for zipped multi-dex files."

Merge "Fix the Thumb-2 definition of the CLREX instruction."

Merge changes Iae3a933e,I08ff5d6e

* changes:
  ART: Make IndirectReferenceTable resizable
  ART: Change IndirectReferenceTable

Merge "ART: Clean up IndirectReferenceTable"

Merge "Clean up app image .bss dex cache arrays fixup."

Fix the Thumb-2 definition of the CLREX instruction.

Test: make test-art-host-gtest-assembler_thumb2_test
Change-Id: I433fa35451aa944e300d5d582dc16b2b9a8bcfb3

Merge "Build target which faciliates ART testing image/image creation"

Merge "Clear OOME from DexCache allocation when another thread succeeded."

Build target which faciliates ART testing image/image creation

Bug: 32365426

Test: m art-job-images
Change-Id: I19f74b7880e8b88c20fcd26729ec05a48b618dbb

Merge "Revert "Disable kBssEntry LoadString sharpening.""

Clean up app image .bss dex cache arrays fixup.

Do not store type and method array forwarding address
in old arrays. They are available from the DexCache.

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

Clear OOME from DexCache allocation when another thread succeeded.

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

Merge "method_handles: Minor refactor of PerformArgumentConversions."

Remove workaround for repo.

The chromium buildbots do not have the issue anymore.

Change-Id: I9f9cf4505dda027fb23883f3f4b311d213b15199

Revert "Disable kBssEntry LoadString sharpening."

Fix .bss GC root walking by registering the oat file
with the class loader's class table.

Also fix potentially outdated ObjPtr<> use in debug build.

This reverts commit b55fdbb30b3bc4e334c241153b98c0a6ea4a4a2b.

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

Merge "Encode quickening info in .vdex."

ART: Make IndirectReferenceTable resizable

Allow backing table to be resized. This can be used for the local
reference table, where synchronization is not an issue.

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

Clear (madvise) card table for CC

Since the cards are not really used, we can madvise them to reduce
RAM. Image and zygote cards are cleared in pause to prevent races
between with mutators.

Pause time goes up by only 1.5us on N6P maps:
(Paused)ClearCards: Sum: 755us 99% C.I. 0.250us-28us Avg: 1.540us Max: 28us

AOSP N6P before (60s after boot system wide CC):
4,194K: .GC
4,172K: .GC
4,110K: .GC

After:
3,253K: .GC
3,205K: .GC
3,245K: .GC

Bug: 12687968

Test: test-art-host

Change-Id: I3194b5b8044c2dca427302c32d9974920fecb289

ART: Change IndirectReferenceTable

Change cookie structure to allow for more entries. Use a local
hole-count caching scheme. The design is driven by two
considerations. For one, the change is small and mostly local.
The other point is to still allow inlining of functions involved
with JNI transitions.

This change is in preparation for a resizable backing table for
"unlimite" local references.

micro_native tests show changes are in the noise.

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

ART: Clean up IndirectReferenceTable

Introduce constants and move some functions into the
IndirectReferenceTable class.

Slightly change IndirectRef encoding to be more obvious (and slighly
more optimized when decoding).

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

Let dexdump prompt filename for zipped multi-dex files.

Test: test-art-host-dexdump test-art-host-gtest-dexdump_test

Bug:32365197
Change-Id: I25a274da93cd3317fd5b18ad578ac4dcad1b83a1

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"

Encode quickening info in .vdex.

We quicken vdex files for performance reasons, but when taking an OTA,
we need to revert the quickening to the original instructions. As vdex
should be independent of the oat file and oat file versions, we encode
the quickening data in the vdex.

test: m test-art-host-jit m test-art-host-gtest
bug:30937355

Change-Id: I9a543a161b70aa1cff99f8fe6f5b5cab7a6c4d31

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

method_handles: Minor refactor of PerformArgumentConversions.

Separate out a method that can perform a single argument conversion.
Useful for Field setters and also for future use in transforming handles
where the input source will not be a stack frame.

Also remove unnecessary JValue* argument.

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

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