Merge "MethodHandles: Add a test case for MethodHandles.permuteArguments."

Merge "MethodHandles: Add tests for MethodHandles.filterReturnValue."

Merge "ART jitted code profiling support."

Merge "Have `make` report a failing gtest run as a top-level target."

Merge "ART: Temporarily disable test 913 for non-aot configs"

ART: Temporarily disable test 913 for non-aot configs

Until the test is generalized to recognize all allowed outputs,
disable non-default configurations.

Follow-up to commit 8c433b9e45a26e3bde123a5b2697ca801ae8b039 to
also disable for no-relocate configurations.

Bug: 31385354
Test: m ART_TEST_RUN_TEST_NO_RELOCATE=true test-art-host-run-test-913-heaps
Change-Id: If29ffdcca1fbd69bbac559dce99e84dc2592539a

Merge "ART: Temporarily disable test 913 for non-aot configs"

ART: Temporarily disable test 913 for non-aot configs

Until the test is generalized to recognize all allowed outputs,
disable non-default configurations.

Bug: 31385354
Test: m ART_TEST_INTERPRETER=true ART_TEST_JIT=true test-art-host-run-test-913-heaps
Change-Id: Iad10879391d8029445032d7b5a69e6ebf2672b9e

Merge "Fix heap poisoning build"

Fix heap poisoning build

Bug: 32577579

Test: mm build-art

Change-Id: I394691334ae2029810e069c768d34077d36679e2

Merge "ART: Add reference following code"

Merge "Add interface check cast fast path to arm, arm64, x86"

Merge "Revert "Rename vdex-fd to output-vdex-fd, add input-vdex-fd.""

Revert "Rename vdex-fd to output-vdex-fd, add input-vdex-fd."

Mishandled create_vdex_filename in installd.

This reverts commit bc654d0cbb4f45057f9482ad81cb613386bbce34.

Change-Id: I78f8fcd1c63133624f5bba01864ab858da0451d1

ART: Add reference following code

Add FollowReferences support. This is a work in progress.
Specific support for the metadata will be added iteratively.

Add a basic test to 913-heaps.

Bug: 31385354
Test: m test-art-host
Change-Id: I832b05a7397dc96cd9a868020dd7029255f6de87

Merge "ART: Refactor some TI test code for reuse"

Merge "Allow read side effects for removing dead instructions."

Add interface check cast fast path to arm, arm64, x86

Bug: 12687968
Bug: 32577579

Test: test-art-host, test-art-target CC

Change-Id: Ia57099d499fa704803cc5f0135f0f53fefe39826

Allow read side effects for removing dead instructions.

Rationale:
Instructions that only have the harmless read side effect may
be removed when dead as well, we were too strict previously.
As proof of concept, this cl also provides more accurate information
on a few string related intrinsics. This removes the dead indexOf
from CaffeineString (17% performance improvement, big bottleneck
of the StringBuffer's toString() still remains in loop).

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

Merge "Revert "JIT root tables.""

Revert "JIT root tables."

May be the offender for jit-gcstress failure of 902.

This reverts commit ac3ebc3150760425ed00abd56da48f9a6e0666bc.

Change-Id: I9ea6c9236fd1729fed7d1868dd8a111172932308

Merge "Revert "Support kJitTableAddress in x86/arm/arm64.""

Revert "Support kJitTableAddress in x86/arm/arm64."

Revert this in order to revert https://android-review.googlesource.com/#/c/285781/

This reverts commit 997d1217830c0a18b70faeabd53c04700a87d7d9.

Change-Id: I1888fba1c6f712cae4aec4ea4719b74a46da156c

Have `make` report a failing gtest run as a top-level target.

Before, if an individual gtest such (as
`test-art-host-gtest-output_stream_test64`) was invoked as
`make`'s top-level target and failing, `make` would not
report it as such through its exit status (it would be 0
whatever the result). However, when run as part of a
compound test target (such as
`test-art-host-gtest-output_stream_test` or
`test-art-host-gtest`), `make` would actually report this
test's (and maybe others') failure with a non-zero exit
status. This CL forces `make` to report a non-zero exit
status in the former case without breaking the latter one.

Test: Amend a gtest such as compiler/linker/output_stream_test.cc
    to make it artificially fail, and run it with:
      make test-art-host-gtest-output_stream_test
      make test-art-host-gtest-output_stream_test64
Bug: 31755702

Change-Id: Icbb6fef592f14d73808576e7fcb0e9d7079ec9a2

ART: Refactor some TI test code for reuse

Pull out helper code for creating arrays of data.

Bug: 31385354
Test: m test-art-host
Change-Id: I916fffa18f5a61bd17acbdbdaffce5b4c66a33ab

Merge "Reduce arena memory usage when changing graph structure."

Merge "Support kJitTableAddress in x86/arm/arm64."

Merge "Remove timeout for the 570-checker-osr test"

Reduce arena memory usage when changing graph structure.

Use swap() to reuse previously allocated memory. Use
range insertion instead of individual push_back()s.

Test: m test-art-host.
Change-Id: I7174f8cc66937f34b452ff7268a8806ca94e7573

MethodHandles: Add a test case for MethodHandles.permuteArguments.

Bug: 30550796
Test: make test-art-host
Change-Id: Idf1f330422a3be769e175f3e46c181d94217b9fa

Support kJitTableAddress in x86/arm/arm64.

test: test-art-host test-art-target, angler boots and runs.
Change-Id: I3654ae2809d4d759db76ee1ada1c17f3a9c3b392

Merge "Fix 552-checker-sharpening for PIC test."

Remove timeout for the 570-checker-osr test

The test is failing when running on emulator or slower devices.
Timeout is not needed anymore since all architectures have support
for OSR.

Test: ./run-test --jit 570-checker-osr

Change-Id: I3e84edfef3487e154e79688b81ad62dba45f9a2f

Merge "MethodHandles: Remove obsolete TODO."

Merge "Rename vdex-fd to output-vdex-fd, add input-vdex-fd."

Fix 552-checker-sharpening for PIC test.

And remove obsolete HLoadString::LoadKind::kDexCacheAddress.

Test: m ART_TEST_PIC_TEST=true test-art-host
Change-Id: I3e7a1a98c2c7eba5ea10954d7efcf743a807c300

MethodHandles: Remove obsolete TODO.

It was addressed by change 0d781e6e17a3f6285736c590001b59d50879f13b.

Test: make build-art-host
Change-Id: Ifb3f7fa2ffc245e88c77ca41268c376be1611308

MethodHandles: Add tests for MethodHandles.filterReturnValue.

Test: make test-art-host
Bug: 30550796
Change-Id: I2c8f25b003cbeee24e7603637169200a02d31e1c

Merge "MethodHandles: Add tests for MethodHandle.bindTo."

MethodHandles: Add tests for MethodHandle.bindTo.

Test: make test-art-host
Bug: 30550796
Change-Id: I7e8bd8c8cae5f874f7743fd1a0a361875fcb997e

Rename vdex-fd to output-vdex-fd, add input-vdex-fd.

In preparation for installd changes around vdex.

bug:30937355
Test: device boots, oat_file_assistant_test

Change-Id: I7eba67f278a03d77b3e66fd24a96fbfd442db678

Merge "JIT root tables."

Merge "MethodHandles: Constructor support"

Merge "ART: Fix tidy warnings"

Merge "Allow HDeoptimize to co-exist with LSE."

Merge "ART: Refactor jfieldID handling"

Merge "Change check cast entrypoint to check instance of"

ART: Fix tidy warnings

Switch to char versions of find variants.

Add "explicit" constructor variants or refactor and
remove defaults.

Use const references.

Bug: 32619234
Test: m test-art-host
Change-Id: I970cc2f47d6cf8f0c74104b994b075b2fafb3d45

ART: Refactor jfieldID handling

ArtField objects have been native since Marshmallow.
Remove the dependency on being runnable. Refactor the
code into the jni_internal header.

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

Merge "ART: Refactor jmethodID handling"

Change check cast entrypoint to check instance of

Reduces code size since we do not need to reload class before
calling slow path.

TODO: Delete read barriers in the check cast code since the slow
path will retry with the proper read barriers if the check fails.

Bug: 12687968
Bug: 29516974

Test: test-art-host + test-art-target with CC

Change-Id: Ia4eb9bbe3fe2d2016e44523cf0451210828d7b88

JIT root tables.

Implement root tables for the JIT. Each JIT compiled method
gets a table allocated before the stack maps.

The table gets visited through Runtime::SweepSystemWeaks.

Implement String roots for x86_64 as an example.

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

Change-Id: Id3d5bc67479e08b52dd4b253e970201203a0f0d2

MethodHandles: Constructor support

Test: m test-art-host-run-test-956-methodhandles
Bug: 30550796

Change-Id: Ied466d7e0f0f0b472e3a28909ca712dfdb99e422

Merge "Fix SimplifyIfs() trying to redirect exception handler edges."

Merge "Add String.indexOf() benchmark."

Fix SimplifyIfs() trying to redirect exception handler edges.

Test: 622-simplifyifs-exception-edges
Test: m test-art-host
Bug: 32545860
Bug: 32546110
Bug: 18051191
Change-Id: I641ed73baace840411fc5651937bbc80e79b0c46

Merge "Argument conversions for setter/getter MethodHandles."

Argument conversions for setter/getter MethodHandles.

Test: m test-art-host-run-test-959-invoke-polymorphic-accessors
Bug: 30550796

Change-Id: I9898605fc9f9f8a80f47f9559f3ccb99b02b07c8

ART: Refactor jmethodID handling

ArtMethod objects have been native since Marshmallow.
Remove the dependency on being runnable. Refactor the
code into the jni_internal header.

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

Merge "ART: Extend Object tag table to be explicitly lockable"

Merge "Add forwarding address checks for X86, arm, arm64"

Merge "ART: Add Unlink to FdFile"

Merge "ART: Make excludes explicit"

Add forwarding address checks for X86, arm, arm64

Added to READ_BARRIER_MARK_REG.

Bug: 30162165

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

Change-Id: I15cf0d51ed3d22fa401e80ffac3877d61593527c

Merge "Support genrules with multiple tools"

ART: Add Unlink to FdFile

Add Unlink function that tries to unlink the file if it
was created with a file path. The function tries to
ensure that it does not unlink a newer file.

Add a parameter to Erase to add unlinking.

Test: m test-art-host-gtest-fd_file_test
Change-Id: I49993bb94aec10d5c8d9b2cbea30ebaa255b99e1

Support genrules with multiple tools

To allow genrules with more than one tool, rename the tool property to
tools and make it an array, replace $tool with $(location <label>),
and use $() for other variables for consistency.

Bug: 31948427
Test: compare build.ninja
Change-Id: I8d53510ac081ae20835b3ab7dfee3d74c520ab4d
(cherry picked from commit 7d332965ea44f54ddeedc9b7fadcf2e5b3259f3a)

ART: Extend Object tag table to be explicitly lockable

Expose the lock for the system weak holder. Add functions to lock
and unlock the object tagging table, and add accessors that assume
the table is already locked.

Bug: 31385354
Test: m test-art-host
Change-Id: I5a4328b0d439d13e6ace7965e22f01bbf8130f20

ART: Make excludes explicit

Fix some missing lock excludes.

Test: m
Change-Id: If74708c5d16469490e3374b7d5ffc23dd2ea5ec1

Merge "NB getError returns const char"

Merge changes I04517ee2,I476792ca

* changes:
  ART: Remove unused lock levels
  ART: Clean up old mutex

Merge "MethodHandles: Add tests for array accessors / identity / constant."

Merge "Revert "Revert "Enable IntermediateAddress for primitive arrays with read barriers."""

MethodHandles: Add tests for array accessors / identity / constant.

Accompanies libcore change a93a1fcc5b0e.

Test: make test-art-host
Bug: 30550796
Change-Id: Ic793687d98f850ffcd5f09fd3ab02d44644787d5

Add String.indexOf() benchmark.

Test: vogar --benchmark art/benchmark/string-indexof/src/StringIndexOfBenchmark.java
Change-Id: I57c8c9c7ede2b0e86a6142e713b4e8be094c82ce

Merge "Increase file size limit."

Increase file size limit.

Some checker tests now require more, as the cfg they
generate got bigger.

test: test-art-target
Change-Id: I301bc99730b139f7f5bdec77df3d6bee272d609e

Merge "Fix poison build."

Fix poison build.

The slow path for interface check cast isn't fatal for that build.

Test: ART_HEAP_POISONING=true/false m test-art-host-run-test
Change-Id: I3daf59e7f1e828ec72faf899adfafc9abd0fed88

Merge "ART: Make TypeCheckSlowPath call kind consistent"

ART: Make TypeCheckSlowPath call kind consistent

Slight refactor to make location and code generator consistent.

Bug: 32577579
Bug: 29516974
Test: m test-art-host
Change-Id: I11f8e227d4088d4922e6fdac3348cfe0f6690216

ART: Remove unused lock levels

Removes three obsolete lock levels.

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

NB getError returns const char

We fixed the NB interface to have the callback return const char*
instead of char*, this commit fixes the test.

Bug: 32542970
Test: make test-art-host -j48
Change-Id: Iebab91c94dcc5699623226f220d6ef474b1a5f1f

Merge "X86_64: Inline checking for kInterfaceCheck check cast"

ART: Clean up old mutex

Remove obsolete mutex.

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

Allow HDeoptimize to co-exist with LSE.

The env use of HDeoptimize counts as escaping at the end of
executing the compiled code for a singleton, just like a return.
So store elimination won't happen for that singleton's locations.

Test: make test-art-host
Bug: 31716107

Change-Id: I09838e80586d279714c676a2f7294ae518f61457

X86_64: Inline checking for kInterfaceCheck check cast

Interface check-cast with read barriers:
Before: 5.27 ns/check
After: 2.36 ns/check

Code expansion is 0.29%.

Test: test-art-host

Bug: 32577579
Bug: 29516974

Change-Id: Ie48251f97fbf2c95bd0a7c4bd62075414f8e3ae3

Merge "ART: Ignore google-default-arguments tidy check"

Merge "ART: Add NOLINT to ObjPtr"

Merge changes I8bbef5e3,Ib68bdc31,Ie39b7f59,I1ef8b65a

* changes:
  ART: Remove "explicit" annotation
  ART: Use references in assembler tests
  ART: Make ManagedRegister ValueObject
  ART: Make Location trivially copyable

Merge "Account for early exit loop."

Merge "Fix spin waiting for weak ref access in the intern table for CC."

Merge "Avoid visiting just eliminated bounds check."

Revert "Revert "Enable IntermediateAddress for primitive arrays with read barriers.""

This reverts commit 4a3aa578eff94eb10450fae1772deb7cb8ddc6a6.

The failing assertion (see b/30762467):

08-09 11:32:46.767  1654  1656 F dex2oatd: art/compiler/optimizing/register_allocation_resolver.cc:325] Check failed: interval->GetDefinedBy()->IsActualObject() IntermediateAddress@InstanceFieldGet

that motivated the initial revert has been removed by a
previous CL (commit
70e97462116a47ef2e582ea29a037847debcc029,
https://android-review.googlesource.com/#/c/254920/).

Test: ART host and target (ARM, ARM64) tests with `ART_USE_READ_BARRIER=true`.
Bug: 26601270
Bug: 12687968
Change-Id: I09cae0c6c38ca403924153e9f0eb0cc3ff4540e7

Merge "Fix ART run-test 562-no-intermediate."

ART: Ignore google-default-arguments tidy check

Ignore the check for default arguments in virtual functions.

Bug: 32619234
Test: m
Change-Id: I62f1b36ca6fd0965cfd9f6be343b8a63ac4642c6

ART: Add NOLINT to ObjPtr

The ObjPtr constructors are constructed to allow implicit conversion.

Also ensure that ObjPtr is trivially copyable.

Bug: 32619234
Test: m
Change-Id: I022e8d7d5a54c0057e9007bb7c13312b343c23b6

ART: Remove "explicit" annotation

Remove unnecessary annotation.

Bug: 32619234
Test: m
Change-Id: I8bbef5e3b4b8d3f5ddeaece08fe614b9eeae6f2c

ART: Use references in assembler tests

Move parameters to const references.

Bug: 32619234
Test: m
Change-Id: Ib68bdc313b91fee1e9e4e1e794eeca630837b005

ART: Make ManagedRegister ValueObject

Make ManagedRegister a value class, and ensure it is trivially
copyable.

Bug: 32619234
Test: m
Change-Id: Ie39b7f59a01ed293891e97110db3f5262f25fb4e