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

Merge "ART: Remove IRT cruft"

Merge "Update the Begin() of LargeObjectMapSpace"

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

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"

Merge "Fix invoke-virtual not throwing ICCE in some cases"

event interface going private

Test: compile
Bug: 30465923
Change-Id: I704e0e6ae886207a9625fcb59be4876307d21e5f

Remove logging dependency on runtime

Moved the abort backtracing function to runtime, forcing callers to
supply the aborter at InitLogging.  This makes runtime properly layer
on top of logging by removing the cyclic dependency.

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

Merge "Move most mirror:: args to ObjPtr"

Fix invoke-virtual not throwing ICCE in some cases

Due to an oversight invoke-virtual on an interface method would
not cause an ICCE to be thrown if the target method is default. This
could potentially cause incorrect methods to be called at runtime.

Bug: 32201623
Test: mma test-art-host-run-test-978-virtual-interface
Change-Id: Ie565cf2fbe8602b17be0fb051e21d221a17b518f

More store/allocation elimination for singletons in case of loops

For a store into a singleton's location, if it happens inside a loop, it
means the singleton's location value may be killed by loop side effects.
However if the singleton is defined inside that loop, that loop should
be skipped since its loop side effects kill values at loop header where
the singleton's location doesn't exist yet.

Test: test-art-host

Bug: 31716107
Change-Id: Iae2494ea93295977f90d1463ee136a7e2e09ba9b

Move most mirror:: args to ObjPtr

Fixed possible moving GC bugs in ClinitImageUpdate class.

Bug: 31113334

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

Merge "Use FdFile::Copy() in dex2oat for better error checking."

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

This reverts commit 5665f6a561262b78d85afc975eca5a3138f4efa2.

Test failures were fixed in d4fd5b1ec3bea01ad6985b7fe19a0f80b5fc970b.

Test: make test-art-host.

Change-Id: I45901836a92336ed1a337cdbf3fa9a0181c3d975

Fix handling of dex cache arrays for method types.

- Add support for relocating method type dex cache arrays.
- Add missing call to MethodTypeDexCachePair::Initialize in the
  class_linker.
- Fix alignment calculation for method_types in
  DexCacheArraysLayout.
- Removed unused MethodTypeOffset(uint32_t) method.

Fixes tests with --pic.

Test: make test-art-host
Bug: 30550796

Change-Id: I2bba7228762b9e9834e7659fe62090be78afdfa8

Merge "Ensure OpenDexFilesFromImage closes file to prevent file descriptor leak"

Merge "Add DCHECK() for class loader in artResolveStringFromCode()."

Use FdFile::Copy() in dex2oat for better error checking.

Previously, read errors would have been silently ignored
and write errors would ungracefully crash.

Also improves performance on Linux, including Android,
by using sendfile(). (We still keep Mac builds around.)

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

Merge "Do not write empty class table and intern table to image."

Merge "ART: Blacklist test 907 for target"

ART: Blacklist test 907 for target

Bug: 31684578
Test: m test-art-host
Change-Id: If2353bef8601ba16941a7b1df2befc215b493431

Merge changes If95ecaf4,Ifb33cae2

* changes:
  ART: Temporarily use global references for GetLoadedClasses
  ART: Add GetLoadedClasses support

Merge "ART: Make JNI global and weak global locks global"

Ensure OpenDexFilesFromImage closes file to prevent file descriptor leak

(cherry picked from commit 645e477c2ce7a59ac98bc86463826cf082a13ad8)

Test: m -j32 test-art-host
Bug: 32095919
Change-Id: I44541edeb403a508e1b9794ee09fb1f55e3f42bf

Merge "Move reference processor and mirror::Reference to ObjPtr"

ART: Temporarily use global references for GetLoadedClasses

Until we have enough local references, use global references.

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

ART: Add GetLoadedClasses support

Add support to retrieve all loaded classes. Use the standard
ClassLoader callback infrastructure.

Bug: 31684578
Test: m test-art-host-run-test-907-get-loaded-classes
Change-Id: Ifb33cae27dc32602bcd8fd930d3c7eda02598c26

ART: Make JNI global and weak global locks global

Add them to Locks. Give the JNI global table lock
a lock level, close to the weak global JNI references.

Bug: 31684578
Test: m test-art-host
Change-Id: I3857a3b0be69b16811d9999096b2c42e7a25d227

Merge "Add a mark bit check in the read barrier jni entrypoint."

Move reference processor and mirror::Reference to ObjPtr

Bug: 31113334

Test: test-art-host

Change-Id: I2c7c3dfd88ebf12a0de271436f8a7781f997e061

Add a mark bit check in the read barrier jni entrypoint.

Bug: 30437917
Bug: 12687968
Test: test-art host/N9, N9 libartd boot, Ritz EAAC.
Change-Id: Ic5f8080959f243e80f00da73ba5f4b91a8d8b617

Merge "Move mirror::ObjectArray to ObjPtr"

Add DCHECK() for class loader in artResolveStringFromCode().

Address late comment from
    https://android-review.googlesource.com/281642

Test: m test-art-host
Bug: 20323084
Bug: 30627598
Change-Id: I3ba2617d2bc52ac8ba11307d206e6f5e6c017fb7

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

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

This reverts commit
f18d84699a7c18802f9774069143c173c6b927fb.

Tests are red on the bot. Blacklisting until this is investigated further.

Change-Id: Ibb9b4066abe28e8625c4eed5e0818e6a3fdd4f44

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

Merge "Small clean up in art::x86_64::ReadBarrierMarkSlowPathX86_64."

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

This reverts commit adc2ef25244645a650111560d252e32b3e22f463.

The test is green, now that the required smali changes have been
merged into aosp/master.

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

Do not write empty class table and intern table to image.

And remove dead ClassLinker and InternTable code that would
be erroneously triggered by not writing the tables.

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

Merge "interpreter: support for non-exact invokes."

interpreter: support for non-exact invokes.

Performs primitive argument conversions as well as boxing and unboxing
operations. Support for return value conversions will be added in a
follow up change.

Test: make test-art-host

Change-Id: I2e3348ff64a5826e477f87c12a7d5c390eb3a653

Merge "MIPS32: Improve integer comparison with constants."

Move mirror::ObjectArray to ObjPtr

Also reduce calls to ObjPtr::Ptr.

Bug: 31113334

Test: test-art-host -j32

Change-Id: I73e1b356972bb079b66332f00f7f07451601044e

Merge "jni: Add read barrier fast path to jni compiler"

Merge "Move common throws to ObjPtr"

Move common throws to ObjPtr

Also a few other throws. Removed some calls to ObjPtr::Ptr.

Test: test-art-host

Bug: 31113334

Change-Id: I4082d64134a5a8961489f4309522c28a30968a18

Merge "Move mirror::Class to use ObjPtr"

Move mirror::Class to use ObjPtr

Leave the return types as non ObjPtr for now. Fixed moving GC bugs
in tests.

Test: test-art-host

Bug: 31113334

Change-Id: I5da1b5ac55dfbc5cc97a64be2c870ba9f512d9b0

jni: Add read barrier fast path to jni compiler

Static method dispatch via JNI requires a read barrier
for the ArtMethod::GetDeclaringClass() load before adding it to the
JNI StackHandleScope.

We used to call ReadBarrierJni unconditionally but add a branch
to skip calling it if the GC is not currently in the marking phase.

Test: ART_USE_READ_BARRIER=true make test-art-host test-art-target
Bug: 30437917
Change-Id: I4f505ebde17c0a67209c7bb51b3f39e37a06373a

Small clean up in art::x86_64::ReadBarrierMarkSlowPathX86_64.

Clearly separate art::x86_64::CpuRegister (register object
used by the x86-64 assembler) from art::x86_64::Register
(register number) in
art::x86_64::ReadBarrierMarkSlowPathX86_64::EmitNativeCode.

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

Merge "Fix optimizing_cfi_test and arm64 code generation."

Fix optimizing_cfi_test and arm64 code generation.

Change https://android-review.googlesource.com/#/c/287582/ broke it.

test: m test-art-host-gtest-optimizing_cfi_test
test: m test-art-target on angler
Change-Id: I7fc74a87ffa0b26b8e103b87a2ac1179bea2145a

Merge "Fix missing symbols for multiple oat files case"

Merge "Generate SHA-1 build ID for host-generated *.oat files (1/2)."

Merge "Update HInstruction::NeedsCurrentMethod."

Merge "ART: Disable test 906 for JIT"

Merge "Revert "Revert "Remove dead code from compiler driver."""

Merge "Move Heap::GetInstances to use Handles"

Move Heap::GetInstances to use Handles

Prevent moving GC bugs from VisitObjects. Fix JDWP tests.

Test: art/tools/run-jdwp-tests.sh '--mode=host' '--variant=X32' --debug

Bug: 31113334
Change-Id: Ie7f51f1980ec0c1eddc3b59c3e49564eacb3be85

Fix missing symbols for multiple oat files case

Generate the missing symbols when enabling "--multi-image"

Test: m test-art-host-gtest-dex2oat_test
Change-Id: I0ae2fb552ecd741f0b06414bc4a37c2f93ce5389

MIPS32: Improve integer comparison with constants.

Test: booted MIPS32 in QEMU
Test: test-art-target-run-test-optimizing on CI20
Test: test-art-target-gtest on CI20

Change-Id: Icbe74099fdebc29a38055ef70d7f14eed770b0a7

Merge "Use handles for GetReferringObjects"

Use handles for GetReferringObjects

Required since VisitObjects may cause thread suspension. Fixes
failing jdwp tests.

Test: test-art-host
Test: art/tools/run-jdwp-tests.sh '--mode=host' '--variant=X32' --debug

Bug: 31113334
Change-Id: Ic11780b3778c83e7812bc90c0723a293537f376c

Merge "Fix moving GC bug in cloning finalizable objects"

ART: Disable test 906 for JIT

Suppress test which has potential allocations at inopportune
moments.

Bug: 31385354
Test: m ART_TEST_JIT=true test-art-host-run-test-906-iterate-heap
Change-Id: I137d5d2ba20b95ee4c4421e9da7526bee0c57d20

Fix moving GC bug in cloning finalizable objects

It is not safe to have thread suspension in the PreFenceVisitor. The fix is
to add the finalizer reference in the caller.

Test: art/tools/run-libcore-tests.sh '--mode=host' '--variant=X32' --debug

Bug: 31113334

Change-Id: I1e4650f8b75408a3d07e2c51ac334ff98552cfb5

Merge "Move registration into android package"

Merge "Replace StackHandleScopeCollection with VariableSizedHandleScope"

Merge "Add handle wrapper to allocation listener"

Merge "Recognize XOR-based periodic induction."

Merge "ART: Fix typo"

Add handle wrapper to allocation listener

Fix failing test for interpreter.

Test: mm -j32 test-art-host-run-test-debug-prebuild-interpreter-relocate-trace-cms-checkjni-picimage-npictest-ndebuggable-904-object-allocation32  ART_TEST_TRACE=true ART_TEST_OPTIMIZING=false ART_TEST_INTERPRETER=true

Change-Id: I9e7e09db82cf13730dc2471ac05260bf6a69ab7c

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

Move registration into android package

Mutator registration is tightly coupled with the android package, move
all registration from the soong package to the android package.

Test: build.ninja identical
Change-Id: I0326d0c88f78edb79eebedb155b69415e351fb61

Update HInstruction::NeedsCurrentMethod.

HLoadString and HLoadClass when sharpened may not need it
anymore. Instead just rely on the HCurrentMethod being the
SSA dependency of those instructions.

Also save storing the current method in the stack if the
graph actually doesn't need it.

test: m test-art-host test-art-target
Change-Id: I235d8275230637cbbd38fc0d2f9b822f6d2a9c1e

Recognize XOR-based periodic induction.

Rationale:
This is a commonly used construct (e.g. x = !x for booleans
and x ^= 1 for integers). This CL prepares some upcoming
optimizations that exploit such inductions.

Change-Id: I46edffb9de1075a836995daf5c2dfff7891f3034
Test: 530-checker-loops2 and induction_var_analysis_test

ART: Fix typo

Follow-up to commit 27fa96c285c4aa2f64e9bb63a3f38ffdc98c282a.

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

jni: Disable FastNative path for methods returning objects

Bug: 32088975
Change-Id: Iba3a8f5462829ccb38f771f8f940b57c4e6a1c3f

Merge "Fix the CC monkey crashes."

Replace StackHandleScopeCollection with VariableSizedHandleScope

VariableSizedHandleScope's internal handle scopes are not pushed
directly on the thread. This means that it is safe to intermix with
other types of handle scopes.

Added test.

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

Change-Id: Id2fd1155788428f394d49615d337d9134824c8f0