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

Merge "Avoid visiting just eliminated bounds check."

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

Fix ART run-test 562-no-intermediate.

- Rename it to 562-checker-no-intermediate, so that Checker
  assertions are actually verified.
- Fix Checker assertions, as they did not match the
  code expected from the compiler.
- Have the test use an integer array (instead of a double
  array), so that Checker assertions can be extended to ARM
  as well (the ARM back end does not support the
  IntermediateAddress instruction on long, float and double
  arrays).

Test: make test-art-target-run-test-562-checker-no-intermediate
Bug: 30888043
Change-Id: I32c891948c585983d5fe5d6df092bc7a59c19ae6

Merge "Workaround adb issue in run-test."

Workaround adb issue in run-test.

Try a few times before giving up on fetching the architecture.

bug:32655576
Change-Id: I05c6520efa56570ebe23820fca30fc234c368c88

Merge "MethodHandles: Implement MethodHandle.asType."

Merge "Integrate VerifierDeps in compiler driver."

Merge "MethodHandles: Track libcore change eb497095e03068459."

Merge "More loop-body simplifications."

Merge "ART: Make Handle trivially-copyable"

Avoid visiting just eliminated bounds check.

Test: test-art-host
Bug: 32547652
Change-Id: Ifaed3d4eee08c6d044a41ade6c1ee73989489110

ART: Make Handle trivially-copyable

Use default copy constructor and copy assignment to make Handle
trivially copyable. Do the same for MutableHandle.

Fix up unused Handle warnings. Add asserts in the HandleScope
test.

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

More loop-body simplifications.

Rationale:
This removes all dead induction from the CaffeineLogic loop,
giving yet the next performance boost  (2700us->1700us).
Also, the runtime is now the same between a DX compiled
and JACK compiled version, giving confidence that all
recent introduced optimizations are generally useful
and something expected from any optimizing compiler.
Last, less realistic improvement will pale anything
seen so far, since it removes the full loop (still TBD).

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

Merge changes I86bc206d,If20a34e7,I6ee6c826,I240341a0

* changes:
  ART: Make some variables references
  ART: Put macro parameters into parentheses
  ART: Add "explicit" to constructors
  ART: Change iterator value type to match

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

Bug: 32637633
Test: test-art-host with CC
Change-Id: I8f0f9f5ca24687f688318ea5cafcbb7f6597448c

Merge "Remove lock from ClassExt installation procedure."

Merge "Pass some arguments by const reference."

ART: Make some variables references

Avoid copying values.

Bug: 32619234
Test: m
Change-Id: I86bc206d6f6fccdf730c62c60317a7a4ae26c11b

ART: Put macro parameters into parentheses

Add parentheses to make macros robust.

Bug: 32619234
Test: m
Change-Id: If20a34e7e3497b1d969800d5004c8535810d90a6

ART: Add "explicit" to constructors

Add explicit to two constructors with single argument.

Bug: 32619234
Test: m
Change-Id: I6ee6c8264d76017247e2236170c776c801719a0a

ART: Change iterator value type to match

Make keys const.

Bug: 32619234
Test: m
Change-Id: I240341a0fec79754bea2ed21c0d0a4ddd119ce35

Merge "Check for forwarding address in READ_BARRIER_MARK_REG"

Merge "Update AArch32 VIXL intrinsics w.r.t. the new art::ReadBarrier API."

Update AArch32 VIXL intrinsics w.r.t. the new art::ReadBarrier API.

Update with respect to changes introduced by commit
12b58b23de974232e991c650405f929f8b0dcc9f.

Test: ART_USE_READ_BARRIER=true make
Change-Id: Iaefd664172ff19cce17cf4d3b2064dba86170c22

Merge "Add more missing definitions in art::arm::CodeGeneratorARMVIXL."

Add more missing definitions in art::arm::CodeGeneratorARMVIXL.

This fixes build breakages on the read barrier
configuration.  These are just dummy definitions, as ART's
AArch32 VIXL back end is work in progress.

Test: ART_USE_READ_BARRIER=true make
Change-Id: I8a0dd9724e4bf95664330232db40fd58bb8d234e

Merge "Clean up the runtime read barrier and fix fake address dependency."

Merge "More store/allocation elimination."

Merge "Output more for "Unable to determine architecture"."

Merge "ART: Additional initialization for MethodHandle accessors."

Merge "Add missing definition of CodeGeneratorARMVIXL::GenerateReadBarrierSlow."

ART: Additional initialization for MethodHandle accessors.

Ensure classes are initialized before static fields are touched by
MethodHandle setters and getters.

Bug: 30550796
Test: m test-art-host-run-test-979-invoke-polymorphic-accessors
Change-Id: Ibac4372607ecbb4e6f7347b89cef6d280632c835

Remove lock from ClassExt installation procedure.

We were using a lock on the class to ensure that we avoid races in
setting the ext_data_ field of a class object. We replace this with a
CAS of the field in order to prevent deadlocks.

Test: mma test-art-host
Change-Id: Ie436ff9526f2c3b38a9af49c5606a7cee6d718f1

Add missing definition of CodeGeneratorARMVIXL::GenerateReadBarrierSlow.

This fixes build breakages on the read barrier table lookup
configuration.  This is just a dummy definition, as ART's
AArch32 VIXL back end is work in progress.

Test: ART_USE_READ_BARRIER=true ART_READ_BARRIER_TYPE=TABLELOOKUP make
Change-Id: Ifef39148fb7d9d8a5ad309ec0a2edc8a97d72f3d

Merge "Clean up parameter names in ManagedRegisterEntrySpills."

Pass some arguments by const reference.

Test: Rely on TreeHugger.
Bug: 32619234
Change-Id: Ia26277893641dcb3562eaeacaec4e089a978d56a

MethodHandles: Implement MethodHandle.asType.

Tracks libcore change 16fa583fb5ee489.

Test: make test-art-host
Bug: 30550796

Change-Id: I2457b563f67a183c4eebf94ddbe74cc55f772ee0

Output more for "Unable to determine architecture".

test: run-test

Change-Id: I3a3637ba5a38f5af7cdf1e4b337e11ff652db893

MethodHandles: Track libcore change eb497095e03068459.

Adds tests for Lookup.unreflect*

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

Merge "ARM: VIXL32: Implement intrinsics."

Clean up parameter names in ManagedRegisterEntrySpills.

Avoid the double underscore prefix. Identifiers starting
with two underscores are reserved for C++ implemenations,
i.e. the C++ compiler and the standard library.

Test: Rely on TreeHugger.
Change-Id: If6d80a63badb2f95f4cbcef946c10911c954419b

Merge "Address review comments from change 000e1885701c8f."

Integrate VerifierDeps in compiler driver.

If a VerifierDeps is given, try fast verify. Otherwise create
a new VerifierDeps and do the full verification.

test: test-art-host, verifier_deps_test
bug: 30937355
Change-Id: Ifb030d7b5232c95872164f601057a56ab96038e1

Merge "ARM: VIXL32: Implement the functionality needed by intrinsics."

Merge "Add print statements for "Unable to determine architecture"."

Add print statements for "Unable to determine architecture".

test: test-art-host-run-test
Change-Id: Ifa57e1782450456622843e4fae15e230c0d19dea

Merge "MIPS32: Improve and moderately clean up mterp."

Merge "New instruction simplifications. Extra dce pass. Allow more per block repeats."

Merge "Call ClassExt::VisitRoots"

Call ClassExt::VisitRoots

We were missing this call.

Test: ART_DEFAULT_GC_TYPE=GSS mma test-art-host
Change-Id: I559a64b04dde514888e37a86fafc260e55196f95

Clean up the runtime read barrier and fix fake address dependency.

- Rename GetReadBarrierPointer to GetReadBarrierState.
- Change its return type to uint32_t.
- Fix the runtime fake address dependency for arm/arm64 using inline
  asm.
- Drop ReadBarrier::black_ptr_ and some brooks code.

Bug: 12687968
Test: test-art with CC, Ritz EAAC, libartd boot on N9.
Change-Id: I595970db825db5be2e98ee1fcbd7696d5501af55

Merge "Improved range analysis (and thus BCE) around min/max/abs intrinsics."

Merge "Add ClassExt mirror."

New instruction simplifications. Extra dce pass. Allow more per block repeats.

Rationale:
We were missing some obvious simplifications, which left performance
at the table for e.g. CaffeineLogic compiled with dx (4200us->2700us).
The constant for allowing a repeat on a BB seemed very low, at the
very least it should depend on the BB size.

Test: test-art-host

Change-Id: Ic234566e117593e12c936d556222e4cd4f928105

Check for forwarding address in READ_BARRIER_MARK_REG

When the object is in the from-space, the mark bit is not set.
In this case, we can also check the lock word for being a forwarding
address. The forwarding address case happens around 25% of the time.
This CL adds the case for forwarding address lock words to
READ_BARRIER_MARK_REG.

Reduces total read barriers reaching runtime on ritzperf:
Slow paths: 20758783 -> 15457783

Deleted the mark bit check in MarkFromReadBarrier since most of the
callers check the bit now.

Perf:
ReadBarrier::Mark: 2.59% -> 2.12%
art_quick_read_barrier_mark_reg01: 0.79% -> 0.78%
art_quick_read_barrier_mark_reg00: 0.54% -> 0.50%
art_quick_read_barrier_mark_reg02: 0.31% -> 0.25%

Only X86_64 for now, will do other archs after.

Bug: 30162165

Test: test-art-host

Change-Id: Ie7289d684d0e37a887943d77710092e380457860

Add ClassExt mirror.

We add the ClassExt type to hold values in Class that are not always
needed without making the Class type larger. For the initial change we
move the verifyError field into this type. It is expected to grow as
time goes on.

Test: mma test-art-host

Change-Id: I1d97df2e1267203841ad47b2effcb66dd76ac12a

More store/allocation elimination.

When merging values of an unreturned singleton's location,
do more analysis on the singleton's liveness since the singleton
may not be live in some of the predecessors. And only those
predecessors where the singleton is live should participate in
merging the value of the location.

Test: make test-art-host
Bug: 31716107

Change-Id: I1bf61a84d3473646eb38fd9924bbb063cf664f7f

Improved range analysis (and thus BCE) around min/max/abs intrinsics.

Rationale:
Inspection of some typical bit set utilities revealed that we
were missing obvious cases where two or more array lengths
were combined using a min.

Change-Id: I3e6463f221c793aaa1d592d4caabef0511754ae9
Test: test-art-host-run-test-620-checker-bce-intrinsics

Address review comments from change 000e1885701c8f.

Style fixes, and better OOM handling.

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

ARM: VIXL32: Implement 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: I64397a0c4fd40fd7cd0049929fcce98b935d0372

ARM: VIXL32: Implement the functionality needed by intrinsics.

This patch passes:
* 004-UnsafeTest

Test: ART_USE_VIXL_ARM_BACKEND=true m test-art-host
Test: ART_USE_VIXL_ARM_BACKEND=true m test-art-target
Change-Id: I7a3cd410411ef3a520d419734a835090097b2aee

Merge "Revert "Revert "ART: Generalize FindClassInPathClassLoader"""

Merge "Revert accidental change introduced in 000e188570."

Merge "It exposes a few bugs that should be address before submitting it. "

It exposes a few bugs that should be address before submitting it.

Revert "Use interpret-only instead of verify-at-runtime when testing JIT"

This reverts commit 5db5b1e6bc874373dafc73731669e5022ac93b10.

Change-Id: Ib87730ca4c9744a289d267af0b67dce2b6075cfd

Revert "Revert "ART: Generalize FindClassInPathClassLoader""

Bug: 32510706

Change is not responsible for the flakiness after all.

This reverts commit af3572bc21cf66f970090c852b976095bb028083.

Change-Id: I399049c217deeac6a5c3f9d678a46bdb84386e3f

Merge "MIPS32: Fix string loads that use kDexCacheViaMethod."

Merge "ARM: VIXL32: Implement ArrayGet, ArraySet, BoundsCheck etc."

Revert accidental change introduced in 000e188570.

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

Merge "MethodHandles: Add tests for transforms in change 598d43d0dec5925c77"

MethodHandles: Add tests for transforms in change 598d43d0dec5925c77

Test: make test-art-host
Bug: 30550796

Change-Id: I41e4c7f9398e5120b65577aa559c63fcc1d25508

Merge "Revert "ART: Generalize FindClassInPathClassLoader""

Revert "ART: Generalize FindClassInPathClassLoader"

Bug: 32510706

This reverts commit fd1ae75a03b0b17fdbbb0097e856f494edf12312.

Change-Id: Ie247477eea44a6a241d90864bf5dca29897ff4b8

Merge "MethodHandles: Track libcore change cc4ded."

Merge "Add VerifierDeps::Verify."

Merge "ART: Simplify test 913"

ART: Simplify test 913

To work around unexpected GCs popping up and influencing this
not-very-robust test.

Bug: 32516268
Test: m test-art-host-run-test-913-heaps
Change-Id: I9ba831718de5487f01245238c30449e048e38ff2

Merge "Fix heap poisoning build"

Fix heap poisoning build

Remove extra %.

Bug: 32577579

Test: mm
Change-Id: I22a144da68c2b79ef38cd3f4522e040721517260

Merge "Fix read barriers in VisitDexCachePairs"

MIPS32: Fix string loads that use kDexCacheViaMethod.

This fixes some string-related crashes, in particular,
failures in test 563-checker-fakestring.

The bug was introduced in commit
aad75c6d5bfab2dc8e30fc99fafe8cd2dc8b74d8.

Test: booted MIPS32R2 in QEMU
Test: test-art-target-run-test-optimizing (MIPS32R2) on CI20

Change-Id: Ie7d9a5b2c572b0933756f6e95c567968f811342c

Merge "Add interface fast path to art_quick_check_cast for X86_64"

Fix read barriers in VisitDexCachePairs

It was ignoring kReadBarrierOption and marking through the read
barrier.

Bug: 12687968

Test: test-art-host CC
Change-Id: I5506acb5efb4b46be115ac899640522683898b47

Merge "ART: Rename heap support file"

Add interface fast path to art_quick_check_cast for X86_64

X86_64 CC ritzperf results from perf:
art_quick_check_cast: 0.44% -> 0.76%
artIsAssignableFromCode: 1.78% -> 0.11%

Added stub test.

Bug: 32577579

Test: test-art-host

Change-Id: I5ed5675c4674fac8eed8826eb50527f4876e5f07

Merge "ART: Fix test 913 for concurrent collector"

ART: Fix test 913 for concurrent collector

First empty the internal counts without printing. Prevents picking
up on stray garbage collections.

Bug: 32516268
Test: m ART_USE_READ_BARRIER=true ART_READ_BARRIER_TYPE=TABLELOOKUP ART_HEAP_POISONING=true test-art-host-run-test-913-heaps
Change-Id: Ib1e1993c19bbaa0bf77aeec4eedc6d67e59df840

MIPS32: Improve and moderately clean up mterp.

Improvements:
- use seb, seh, ins on R2+
- use lsa, trunc.l.(s|d), cvt.(s|d).l, jic on R6
- shorter float/double comparison
- shorter float/double to int/long conversion
- fewer memory reads in float/double to int/long conversion
- remove unnecessary %break's and branches across breaks
- use branch delay slots more efficiently on R2

Test: booted MIPS32R2 in QEMU
Test: test-art-target-run-test-interpreter (MIPS32R2) on CI20
Test: booted MIPS64 (with 2nd arch MIPS32R6) in QEMU
Test: test-art-target-run-test-interpreter (MIPS32R6) in QEMU

Change-Id: I9959bec08e20c2423deae31f71b523ad36b4be9a

Merge changes I7d79aa90,Ie051e7d4

* changes:
  ART: Generalize FindClassInPathClassLoader
  ART: Move pathList well-known-field to actual class

Merge "tools: Fix generate-asm-support script"

Merge "Use interpret-only instead of verify-at-runtime when testing JIT"

ART: Rename heap support file

Differentiate it from the runtime naming.

Bug: 32516268
Test: m test-art-host
Change-Id: I19a09933ada0f312930255a22f306527ce3b89db

Merge changes Ica4f766c,Iaa97520d

* changes:
  ART: Blacklist test 913 for target
  ART: Add forced garbage collection

Add VerifierDeps::Verify.

Method verifies that the recorded dependencies still hold.

test: verifier_deps_test.cc
bug: 30937355
Change-Id: I137f77739355141e46994b4cb6043dbbc9353637

tools: Fix generate-asm-support script

A recent change of cpp-define-generator to bp
had removed the 'd' suffix from the binary name.

Change-Id: Ie3fe3f504789798ccfb83b77d5b47e8aeb65e0cc

Use interpret-only instead of verify-at-runtime when testing JIT

That's because we run most of our products with interpret-only and not
verify-at-runtime.

Test: ART_TEST_JIT=true m test-art-host
Change-Id: Ic5a9384adbf8a27977d230fe1b1c5858945dbc51

Merge "Flush the CFG visualizer output after writing to it."

ARM: VIXL32: Implement ArrayGet, ArraySet, BoundsCheck etc.

Over 100 more ART tests now start to pass.

Test: export ART_USE_VIXL_ARM_BACKEND=true && \
      mma test-art-host dist && \
      mma test-art-target dist

Change-Id: I8b7c2e29886981d72057d36347bca0626aabfe81

ART: Generalize FindClassInPathClassLoader

Also accept DexClassLoader. Do a fast check for subclasses of
BaseDexClassLoader that are in the boot classpath.

Bug: 32510706
Test: m test-art-host
Change-Id: I7d79aa90e0241e7717c5d146d1123f41746e4ffa