Force stack dump to diagnose empty checkpoint timeout.

Bug: 33006388
Bug: 12687968
Test: test-art-host
Change-Id: Ida149c98eeb05ca4ad8d0f7ef49228c9b45a6312

Merge "Remove alignment bits in between stack maps"

Merge "Ensure ClassPreDefine returned dex file is on the Classpath"

Merge "Avoid read barriers for ArtMethod::GetDexFile"

Ensure ClassPreDefine returned dex file is on the Classpath

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

Merge "ART: Add OnLoad system classloader search support"

Remove alignment bits in between stack maps

Saves 0.65% of boot.oat size, probably similar on apps. Added
BitMemoryRegion to avoid requiring adding state to StackMap. Added
test to memory_region_test.

Test: clean-oat-host && test-art-host

Bug: 34621054

Change-Id: I40279c59e262bd5e3c6a9135f83e22b5b6900d68

Merge "Don't use VIXL's temp registers in LocationsBuilderARM64::HandleFieldGet."

ART: Add OnLoad system classloader search support

Add support for extension of the system classloader search path
during the OnLoad phase. Add test.

Bug: 34359699
Test: m test-art-host-run-test-936-search-onload
Change-Id: Ie0a5b4bb77999dfae87f955dde40b73ffe053b8e

Merge "Add ARM support for `floor` and `ceil` intrinsics."

Merge "MethodHandles: Support and tests for revealDirect / reflectAs."

Merge changes from topic 'VIXL-update-270117'

* changes:
  ARM: VIXL32: Keep jump table base label within range of adr.
  ARM64: VIXL: Fix breaking changes to ternary operator with Register inputs.

MethodHandles: Support and tests for revealDirect / reflectAs.

We need runtime support to materialize a Constructor, Field or
Method object from a MethodHandle and its associated ArtField
or ArtMethod pointer.

An alternate approach might be to have all handles hold a reference
to their associated Field or Method object (in addition to the
raw ArtField or ArtMethod) but that seems unnecessary given that
these APIs are not expected to be called often.

Test: test-art-host

Change-Id: I9450706b9c30359edebf2e8a7afbc8d89bd68b26

Don't use VIXL's temp registers in LocationsBuilderARM64::HandleFieldGet.

Before this CL, when emitting a volatile field load with a
large offset, it was possible to deplete the pool of VIXL
ARM64 temporary registers (IP0, IP1) in the concurrent
collector configuration.  To avoid this, we now request a
temporary register from the register allocator instead.

Test: m test-art-target-run-test-635-checker-arm64-volatile-load-cc
Bug: 34726333
Change-Id: Idf73a0306142c6133e259783aacaf7ad5401a2fd

Add ARM support for `floor` and `ceil` intrinsics.

Test: ART_USE_VIXL_ARM_BACKEND=true m test-art-host
Test: ART_USE_VIXL_ARM_BACKEND=true m test-art-target

Change-Id: Ia6fe41ea912e32a73d78629c616bc64ad1ecfe66

Merge "Revert "Revert "CHA for abstract methods."""

ARM: VIXL32: Keep jump table base label within range of adr.

Macro assembler instructions between adr and the jump table base label
could cause literal pool or veneer emission, which would push the jump
table start out of adr range. Such a large emission is highly unlikely,
but possible, so prevent this by using an ExactAssemblyScope.

Test: export ART_USE_VIXL_ARM_BACKEND=true && test-art-target
Change-Id: I46a038e452e197d73206b415376b5f2246388317

ARM64: VIXL: Fix breaking changes to ternary operator with Register inputs.

Test: mma art
Change-Id: I33d1e05e47f337f1a9271b35dba9227057cda096

Merge "Shoot in the dark to try and fix mac build."

Shoot in the dark to try and fix mac build.

Use MACRO_LITERAL in assembly macros instead of LITERAL.

Change-Id: I9fd921e4ae59550cfcc1f573be1798ea9823a0cb

Merge "Add per array size allocation entrypoints."

Revert "Revert "CHA for abstract methods.""

This reverts commit 8ebc8bf055e8bd8f6f167e65a69cf4dae136db55.

When we set the single-implementation method for an abstract method, it
need to be protected by the cha-lock.

Test: new testcase and ART_TEST_JIT=true m -j20 test-art-host-run-test
Change-Id: I66acb20ffa7e49dd9c391f001e3bb52f961872e4

Merge "Pass the correct name to ClassFileLoadHook"

Pass the correct name to ClassFileLoadHook

We were passing the dotted name instead of the slashed one.

Test: ./test/run-test --host 937
Change-Id: I4f5f4128cd87ed3d592b7af5724554160f15bdae

Avoid read barriers for ArtMethod::GetDexFile

Shows up in pmd benchmark from Class::FindDeclaredDirectMethod and
Class::FindDeclaredVirtualMethod. There are still calls to
IsProxyMethod that could probably be eliminated.

ReadBarrier::Mark goes from 12.39% to 3.45% according to perf.

Test: test-art-host

Change-Id: I6a4f2fa2d68bf5f393f83b9b70e8d6fcc9dbdaa2

Merge "Re-enable tests 934 & 935"

Merge "You shall know your arm32 calling conventions."

You shall know your arm32 calling conventions.

64bit values are passed starting an even register.

test: ./art/test/run-test --no-relocate 529-checker-unresolved

Change-Id: Ic3012f34898702b48fdab4d7a77dfdd4c2a1d916

Re-enable tests 934 & 935

There were two issues causing problems with these tests.

1) The loading of classes could occur prior to the transformer being
   enabled. This was fixed by putting the classes under test into a
   different ClassLoader and disabling app-images.

2) DexToDex compilation was devirtualizing some calls meaning that
   reordering the methods was causing incorrect dispatch.

This reverts commit 50cc0cceb5ca0abd4fff2cb3467ea74e7590cb10.
This reverts commit e9f36b7cb42348ef5c4eb3c88bd3cfdd56390a3e.
This reverts commit 1e7e96e4a66f668255a1683b837863228489faf9.

Test: ART_TEST_INTERPRETER=true \
      ART_TEST_JIT=true \
      mma -j40 test-art-host

Test: ART_TEST_INTERPRETER_ACCESS_CHECKS=true \
      ART_TEST_RUN_TEST_NO_DEX2OAT=true \
      ART_TEST_RUN_TEST_RELOCATE_NO_PATCHOAT=true \
      ART_TEST_RUN_TEST_NO_RELOCATE=true \
      mma -j40 test-art-host-run-test-934-load-transform \
               test-art-host-run-test-935-non-retransformable

Change-Id: Ia600ba37d45a5f9489b03c7f4d143849c887b8a7

Merge "Make verify-profile not look at the profile."

Merge "Avoid accessing the heap without mutator lock in Monitor::Lock."

Make verify-profile not look at the profile.

It doesn't work well with vdex. In follow-up CLs,
I'll just remove/rename  the filter.

For the record, compiling sheets on an angler:
verify-profile: dex2oat took 5.383s (6.672s cpu)
vdex: dex2oat took 1.686s (1.562s cpu)

bug: 34715556
bug: 34691602
bug: 34691854

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

Merge "Forgot stack alignment on x86."

Forgot stack alignment on x86.

test: ART_TEST_RUN_TEST_NO_RELOCATE=true m test-art-host test-art-target
Change-Id: I1996c0d85228e5cad7bee4387260821d9d209b11

Merge "Forgot this change from last commit."

Forgot this change from last commit.

Test: 529-checker-unresolved
Change-Id: I14b6a8d81a7a885cff36d3361f83d891e3ff9956

Merge "ARM: Update `ArmInstructionSetFeatures` to track ARMv8-A."

Merge "Refactor code for unresolved field entrypoint."

Merge "Also blacklist 934 and 935 for no-relocate."

Merge "Update vixl/arm32 backend after new allocation entrypoint change."

Also blacklist 934 and 935 for no-relocate.

test: test-art-host
Change-Id: I0e88ffffa25b5174eecfc4e0bfebf43811d33959

Refactor code for unresolved field entrypoint.

- Do macro magic to avoid source code duplication.
- Do not fetch the referrer from the assembly, but
  from the C entrypoint instead.

Test: test-art-host test-art-target

Change-Id: Ib139c94bc8f74686640cad538ba75dc56fa00e1d

Update vixl/arm32 backend after new allocation entrypoint change.

Fix after https://android-review.googlesource.com/#/c/325419/

Spotted by artem.serov@linaro.org. Thanks!

test: test-art-target with vixl/arm32
Change-Id: I5fdb4836c80f98696b07a06f52919f33a6284fbc

Add per array size allocation entrypoints.

- Update architectures that have fast paths for
  array allocation to use it.
- Will add more fast paths in follow-up CLs.

Test: test-art-target test-art-host.
Change-Id: I138cccd16464a85de22a8ed31c915f876e78fb04

Merge "Revert "Disable optimization of Atomic::LoadJavaData for x86 targets.""

Merge "Revert "Ignore warning for instantiation after specialization.""

Revert "Disable optimization of Atomic::LoadJavaData for x86 targets."

This reverts commit fbd5118c1c36900910cabe89e3a73d598160e629.

Reason for revert: <INSERT REASONING HERE>

Change-Id: If7aac5e7f4e5376167e896c40f8bf30287f81dd1

Revert "Ignore warning for instantiation after specialization."

This reverts commit 445584d3d1f8e3a9a7d12eded3b2dce5f8691d8c.

Reason for revert: <INSERT REASONING HERE>

Change-Id: I911049f64c4e963cfa47fd8bc7f16ed14e25b057

Merge changes from topic 'clang-3625443'

* changes:
  Disable optimization of Atomic::LoadJavaData for x86 targets.
  Ignore warning for instantiation after specialization.

Merge "Temporarily disable tests 934 & 935 in fallback and interp_ac config"

Temporarily disable tests 934 & 935 in fallback and interp_ac config

These should have been with the earlier disable but I missed them.

Test: build change
Change-Id: I491f14d5666fff30dbadcf9105a96748a4aaaf5e

Merge "Add capability checks to all functions."

Avoid accessing the heap without mutator lock in Monitor::Lock.

ScopedThreadStateTransition doesn't do annotations and we are
accidentally accessing the heap without holding the mutator lock in
Monitor::PrettyContentionInfo. Use ScopedThreadSuspension instead.

Bug: 34674595
Test: test-art-host

Change-Id: If3ec86429ff99405ab14b3588f192ab1c0472cc8

Merge "Make jvmti tests runnable on the RI again."

Add capability checks to all functions.

Also ensure that we declare all capabilities we support.

Bug: 31684920
Bug: 31455788
Test: mma -j40 test-art-host
Change-Id: Ic3f251f451cf6dad9934109594652f23d228d0f4

Merge "Minor edits to make nightly reports."

Merge "[ART] Prepare profile streaming for framework to use."

Merge "Refactor and optimize memory region bit functions"

Merge "ART: Remove experimental flags for plugins and agents"

Merge "Preserve verification invariant after compile-time verification."

Merge "Temporarily disable tests 934 & 935 for interpreter and jit."

Temporarily disable tests 934 & 935 for interpreter and jit.

These need some additional support from the ClassPreDefine event which
will be added shortly.

Test: mma -j40 test-art-host ART_TEST_INTERPRETER=true ART_TEST_JIT=true
Change-Id: Id60da22d96a9f6c9084fdf6bf810874284d5d2a8

Make jvmti tests runnable on the RI again.

Test: ./test/run-test --host --jvm 932
Change-Id: Ia35c9e2b4f93884956993dc9c5e4d041344c9680

ART: Remove experimental flags for plugins and agents

Remove experimental flags guarding plugins and agents.

Partial revert of commit 7233c7e752c0d26387d143ee74420e9cd1f09390.
Partial revert of commit 185d134a3b43ab7529053e965917e0fa74bceba4.

Bug: 31455788
Test: m test-art-host
Change-Id: I11a19f24b88ced8da7a28e2cb7a2f602aab884a8

Refactor and optimize memory region bit functions

Move optimized bit reading from FieldEncoding to MemoryRegion,
added optimized StoreBits to MemoryRegion.

Compilation of a large app on host:
Before:
Time -j1: 31.897s
2.00% art::MemoryRegion::StoreBits(unsigned long, unsigned int, unsigned long)

After:
Time -j1: 29.620s
0.39% art::MemoryRegion::StoreBits(unsigned long, unsigned int, unsigned long)

Bug: 34621054

Test: test-art-host

Change-Id: I0509613da83cc5741d5cfada3f8a8af503784e9e

Merge "Implement class-pre-define time redefinition."

[ART] Prepare profile streaming for framework to use.

Make it possible for framework's Java code to optionally
enable streaming output of profile.

Test: m -j48 ART_TEST_TRACE=true ART_TEST_TRACE_STREAM=true test-art-host
Change-Id: I70f6cf3b72c6cec2ac1f1f1e9c2b2c4727d72e8a

Implement class-pre-define time redefinition.

This allows one to redefine classes as they are being loaded without
restriction as to the types of transformations that may be included.
For example one is allowed to add additional methods or fields to the
class being defined.

Bug: 31684920
Test: mma -j40 test-art-host
Change-Id: I671ee13444c05f28424e727fd80d9d46c78ca287

Preserve verification invariant after compile-time verification.

After the verification step of the compiler, all classes in the
dex file that are being compiled should be either erroneous,
verified, or verify at runtime.

vdex used to break that invariant by only setting the status of
verified classes.

bug: 34691602
bug: 34691854

test: 634-vdex-duplicate, which breaks for the newly added
      EnsureVerifiedOrVerifyAtRuntime check.

Change-Id: I67bd57880ed3238525f36fde484e36ca535ae5d5

Merge "MethodHandles: Write tests for MethodHandle.asCollector."

MethodHandles: Write tests for MethodHandle.asCollector.

Tracks libcore change 0e8de7372de378e00c429dbf9d55526d433a7a21.

Test: make test-art-host

Change-Id: I1d7e33f1cc11104b7b015afd7d49a48f2f2c5312

Merge "MethodHandles: Support and tests for invokeWithArguments."

Merge "Rewrite retrieval of resolved classes."

Merge "Revert "OatFileAssistant: look at vdex file for IsDexOptNeeded""

Merge "Disable test 154 for GC stress"

Merge "ART: Refactor TI events"

Disable test 154 for GC stress

GC stress does too many GCs and may cause the test to fail.

Test: test-art-host -j32 ART_TEST_GC_STRESS=true ART_TEST_INTERPRETER=tru

Change-Id: I7a46d185b718a3faaa74dbc561506e4844b79032

Minor edits to make nightly reports.

Rationale:
bit more output that helps interpreting the nightly results;
also fixed a bit of bitrot in the dexfuzz runners.

Test: nightly fuzzer
Change-Id: I342968bd2b21e86539c4ec03b6a167c7844d2a7c

Revert "OatFileAssistant: look at vdex file for IsDexOptNeeded"

This reverts commit 402089a27b25a9d1112c49189eb83a567719188f.

Leads to crashes when vdex doesn't seem to contain dex files.

Bug: 34339100
Bug: 34685231
Change-Id: I57c3134892a190f792e196e6d7162285e74a64e4
Test: m test-art-host

Merge "Enable PRODUCT_ART_USE_READ_BARRIER by default."

ART: Refactor TI events

Refactor for type safety of event dispatch. Use template specialization
to ensure that only correct calls will work.

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

Rewrite retrieval of resolved classes.

Use ClassLinker::VisitClasses() instead of walking over dex
cache type arrays. The old technique relies on the dex cache
holding references to all resolved classes, an assumption
that we're going to break soon.

Test: m test-art-host-run-test-595-profile-saving
Bug: 30627598
Change-Id: Icae71f53b5a33ff931d83aa8d44cc182889f26da

Merge "Add dexoptanalyzer tool"

Merge "Temporarily increase heap min and max free"

Merge "Fix typo in argument name."

Temporarily increase heap min and max free

Make sure that GC doesn't run too often for TLAB ergonomics. Added a
regression test.

Bug: 34576638

Test: test-art-host

Change-Id: Ie07c7c470aaca044fea20b21cbe6bfe8667d082f

Add dexoptanalyzer tool

dexoptanalyzer is a command line tool which analyzes whether or not a
given dex file needs to be dexopted. It is a wrapper around
OatFileAssistant::GetDexOptNeeded and its results have a mapping to
OatFileAssistant::DexOptNeeded.

This is needed for the management of secondary dex file. We need a way
to retrieve the DexOptNeeded status from installd outside system server.
We can't simply use the system server path (as we do for the main apks)
because system server can't access content writable by the apps.

Bug: 32871170
Test: m test-art-(host|target)
Change-Id: Ife1534560ca2658079d4a38a30bc8d94f35fe9c8

Enable PRODUCT_ART_USE_READ_BARRIER by default.

Bug: 34084559
Bug: 34576638
Test: Boot gce_x86_phone on master
Change-Id: I4ccedf52b71f305792998262bcee4110a05df9cc

Merge "run-test: ThreadSuspendTimeout only applies to ART."

Fix typo in argument name.

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

Merge "Add class status for resolved erroneous classes."

run-test: ThreadSuspendTimeout only applies to ART.

Test: art/test/run-test --jvm works
Test: art/test/run-test --host works
Change-Id: I58f10b6c7b19df0827e108d2da89d873582bafbe

MethodHandles: Support and tests for invokeWithArguments.

Tracks libcore change a8cf0bffdb9e9cf031efd0d3c8b5645d45963562.

Test: make test-art-host
Change-Id: I65fbf3a82b629585324c477bdce6dabd63ae408e

Merge "Revert "Redesign implementation of RegisterNativeAllocation.""

Revert "Redesign implementation of RegisterNativeAllocation."

This reverts commit 6bfc37d875b2556b3f1d95b00a785a09dc94ccb0.

Test failures in 004-NativeAllocations.

Bug: 29156652
Bug: 32576211

Change-Id: Ic54c67caf913024485a5c0621857d68a6fb710fa

Merge "MethodHandles: Tests for asSpreader / spreadInvoker."

Merge "Disable 912 for JIT."

Disable 912 for JIT.

bug: 34655682
test: test-art-host
Change-Id: I80e51d2866dfb47fc725877356168d2c27255bf5

Add class status for resolved erroneous classes.

Split the old ambiguous status mirror::Class::kStatusError
into kStatusErrorUnresolved and kStatusErrorResolved. Once
a class has been resolved, IsResolved() shall return true
even if the class later becomes erroneous. Allow returning
erroneous class from ClassLinker::EnsureResolved() if it has
been previously resolved. This allows consistent behavior
for retrieving classes, immune to multi-threaded races and
multi-dex weirdness. It also allows JVMTI to properly report
"prepared" (i.e. resolved) classes that are also erroneous.

The new behavior is consistent with the RI.

Add regression tests to 008-exceptions for inconsistent
behavior for multi-dex retrieval of erroneous resolved class
(wrapping or not wrapping the old exception based on which
dex file is used for lookup) and for a CHECK(IsResolved())
crash in ClassLinker::LoadSuperAndInterfaces() (without any
tests for similar checks that could have previously failed
only due to extremely unlikely race conditions; these should
now also be fixed).

Inconsistency still remains for class verification as shown
by the new exceptionsForSuperClassInitFailure() test in
008-exceptions, where interpreter and Optimizing still
cause different exceptions to be thrown.

Note: This is partially changing behavior implemented for
bug 28787733. Since we allow the class loader to retrieve an
erroneous resolved class, the ExceptionInInitializerError is
not thrown at all from VMClassLoader_findLoadedClass(), so
there is nothing to wrap in ClassNotFoundException.

Test: m test-art-host
Bug: 30627598
Bug: 28787733
Change-Id: I86cdca00f35a0d6221d2559e3026ac0428a3613c

Merge "Redesign implementation of RegisterNativeAllocation."