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

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"

Merge changes Ica4f766c,Iaa97520d

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

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

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

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

Merge "Use CLREX in ARM/ARM64 CAS intrinsic Baker read barrier slow paths."

method_handles: Complete support for emulated stack frames.

Most of this change is a refactor that templatizes the code
that performs argument conversions. This allows us to copy arguments
between two shadow frames, or an emulated stack frame and a shadow
frame.

Test: make test-art-host
Bug: 30550796

Change-Id: I23e65735a2dbd28f3c7b7d1ccf9762e77e0cf1f1

Merge "method_verifier_test: Allow soft verifier failures for java/lang/invoke.*"

method_verifier_test: Allow soft verifier failures for java/lang/invoke.*

Classes in this package might use experimental opcodes.

Bug: 32496585
Bug: 30550796
Test: make test-art-host
Change-Id: Ibf1ea78b090fff065b792f00afa143182ef195d4

Merge "class_linker: Change allocation of MethodType dex_cache entries."

class_linker: Change allocation of MethodType dex_cache entries.

We allocate them regardless of the value of the runtime flag to
avoid needing changes to the compiler etc. The memory hit can be
mitigated in production systems that don't use MethodHandles by
setting kDexCacheMethodTypeCacheSize to 0.

Also, add DCHECKS in the interpreter to make sure the runtime
never executes an invoke-polymorphic instruction when method handles
are disabled.

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

Merge "Add support for Baker read barriers in UnsafeCASObject intrinsics."

Merge "compiler_driver: loosen DCHECK for java/lang/invoke classes."

compiler_driver: loosen DCHECK for java/lang/invoke classes.

Verifier support for invoke-polymorphic hasn't been implemented
yet, so boot classpath classes that use it will not be verified
at compile time. This change will be reverted once verifier support
is implemented.

This change is a hack.

Test: make test-art-host
Bug: 32496585
Bug: 30550796

Change-Id: Id8fcb3fd0d5ddd09f1c80f751a5f96364b137855

Merge "ART: Add missing PrettyCause"

Merge "ART: Blacklist test 908 for GC_STRESS"

Merge "ART: Add function to retrieve all tagged objects"

ART: Blacklist test 908 for GC_STRESS

The test "measures" GC invocations and expects stable deterministic
behavior that cannot be guaranteed with a small heap.

Bug: 31684633
Test: m ART_TEST_GC_STRESS=true test-art-host
Change-Id: Id40f42e34b373643802e393fce7741eef549fb4b

ART: Add missing PrettyCause

Add missing name. Refactor so that build breaks next time anyone
forgets adding this.

Bug: 31385027
Test: m test-art-host
Change-Id: I7f5a510b56829aa39e8ca6bcff11c3e9eee95000

Remove default argument values in GenerateGcRootFieldLoad.

These values were never or rarely used.

Test: mmma art (with and without `ART_USE_READ_BARRIER=true`)
Bug: 12687968
Bug: 29516974
Change-Id: I5d15140ce501bf50d7a87871b1e492cee54913db

Merge "Remove H[Reverse]PostOrderIterator and HInsertionOrderIterator."

Merge "Remove some unneeded header includes"

Merge "ARM: VIXL32: Exclude recently added test that fails."

Remove H[Reverse]PostOrderIterator and HInsertionOrderIterator.

Use range-based loops instead, introducing helper functions
ReverseRange() for iteration in reverse order in containers.
When the contents of the underlying container change inside
the loop, use an index-based loop that better exposes the
container data modifications, compared to the old iterator
interface that's hiding it which may lead to subtle bugs.

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

Merge "ARM: VIXL32: Pass test 406-fields and a few others."

ARM: VIXL32: Exclude recently added test that fails.

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

Change-Id: I49b28c7bfbe178bf11f0c4b3e61e0372317ae215

Merge "MethodHandles: Remove reference to Jack 4.11.BETA from build rule."

Use CLREX in ARM/ARM64 CAS intrinsic Baker read barrier slow paths.

Follow clang's implementation, which uses CLREX in
compare-and-exchange operations on the failure path, i.e.
when the value read by the LDREX (ARM) or LDXR (ARM64)
instruction is not the expected value, in order to release
the monitor.  The previous implementation was perfectly
correct, but this one may improve performance on some
micro-architectures.

This change only affects the
art::arm::ReadBarrierMarkAndUpdateFieldSlowPathARM and
art::arm64::ReadBarrierMarkAndUpdateFieldSlowPathARM64 slow
paths.

Test: make test-art-target-run-test-004-UnsafeTest
Bug: 29516905
Bug: 12687968
Change-Id: I99edd1ae6489dcec4a0089bfef52736114c6cd48

MethodHandles: Remove reference to Jack 4.11.BETA from build rule.

It is no longer necessary, since the default Jack can handle
invoke-polymorphic now.

Test: make test-art-host

Change-Id: I0439fe1729c2204627408c1975b9c8b965328397

Merge "Add handle wrapper for interpreter iget/iput quick"

Add handle wrapper for interpreter iget/iput quick

JDWP has thread suspension in instrumentation listeners. The fix is
to use handle wrappers here. This change fixes JDWP tests.

Bug: 31113334

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

Change-Id: Ic7dcb1201ec44946e8002547b2f7f5645b4dea48

Merge "Add handle wrapper for DoCallCommon"

Add handle wrapper for DoCallCommon

Prevents potential moving GC bugs from thread suspension caused
by GetTypeItem. Fixes build-art.

Bug: 31113334

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

Change-Id: I9ef18b1d1dab61cc86a7468d535972eba51763f5

Merge "Move interpreter to ObjPtr"

Merge "Fix bad auto merge conflict resolution in CC"

Fix bad auto merge conflict resolution in CC

We should not be clearing the region space since:
https://android-review.googlesource.com/#/c/294708/

Bug: 12687968

Test: test-art-host CC baker
Change-Id: I78050c61c8cdcc0b777d3324ba3f7ded80b2cf52

Remove some unneeded header includes

Bug: none
Test: Builds still work
Change-Id: I6c83326cc12aa0e7825312b238f5e0ae694ccffc

Move interpreter to ObjPtr

Moved most of interpreter, interpreter_common,
interpreter_switch_impl, and some of mterp to ObjPtr.

Bug: 31113334

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

Change-Id: I0935d18287e1332205c17c5a018aa167788ab897

Merge "Avoid unnecessarily clearing/resetting region_space_ for each GC."

Merge "Fix build on XCode 8 w/OSX 10.12 SDK"

Merge "Add mode bits to open() with O_CREAT."

Merge "Blacklist 130-hprof for tracing configs."

Merge "Always call into native loader when loading a native library"

Merge "Improved induction variable analysis and loop optimizations."

Merge "Reduce file descriptor limit for 151-OpenFileLimit."

Merge "ART: Change InstructionSetFeatures to return unique_ptr"

Merge "ARM: VIXL32: Implement more codegen code to pass a few more tests."

Fix build on XCode 8 w/OSX 10.12 SDK

CLOCK_REALTIME (along with clock_gettime and friends) are now defined,
so the build fails with a redefinition error.

Test: m checkbuild (w/XCode 8 on 10.11 with 10.12 SDK)
Change-Id: Ic8a5cc872ded1766597fda5f098d0e044cd063ee

Merge "Forgot these files from previous VerifierDeps change."

Forgot these files from previous VerifierDeps change.

https://android-review.googlesource.com/#/c/294326/ was missing
these files.

bug: 30937355
test: m test-art-host
test: verifier_deps_test.cc
Change-Id: Ic17ee6c70d17e998a2835415a6e7b25a2473ac37