Pass some arguments by const reference.

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

Merge "Address review comments from change 000e1885701c8f."

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

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

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

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

ART: Move pathList well-known-field to actual class

Use the actual declaring class, BaseDexClassLoader.

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

Merge "Dump VerifierDeps in oatdump."

Flush the CFG visualizer output after writing to it.

This fixes test failures exposed by the ART Buildbot in
608-checker-unresolved-lse (AOT) on the hammerhead-ndebug
configuration.  The test used to fail because the CFG file
dumped by second invocation of dex2oat on device would be
truncated (at the beginning of the disassembly section
of the second compiled Dex file's contents) because of
dex2oat's fast exit.

Interestingly enough, this is the only case where this
failure has been observed, which seems to be due to a
combination of:
- targeting (32-bit) ARM;
- using a secondary Dex file (used to create an unresolved
  access for that test);
- compiling that secondary Dex file with dex2oat (ndebug
  mode) instead of dex2oatd;
- supporting multithread CFG graph dumping (by having the
  write-to-file operation in a critical section) since
  https://android-review.googlesource.com/#/c/296224/.

Test: art/test/run-test -O --debuggable 608-checker-unresolved-lse
Change-Id: Ifc1a23a3708b8645fd36c148312074bb9fe00cfc

Dump VerifierDeps in oatdump.

bug:30937355
test: oatdump_test
Change-Id: I344726a20f21a6c6986047c21278c753b9e24575

MethodHandles: Track libcore change cc4ded.

Track increase in vtable size of java.lang.Class, and add additional
tests for the method that resulted in the increase as well as its
caller.

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

Merge "Add a VerifierDeps::Dump method."

Merge "Address missed review comment."

Merge "MethodHandles: Better detail message for WrongMethodTypeExceptions"

ART: Blacklist test 913 for target

As usual, black-list an agent test for the target.

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

ART: Add forced garbage collection

Add ForceGarbageCollection support to the agent interface.

Add a test.

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

Merge "Move intern table to ObjPtr"

Move intern table to ObjPtr

Also moved StringLog to use a GcRoot.

Also removed read barriers from lookup to prevent making weak strings
strongly live more often than necessary.

Bug: 31113334

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

Change-Id: I05586e125d5dfed8d184890468fd398c79f32619

Merge changes I9afd9f88,I1cf94699

* changes:
  ART: Blacklist test 912 for target
  ART: Add GetClassSignature

ART: Blacklist test 912 for target

As usual, black-list an agent test for the target.

Bug: 31684578
Test: m test-art-host-run-test-912-classes
Change-Id: I9afd9f88bf794e6d90d1ea5cd259de4fbed7e27e

ART: Add GetClassSignature

Add support to retrieve the signature of a JNI class.

Add a test.

Bug: 31684578
Test: m test-art-host-run-test-912-classes
Change-Id: I1cf946996f9089cb5d41e93b4f1064193e9d6350

Merge changes Ib44df099,I8508f96f

* changes:
  ART: Add GetMethodModifiers
  ART: Add GetMethodDeclaringClass

Merge "MethodHandles: Remove obsolete test workaround."

MethodHandles: Remove obsolete test workaround.

Support for recognizing exact invokes was added by commit
3d617ac3d87f7fa98793406818e54b057bd701a1.

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

MethodHandles: Better detail message for WrongMethodTypeExceptions

We now include the type of the callee and the callsite.

This change also fixes a call to ThrowWrongMethodTypeException that
passed arguments in the wrong order.

Test: make test-art-host

Change-Id: I3ced2e331cb49b616a8374f5a604dafa37330059

Merge "MethodHandles: Temporarily rewrite test of exact invokes."

MethodHandles: Temporarily rewrite test of exact invokes.

Go through an additional layer of indirection to fool the overly
smart type inference.

The issue needs to be investigated on the Jack side, as per
the associated bug.

Test: make test-art-host
Bug: 32536744

Change-Id: Ic02bf155822e76332eaa7a300ba9c0da8e751e26

Merge "Interpreter: fix build."

Interpreter: fix build.

Due to unfortunate commit timing.

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

Merge "Getter/Setter support for invoke-polymorphic of invokeExact()."

Merge "method_handles: Complete support for emulated stack frames."

Getter/Setter support for invoke-polymorphic of invokeExact().

Test: make test-art-host
Bug: 30550796

Change-Id: I427a6e0afba88b223655ad1ba30843aaf255182b

Add a VerifierDeps::Dump method.

Also, fix the Encode/Decode logic around dex file order:
iterating over a std::map doesn't guarantee insertion order.

test: verifier_deps_test
bug: 30937355

Change-Id: Id3efd703b3c8ca21ab49c2600c612c132fb021cd

Merge "ART: Blacklist test 911 for target"

ART: Blacklist test 911 for target

As usual, black-list an agent test for the target.

Bug: 31684812
Test: m test-art-host-run-test-911-get-stack-trace
Change-Id: I477ede7eb9b0ce7e76e3705d911052b332087e35

ART: Add GetMethodModifiers

Support GetMethodModifiers to retrieve the modifiers of a JNI
method.

Extend test 910.

Bug: 31684812
Test: m test-art-host-run-test-910-methods
Change-Id: Ib44df099bc056c557e526d5fa267df48ac2fd86c

ART: Add GetMethodDeclaringClass

Support GetMethodDeclaringClass to retrieve the declaring class
of a JNI method.

Extend test 910. Also cover proxies.

Bug: 31684812
Test: m test-art-host-run-test-910-methods
Change-Id: I8508f96f88692e540ef53f693ff85590b7553f19

Merge changes Ifa5818eb,Icbb9f3a6

* changes:
  ART: Add GetStackTrace support
  ART: Add synchronous checkpoint

ART: Add GetStackTrace support

Add support for getting a stack trace.

Bug: 31684812
Test: m test-art-host
Change-Id: Ifa5818ebca38caafb09616ffd7df30186eb8a06f

ART: Add synchronous checkpoint

Add a thread function that will run the given checkpoint function
on the thread and wait for its completion.

Bug: 31684812
Test: m test-art-host
Change-Id: Icbb9f3a6b0bbf31e62be0508714ee172456bea29

Merge changes I369333c1,I55e7b6da

* changes:
  ART: Blacklist test 910 on target
  ART: Add method name support to TI

ART: Blacklist test 910 on target

As usual, blacklist agent test on target until supported.

Bug: 31684812
Test: m test-art-host
Change-Id: I369333c13f8959ceaf847a598f6b3e31597bf0c6

ART: Add method name support to TI

Add support for GetMethodName.

Bug: 31684812
Test: m test-art-host
Change-Id: I55e7b6dac7e7193c3d9979ebb289b0e2849ad030

Merge "ART: Clean up Thread"

Address missed review comment.

Bug: 30351473
test: m test-art-host

Change-Id: I8d0d536e9280555d156280d87055631cabf5b2b7

ART: Clean up Thread

Make some functions private. Move some test-only functionality to
the test using it.

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

Merge changes I6c3a84f5,I05414f3b

* changes:
  ARM: VIXL32: Move code to ease the diff with old codegen.
  ARM: VIXL32: Implement Shifts, Neg, Bitwise and Rotate visitors.

Merge "ART: Make graph visualizer thread-safe"

ART: Make graph visualizer thread-safe

Store visualizer to a temporary string stream during compilation.
Print to the final output on destruction, and do so under a lock.

Note: the order of methods in the output is only guaranteed to be
      deterministic with single-threaded compilation (-j1).

Test: m
Test: Compile multithread
Test: Check output file that begin_compilation and end_compilation
      are interleaved
Test: Check that names of compiled methods match

Change-Id: I7eb3be2f88333cf1f116fe71b77cffa1ac702208

ARM: VIXL32: Move code to ease the diff with old codegen.

This patch doesn't change anything except relocation the code
to preserve original order of functions as in code_generator_arm.cc
to make it easier to review and to fulfill comparison of two arm
backends for reaching the parity between them.

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

Change-Id: I6c3a84f5606069a795d4d8122fc0d4a308e2b7ec

ARM: VIXL32: Implement Shifts, Neg, Bitwise and Rotate visitors.

Tests which now start to pass:
* 112-double-math
* 415-optimizing-arith-neg
* 427-bitwise
* 431-optimizing-arith-shifts
* 436-shift-constant
* 474-fp-sub-neg
* 485-checker-dce-loop-update
* 514-shifts
* 542-bitfield-rotates
* 557-checker-instruct-simplifier-ror
* 564-checker-bitcount
* 565-checker-rotate

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

Change-Id: I05414f3b8bc12cf0fa74b53945eea703c8c55cbe

Merge "Remove default argument values in GenerateGcRootFieldLoad."