Merge "Revert^3 "Remove unused code for patching oat files.""
am: fcb6e39157

Change-Id: Ia38bf92b639e10460b33e7bb7e789bf58f7f06d9

Merge "Revert^3 "Remove unused code for patching oat files.""

Revert^3 "Remove unused code for patching oat files."

This reverts commit e7ec8b25dd6ac1b8ad8a68f808048bd6a61138f0.
Because tests are failing.

Bug: 33192586

Change-Id: I3e575c6c037873195c2177241153e3646d58e8a6

Merge "ART: Improve class initializer and constructor verification."
am: 9e8c8ca2d4

Change-Id: I00c27affa667472bf42ff59692a346efdc732688

Merge "ART: Improve class initializer and constructor verification."

Merge "Revert "Revert "Remove unused code for patching oat files."""
am: ee64b4d83a

Change-Id: I2b61ea5c8bc7518544ca7a42dda18a22553c6533

Merge "Revert "Revert "Remove unused code for patching oat files."""

Merge "Revert "ART: Start RuntimeCallbacks""
am: 44751efea1

Change-Id: I4345c4a2ae676c1a9d66f478a3bd6e369bfbd4ef

Merge "Revert "ART: Start RuntimeCallbacks""

Revert "ART: Start RuntimeCallbacks"

jdwp tests failing with:
    STDERR> dalvikvm32 E 01-18 23:14:50 12180 12198 thread-inl.h:137] holding "runtime callbacks lock" at point where thread suspension is expected

Bug: 31684920

This reverts commit 13093d455b8266338fd713b04261c58e9dc2b164.

Change-Id: I94e4154e273d006eecdd485607dcfd96392d6a00

ART: Improve class initializer and constructor verification.

DEX file verifier checks additional properties of class initializers
and constructors:

(i) Names match expected <clinit> / <init>.
(ii) The method descriptor for <clinit> is ()V.
(iii) The return type of <init> is V.
(iV) No other names start with '<'.

Bug:  31313719
Change-Id: I60bffa6561e1bae353f97c42377ea556bfa790af
Test: m test-art-host-gtest-dex_file_verifier_test

Merge "Revert "Load the array class in the compiler for allocations.""
am: b0355130e3

Change-Id: I4a49af6e60d2efe41dfaee395ca93fc7b0ceee52

Merge "Revert "Load the array class in the compiler for allocations.""

Merge "ART: Note CC configuration down into oat files"
am: 0157eee15c

Change-Id: I3f8c004ee7e39ee127630d627c88a35f84f413c2

ART: Note CC configuration down into oat files
am: f8cd890d86

Change-Id: I9576d4b8f9a2d86b83d6f77ef41ab51a48d022d9

Revert "Load the array class in the compiler for allocations."

libcore test fails.

This reverts commit cc99df230feb46ba717252f002d0cc2da6828421.

Change-Id: I5bac595acd2b240886062e8c1f11f9095ff6a9ed

Merge "ART: Note CC configuration down into oat files"

Merge "Split ArtJvmtiEvent::kClassFileLoadHook in two."
am: 1a901eff5e

Change-Id: Ia6e13ca4b2dfa2258f29e806d3d10efe04b7248e

Merge "Split ArtJvmtiEvent::kClassFileLoadHook in two."

ART: Note CC configuration down into oat files

To detect changes in configuration, write the runtime configuration
of ART_USE_READ_BARRIER into the oat file key-value store.

Bug: 34084559
Bug: 12687968
Test: m test-art-host
Test: m ART_USE_READ_BARRIER=true test-art-host
Change-Id: I0b2bd9aa5546538e2b4b669b0acc0a4bebfd7bf0

Merge "ART: Start RuntimeCallbacks"
am: a7e64f00df

Change-Id: I34c7a98c33de5e03ac61ac8f43e387691a231888

Merge "ART: Start RuntimeCallbacks"

Split ArtJvmtiEvent::kClassFileLoadHook in two.

Since the ClassFileLoadHook event is sent to different environments
based on when it is invoked we split the event in two behind the
scenes. The event dispatcher is responsible for making sure that
either or both of the appropriate underlying events are invoked when a
JVMTI_EVENT_CLASS_FILE_LOAD_HOOK is sent.

We also make sure to modify the EventHandler so it sends the correct
events in the correct places when an environment changes its
capabilities.

Bug: 32369913
Bug: 31684920

Test: mma -j40 test-art-host

Change-Id: I82567fc66debe0b658e8d7fced6284a8c4355b7a

Merge "Refactor jvmtiEvent so we can add to it."
am: 5d61614eda

Change-Id: I44a8c4dc8556811cfab56759dd593599347eddc8

Merge "Refactor jvmtiEvent so we can add to it."

Merge "Fix check failure in ConcurrentCopying::IssueEmptyCheckpoint()."
am: 3f38eb44ee

Change-Id: I853281d378a35f7021cb77f739baa61db1fe0542

Merge "Fix check failure in ConcurrentCopying::IssueEmptyCheckpoint()."

ART: Start RuntimeCallbacks

Add a central RuntimeCallbacks structure to handle certain interesting
runtime events.

In a first iteration, add ThreadLifecycleCallback with ThreadStart and
ThreadStop. Move Dbg over to ThreadLifecycleCallback.

Add a test.

Bug: 31684920
Test: m test-art-host-gtest-runtime_callbacks_test
Test: art/tools/run-jdwp-tests.sh --mode=host
Change-Id: Ie0f77739a563207bfb4f04374e72dc6935c40b4f

Merge "ART: Disable test 901 on target"
am: d79ae05256

Change-Id: Ide403397b51e960cdcb8d912daf1515cd8924e46

Merge "ART: Disable test 901 on target"

Refactor jvmtiEvent so we can add to it.

Change all code over to using ArtJvmtiEvent from jvmtiEvent so that we
have a way to add additional internal events if the need arises.

Bug: 32369913
Bug: 31684920

Test: mma -j40 test-art-host

Change-Id: I555475048d2e753db8adf317dc1f66d81533cde0

Merge changes Ib9fafbac,I05d73734,I0e95b0cb
am: 66d2d484a0

Change-Id: I7474d04d0d4fa60330356c7b150e5dadc7af2540

ART: Add JNI API
am: 6f8e4f0fc3

Change-Id: I5ed8cbb5a205338d23f11aeb93dcb13eaa726a19

ART: Align jvmti.h with jni.h
am: 0bd1a16e91

Change-Id: Iccada3f86267639aba62ce0d84672da30cd28184

ART: Add JNI function table manipulation
am: c8089540cc

Change-Id: I646c38e901ceec9d0e76d9a0d1730a34033976ee

Merge changes Ib9fafbac,I05d73734,I0e95b0cb

* changes:
  ART: Add JNI API
  ART: Align jvmti.h with jni.h
  ART: Add JNI function table manipulation

ART: Disable test 901 on target

As this is now doing slightly more work, disable like the other
tests.

Bug: 31681198
Test: m test-art-target
Change-Id: I73c9f2da1024f8eabc67aba12659d34878cec1f1

Merge "ART: Add Search API"
am: d4a2827896

Change-Id: I97932f86195bbcc79490dfdc6661b82c003afca2

Merge "ART: Add GetJLocationFormat"
am: 24e2ded0f9

Change-Id: Idb6bef4d3f6d5f1af10059f11a3f48df5154a408

ART: Add JNI API

Add support for GetJNIFunctionTable and SetJNIFunctionTable. Add
tests.

Bug: 34343708
Test: m test-art-host-run-test-928-jni-table
Change-Id: Ib9fafbac2781c570aa6eacf5242afbbf6c3ee7a9

ART: Align jvmti.h with jni.h

Android follows the JNI specification and names the JNI function
table without the underscore.

Bug: 34343708
Test: m test-art-host
Change-Id: I05d73734545f9077ad66aceb335bdc8a76603a71

ART: Add JNI function table manipulation

Add support for a function table override. This will override the
decision between the regular and the CheckJNI function tables, if
set.

Bug: 34343708
Test: m test-art-host-gtest-jni_internal_test
Change-Id: I0e95b0cbd21f4efdcd8c3d312781d9aeeff54a1e

Merge "ART: Add Search API"

Merge "ART: Add GetJLocationFormat"

Merge "Load the array class in the compiler for allocations."
am: c3b7bf3bcc

Change-Id: I3fa61cd9e27138c4a6fdbc0aff0fe0959c147bf2

Merge "Load the array class in the compiler for allocations."

Revert "Revert "Remove unused code for patching oat files.""

This reverts commit 78985875702c52657066da158c0246ef07115385.

Bug: 33192586
Test: m test-art-host
Test: m test-art-target
Test: device boots

Change-Id: I848b39a2b9b3c48beca2c68f9a316626a81abb39

Load the array class in the compiler for allocations.

Removing one other dependency for needing to pass
the current method, and having dex_cache_resolved_types_
in ArtMethod.

oat file increase:
- x64: 0.25%
- arm32: 0.30%
- x86: 0.28%

test: test-art-host, test-art-target
Change-Id: Ibca4fa00d3e31954db2ccb1f65a584b8c67cb230

Merge "Fix Mips build."
am: 2ee01bff00

Change-Id: I53c80cf9ff1335e7751553e43654768affd584fe

Merge "Fix Mips build."

Merge "Fix IP register usage in LoadClassSlowPathARMVIXL."
am: 404b012061

Change-Id: Id20492f64e0e41cc083cc1050c5305c4989bf62c

Merge "Fix IP register usage in LoadClassSlowPathARMVIXL."

Fix Mips build.

Test: build mips
Change-Id: Icadff73d6a266bbdfd6cb67a66f04f559f28846d

Fix IP register usage in LoadClassSlowPathARMVIXL.

Request a temporary from a UseScratchRegisterScope instead
of explicitly specifying the IP register to avoid assetion
failure in VIXL.

Test: Build aosp_flounder-userdebug with ART_USE_VIXL_ARM_BACKEND=true
Bug: 34340177
Bug: 30627598
Change-Id: Ie39fbbb66165d752634cde8d2963129641558b93

Merge "Entrypoints cleanup."
am: 4507fdcb70

Change-Id: I8dcbe8868f6fe7f040d6d72ca4837cec90167586

Merge "Entrypoints cleanup."

Merge "Blacklist tests for flaky no-image."
am: 419b7f558d

Change-Id: I3452b516b701c8539b682226a96e7075075e20e2

Merge "Blacklist tests for flaky no-image."

Entrypoints cleanup.

Remove unused ones to facilitate the transition to compressed
dex caches.

test: test-art-host, test-art-target
Change-Id: I1d1cb0daffa86dd9dda2eaa3c1ea3650a5c8d9d0

Blacklist tests for flaky no-image.

bug: 34369284
Change-Id: I09db432db8d99223f4f41e2d7bda7927ad5af9d5

Merge "Reduce using ArtMethod's dex_cache_resolved_types_."
am: a397458175

Change-Id: I1cc9a1b85d29af239222779983123acb2efe7d0a

Merge "Reduce using ArtMethod's dex_cache_resolved_types_."

Merge "MIPS: Fix polymorphic invokes."
am: c66a76f94e

Change-Id: I018f136295de9af5b0575d092bff1a538768bb57

Merge "MIPS: Fix polymorphic invokes."

MIPS: Fix polymorphic invokes.

This fixes MIPS-specific optimizing compiler bugs introduced in
commit ac141397dc29189ad2b2df41f8d4312246beec60.

The below tests still fail in interpreter mode and this will be
addressed separately.

Test: booted MIPS32R2 in QEMU
Test: ./run-test --optimizing 953-invoke-polymorphic-compiler
Test: ./run-test --optimizing 955-methodhandles-smali
Test: ./run-test --optimizing 956-methodhandles
Test: ./run-test --optimizing 957-methodhandle-transforms
Test: ./run-test --optimizing 958-methodhandle-emulated-stackframe
Test: ./run-test --optimizing 959-invoke-polymorphic-accessors
Test: booted MIPS64 (with 2nd arch MIPS32R6) in QEMU
Test: same tests as above and
Test: ./run-test --optimizing --64 953-invoke-polymorphic-compiler
Test: ./run-test --optimizing --64 955-methodhandles-smali
Test: ./run-test --optimizing --64 956-methodhandles
Test: ./run-test --optimizing --64 957-methodhandle-transforms
Test: ./run-test --optimizing --64 958-methodhandle-emulated-stackframe
Test: ./run-test --optimizing --64 959-invoke-polymorphic-accessors

Change-Id: I6119e8b86448e022f68b10cd3a8912a2bd1976c5

ART: Add Search API

Add initial support for AddToBootstrapClassLoaderSearch and
AddToSystemClassLoaderSearch. The current implementation does
not support the OnLoad phase.

Add tests.

Bug: 34359699
Test: m test-art-host-run-test-929-search
Change-Id: I42955ff381cd3828bfd772bee2f9b0745195ee51

Merge "Revert "Revert "ART: Add ThreadGroup API support"""
am: a6040d4cca

Change-Id: I527259682522c2aea0d49fed7e1ef202197fce25

Merge "Revert "Revert "ART: Add ThreadGroup API support"""

ART: Add GetJLocationFormat

Add support for GetJLocationFormat. Set the format to bytecode
indices.

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

Merge "Do test_getStackTraces last for test 129"
am: fd5f52b60c

Change-Id: I60340d52f9c2b26005a22f3a48d95e8c39f1369a

Merge "Do test_getStackTraces last for test 129"

Merge "ART: Disable app images for test 912"
am: 3a5e34b657

Change-Id: I5e6571ceba95ee5184c804c0590ebadcf7205f41

Merge "ART: Disable app images for test 912"

Merge "ART: Fix missing CLOCK_MONOTONIC on Mac"
am: 442a83a762

Change-Id: Id723e233328185c09e5ac41b1d10947fa7747997

Merge "ART: Fix missing CLOCK_MONOTONIC on Mac"

Revert "Revert "ART: Add ThreadGroup API support""

This reverts commit 87071bfb6c1b708bdfa2a5f91d4744667b3a0443.

Add an ObjectLock, which corresponds to the synchronized(this)
implementation on the Java side. Wait for the expected five
child threads in the root group before running the actual
child test.

Bug: 31455788
Change-Id: Ib7a065d6a11f06f0325e3a8db040629f3ca69407
Test: m test-art-host-run-test-925-threadgroups

Merge "Introduce a number of SIMD extensions for x86/x86_64 (SSE)."
am: 55ddcc8fe9

Change-Id: Icf3a6d79bc5b1eee1f0082be881fa6de3feff4aa

Fix check failure in ConcurrentCopying::IssueEmptyCheckpoint().

Thread::DumpState() internally uses ScopedObjectAccess which expects
that the caller does not hold the mutator lock if its in a suspended
state or that the caller holds the mutator lock and in the runnable
state. Use ScopedObjectAccess instead on the caller side to match this
expectation.

Bug: 34321097
Bug: 33006388
Bug: 12687968
Test: test-art-host with CC.
Change-Id: I04698d21fb4820a938009c7caeb8e81721329007

Merge "Introduce a number of SIMD extensions for x86/x86_64 (SSE)."

ART: Disable app images for test 912

To have precise info for actually loaded classes.

Bug: 31684578
Test: m ART_TEST_INTERPRETER=true test-art-host-run-test-912-classes
Change-Id: I06a9303befab40c619f9d1e3d677024c8f3b0ba3

ART: Fix missing CLOCK_MONOTONIC on Mac

Fix the Mac build.

Bug: 31455788
Test: m
Change-Id: I03e927c28fe62abc1b14ea799190d00a081f33f7

Merge "Revert "Revert "Try really hard for JIT in test 916"""
am: ce27739a12

Change-Id: I6c67e5b4c26fe9981734b36bffceac611ad7e21a

Merge "Revert "Revert "Try really hard for JIT in test 916"""

Merge "Revert "Remove unused code for patching oat files.""
am: b51dee62cd

Change-Id: If1354ab4abeeae103cea2d4d97a0e04d44f78eae

Merge "Revert "Remove unused code for patching oat files.""

Revert "Remove unused code for patching oat files."

oat_file_assistant_test fails on target.

Bug: 33192586

This reverts commit c9f76628ce1dc54f852b689ed1722b7e154b2a9d.

Change-Id: Ie3b8900ed16d0f4033a76c7eb60ac8e9ad13f089

Do test_getStackTraces last for test 129

Doing this check last leaves time for the heap task daemon to call
TaskProcessor::RunAllTasks. This should hopefully fix the flakiness
of test 129 during startup.

Test: test-art-host-run-test

Bug: 34342857
Change-Id: I85f8d78083092df802728c59a4c86f63be8d949a

Merge "ART: Add three Timer functions"
am: bea5f85f6c

Change-Id: I440be88daf832a4dc712d0f48deec02c5a07f354

Merge "ART: Add three Timer functions"

Merge "ART: Add GetClassLoaderClasses"
am: 7ccdf0929f

Change-Id: Ibf2fe1250b92ed6636f3c072b15a53b04d395c46

Merge "ART: Add SetVerboseFlags"
am: c5603b7daa

Change-Id: I40b2777fabc8c02ad14b6628189fa4719e1f257e

ART: Add three Timer functions

Add support for GetAvailableProcessors, GetTimerInfo and GetTime.
Add tests.

Bug: 31455788
Test: m test-art-host-run-test-926-timers
Change-Id: I9629654349787e27dee686284567b5de70e138f2

Merge "ART: Add GetClassLoaderClasses"

Merge "ART: Add SetVerboseFlags"

Revert "Revert "Try really hard for JIT in test 916""

We were looping forever waiting for the entrypoint to be switched
despite the JIT being disabled by the tracing instrumentation.

This reverts commit ca4feac9484464a858990ca588398fceead55354.

Reason for revert: Problem with ensureJitCompiled fixed

Test: ART_TEST_TRACE=true        \
      ART_TEST_OPTIMIZING=false  \
      ART_TEST_INTERPRETER=true  \
      ART_USE_READ_BARRIER=false \
      mma -j40 test-art-host-run-test-916-obsolete-jit

Change-Id: I11f1e3f6cf65f90509a1ba66625a7988f3d94af6

Merge "MethodHandles: Remove experimental flag."
am: 49a694b84b

Change-Id: I944ec7dec6b4924aa2122ba26f2e671a1afb792e

Merge "MethodHandles: Remove experimental flag."

MethodHandles: Remove experimental flag.

Runtime::IsMethodHandlesEnabled now always returns true.
This change also removes the (now unnecessary) run scripts
for method handle tests.

Note that the --experimental=method-handles flag is left in
place for the test build steps (etc/default-build) because it's still a
convenient way to supply arguments to jack / smali.

Test: make test-art-host

Change-Id: I8475c7b8ac2615ac382a06ce54be39dce3cc7713

Reduce using ArtMethod's dex_cache_resolved_types_.

Avoid using the ArtMethod's dex cache type array shortcut
in runtime, preparing for its removal. We do not completely
remove the shortcut yet because it is still used by array
allocation entrypoints.

Fix ArgArray::BuildArgArrayFromObjectArray in reflection.cc
to not ask for the parameter type to be resolved. It should
have been previously resolved when retrieving the Method.

Also partially revert
    https://android-review.googlesource.com/310717
because it relied on the removed AIOOBE check in the removed
ArtMethod::GetDexCacheResolvedType(). The removed check was
simply defensive but it could not be triggered without some
memory corruption.

Test: m test-art-host
Bug: 30627598
Change-Id: Ic45a5ff8c66b79429e440cbc08d67bf22a083682

Merge "Acquire the mutator lock before comparing classes/strings."
am: 0a384f5beb

Change-Id: I69a0d6cab86d22aa6a2181cb4c8db7a4041adc29