ART: Add GetClassFields support
am: ac58727f26

Change-Id: I7cce7480ac88da2e0c914cdaf7f6f3194f35c89f

ART: Add Field methods
am: ab2f0d0395

Change-Id: Ifdfd28466aab620d433599768076bc6ef72e50e6

ART: Add IsInterface and IsArrayClass support
am: 4fd66ecf2e

Change-Id: I1cfe905046369ffc161ee2e6997ce832242484c1

Merge "Avoid using ThreadList::Dump() in empty checkpoint timeout."
am: 48ff5b9aec

Change-Id: If65a49ee08edc8539c10a63dbc9a3bf6d660c624

Merge "Avoid using ThreadList::Dump() in empty checkpoint timeout."

ART: Add GetClassFields support

Add GetClassFields support. Add a test.

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

ART: Add Field methods

Add GetFieldName, GetFieldDeclaringClass, GetFieldModifiers
and IsFieldSynthetic support. Add tests.

Bug: 34113943
Test: m test-art-host-run-test-918-fields
Change-Id: Id33b8c54c250f73f0658ec45ccb2b55eccb37623

ART: Add IsInterface and IsArrayClass support

Add support for these two required calls. Add a test.

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

Merge "Remove the IsInDexCache flag from HLoadString."
am: 776ba76275

Change-Id: I3c1178a66c1cfbfca1b98cf724f3f24acaad2eca

Merge "Remove the IsInDexCache flag from HLoadString."

Remove the IsInDexCache flag from HLoadString.

This flag was obsolete and always false.

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

Merge "ART: Make method handle runtime code callable from compiler."
am: 6d2741f18b

Change-Id: I17ff895b0eafe68599f27d1872a7f16afee96027

Merge "Remove bogus DCHECK."
am: fe150d9821

Change-Id: Ic849a5672989d9e0d411f22e4c0e02375926ab63

Merge "ARM: VIXL32: Fix crash in Exchange for stack slots."
am: a79003958e

Change-Id: I376622aeffbe9641bf86f0e5c557bf6739f2fb55

Merge "ART: Make method handle runtime code callable from compiler."

Merge "Remove bogus DCHECK."

Merge "ARM: VIXL32: Fix crash in Exchange for stack slots."

Merge "Actually record arrays in classpath."
am: a1d0d63614

Change-Id: Iad707649501c3bcb9bbe591b147666f130345fc4

Merge "Actually record arrays in classpath."

ART: Make method handle runtime code callable from compiler.

Most of this change is moving the existing method handles code, but it
also introduces a new header file, common_dex_operations.h, that has
some operations taken from interpreter_common.{h,cc} that are also used
by method handles (perform call, set field, get field).

Bug: 30550796
Test: m test-art-host
Change-Id: I2235e13770a5562950f2767f65a25ca273479150

Merge "Prevent moving GC from running during redefinition."
am: b51edffe1c

Change-Id: I490c0709510a8ced17316baccc4bae9b7aad72ca

Merge "Prevent moving GC from running during redefinition."

Prevent moving GC from running during redefinition.

This could cause a deadlock during deoptimization.

Test: ART_USE_READ_BARRIER=true ART_READ_BARRIER_TYPE=TABLELOOKUP mma -j40 test-art-host
Change-Id: I57e4f1a50709bf4a1817227913e61f3ef434d04a

Avoid using ThreadList::Dump() in empty checkpoint timeout.

ThreadList::Dump() that uses a checkpoint to dump threads may get
stuck when an empty checkpoint gets stuck and isn't useful when
debugging an empty checkpoint timeout. Instead, directly use
Thread::Dump() for each runnable thread that isn't responding to an
empty checkpoint request.

Bug: 33006388
Bug: 12687968
Test: test-art-host with CC.
Change-Id: If07a205786d469eec9b1f587485c80f54fb84c4d

Merge "Revert "Revert "Cleanup jvmti tests"""
am: 9890a5b579

Change-Id: I44a27f4655bf378c007d7f8814d251e3d04cce77

Merge "Revert "Revert "Cleanup jvmti tests"""

Merge "Add some code info checking in GetCalleeSaveMethodCaller"
am: 6980dc54b9

Change-Id: Ib030b51bb3cc3f38e9e3693dcf92260641b45b3d

Merge "Add some code info checking in GetCalleeSaveMethodCaller"

Revert "Revert "Cleanup jvmti tests""

This reverts commit 2a9d198b9929560c9ab07691b3afbe275483be95.

Test: mma -j40 test-art-host
Test: ART_TEST_RUN_TEST_NDEBUG=true mma -j40 test-art-host

Change-Id: I14bfd02f74f93e4a6b853287fcaa69f1fb027256

Merge "Reduce the number of fences needed for monitors"
am: 8ba6549565

Change-Id: Id737e3646ce29c2d90a0043d03abb09d220960c4

Merge "Reduce the number of fences needed for monitors"

Add some code info checking in GetCalleeSaveMethodCaller

Will print info if the memory region is too small for the stack maps.

Test: test-art-host

Bug: 33924573

Change-Id: I03e0151724d3a0933f3bef004e0d16ba1c6134c4

Merge "Tests on FP Math.abs()"
am: 8a26281ead

Change-Id: I0264a6f44684e53e58830851a448f9f0e006f50c

Merge "Tests on FP Math.abs()"

Merge "Fix unquitable apps when debug enabled."
am: ec107997ab

Change-Id: Ibb5a1ef1ae1a649fbd1aeb48e9337c3e56cb6fa0

Merge "Fix unquitable apps when debug enabled."

Merge "Revert "Revert "Revert "Revert "Basic obsolete methods support"""""
am: 16722603e0

Change-Id: Ib787af7eb142b3f37f9745fce0c19cb2b067c5c4

Merge "Revert "Revert "Revert "Revert "Basic obsolete methods support"""""

Tests on FP Math.abs()

Rationale:
As a "quality of implementation", rather than pure "spec compliance", we
require that Math.abs() clears the sign bit (but changes nothing else)
for all numbers, including NaN (signaling NaN may become quiet though).

These tests ensure that remains the case.

NOTE:
This is a better redo of reverted CL
https://android-review.googlesource.com/#/c/316351/

Test: test-art-host
Bug: 30758343

Change-Id: I2740f9aea19abd82335f1f000d92f1cdb469a1c3

Merge "MIPS64: Implement table-based packed switch"
am: f67dadb555

Change-Id: I33f1422cc5ca728704891da83151ca94d1a983b2

Merge "MIPS64: Implement table-based packed switch"

Merge "Remove racy DCHECK on string dex cache."
am: cda4b75615

Change-Id: Ie61f6bf1922ad441dc6383a0beb4f357521a2094

Merge "Remove racy DCHECK on string dex cache."

Remove racy DCHECK on string dex cache.

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

Actually record arrays in classpath.

With https://android-review.googlesource.com/#/c/316151/,
I was under the wrong impression arrays all have the same
access flags. They actually have the visibility of the inner
most component type.

Therefore, we still need to record visibility of array types whose
innermost component type is in the classpath.

Test: verifier_deps_test
Change-Id: If7b1004efb679e320330258f42ced83b8eedae87

ARM: VIXL32: Fix crash in Exchange for stack slots.

In ParallelMoveResolverARMVIXL::Exchange(int mem1, int mem2)
scratch general purpose register was used without any spilling
(like in StoreToOffset) which led to lack of scratch register
for VLDR with big offset. Now it uses two scratch S-registers.

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

Merge "Refine receiver type when inlining based on inline caches."
am: 168768d203

Change-Id: I827455011a6da3f8db6392da34b29c065c16ba6a

Merge "Refine receiver type when inlining based on inline caches."

Merge "MIPS32: java.lang.String.getChars"
am: 8174621224

Change-Id: I27aba6272c9c3d1af12af308089b4e0fbd72c75d

Merge "MIPS32: java.lang.String.getChars"

Refine receiver type when inlining based on inline caches.

We know we're guarding for a specific type
(eg foo.getClass() == Foo.class), so we can propagate the
exact type when building the callee graph.

Test: ART_TEST_JIT=true test-art-host
Change-Id: I3e1858945afbb651cb8966b2b7d7755eccac09e1

Merge "Revert "Disable 626-const-class-linking for JIT.""
am: f0bfd75c19

Change-Id: Ia712e881e655532f5217fdd4d1347806c7b92ef6

Merge "Revert "Disable 626-const-class-linking for JIT.""

Merge "Fix incorrect string hash value extension during cross-compilation."
am: efe703fa49

Change-Id: Id4a007750b8da2493c91b8177f5673c451df1804

Merge "Fix incorrect string hash value extension during cross-compilation."

Merge "Limit Jack threads during tests"
am: 77e9bddee9

Change-Id: Ie998180dd59076fed3211c7b4aefbe22546ef872

Merge "Limit Jack threads during tests"

Limit Jack threads during tests

In default mode, run Jack with 4 threads to help reduce memory
footprint and thread contention. These are the same settings than
the Android build.

For the buildbot (where the Jack server is disabled), run Jack in
single-threaded mode to reduce the global memory consumption.

Bug: 31737153
Test: make -j32 test-art-host
Change-Id: I6a14efce92ea18dba008e6daa26cd6403e5eb503

Reduce the number of fences needed for monitors

Add the necessary CasWeakAcquire primitives for LockWords.

Have MonitorEnter initially read the lockword using a
memory_order_relaxed operation. In the unlikely case we need more,
compensate with an explicit fence.

In the uncontended case, install the thin lock with Acquire,
rather than SequentiallyConsistent semantics.

Have MonitorExit use a Release instead of SequentiallyConsistent
CAS in the ReadBarrier case. Add TODO for the other case.

Together, these should usually eliminate 3 fences (or acq/rel)
per critical section.

Have Install() only use Release ordering.

Add TODO for inflation spinning, which looks to me like it could be
improved appreciably.

Drive-by fix:

GetMaxSpinsBeforeThinLockInflation spelling

Test: Build for several targets, boot, m art-test-host art-test-target

Change-Id: I2cab09723252065f6365e4234ee3249c69ece888

Merge "ART: Remove dequicken map from VerifiedMethod"
am: 18f7ef3dc0

Change-Id: I4464445f1b3c201249cd40e254d50c62235b0d84

Merge "ART: Remove dequicken map from VerifiedMethod"

ART: Remove dequicken map from VerifiedMethod

Remove now unused VerifiedMethod's dequicken map. Follow-up to
c51c7ca7a779563d153c137c6bf01c3ea532a6c9.

Results on a large well-known app in speed compile with -j4:

Before:
 dex2oat took [...] native alloc=84MB [...] swap=208MB [...]
After:
 dex2oat took [...] native alloc=76MB [...] swap=208MB [...]

Bug: 34053922
Test: m clean-oat-host && m test-art-host
Change-Id: Ie95fd297299a0f5fac0c8c702773ee8372f662b6

Remove bogus DCHECK.

The code may be executed concurrently by multiple threads,
so the flag may already be set.

Test: test-art-host
bug:34074968
Change-Id: I162b90a853ce53e6b7063754bbfce5d68cffc82a

Merge "Revert "Revert "Avoid scratch register exhaustion during ARM64 stack slot moves."""
am: 001cd47ddd

Change-Id: I58e889a010f6683d329755626bc8187ab8b9612a

Merge "Revert "Revert "Avoid scratch register exhaustion during ARM64 stack slot moves."""

Merge "Support non-exact upper bound of HBoundType."
am: b28c8749a5

Change-Id: Id64939c8ed1c9920b314a2d11ef2af56a9bb03df

Revert "Revert "Avoid scratch register exhaustion during ARM64 stack slot moves.""

This reverts commit 122ceb492f37e97d022cba2221a87368f1847f5f.

Test: m test-art-target on ARM64, with and without read barriers/heap poisoning.
Bug: 32545705
Change-Id: I4b447d762082eea8edfabeb070317d274e2f5bd0

Merge "Support non-exact upper bound of HBoundType."

Merge "Do not record dependencies on arrays."
am: 88268f08ad

Change-Id: I60d860cd6a023e5a2b824396d9ea3d709fa41b09

Merge "Do not record dependencies on arrays."

Merge "MIPS64: Drop redundant sign-extensions in integer conversions"
am: 6fc063cd8a

Change-Id: I83e6c3904b12dd6e143129825d7950497cf4069c

Merge "MIPS64: Drop redundant sign-extensions in integer conversions"

Merge "MIPS32: Don't always do a null test in MarkGCCard"
am: 4ca40e89aa

Change-Id: Ic8bdf1a5a7d66b6ed9ef19a8e7f7000ec26c2e85

Merge "MIPS64: Improve integer comparison with constants"
am: 7ee9ee9921

Change-Id: I00bd014b5340d5509b96a205a692ae87211bf66b

Merge "MIPS32: Don't always do a null test in MarkGCCard"

Merge "MIPS64: Improve integer comparison with constants"

Merge changes from topic 'VIXLUpdate'
am: 059802455e

Change-Id: I5b85d6ef53d47d1a5fed422a96795fb81de0a063

ARM: VIXL32: Use a default code buffer capacity of 1Kb.
am: f8d19c2852

Change-Id: I2619d712f7af0127692407bf37a2968c34b868ea

ART: VIXL32: Fix assembler test after VIXL update.
am: f2665fa6e5

Change-Id: I5c6e97bbcdb56aaf66764dd8837fefdbb0270b1e

Merge "MIPS64: Implement branchless HCondition for floats"
am: e2157fb29e

Change-Id: I5cd74fb1b95b2e20c8b3e27918fbf3fb4a6d8b4b

Merge "Disable libcore test that requires IPv6"
am: 0ec9ac0a5f

Change-Id: Ie611e75ecf8659a4cf8b71f33ac05732607ccf51

Merge changes from topic 'VIXLUpdate'

* changes:
  ARM: VIXL32: Use a default code buffer capacity of 1Kb.
  ART: VIXL32: Fix assembler test after VIXL update.

Merge "MIPS64: Implement branchless HCondition for floats"

Merge "Disable libcore test that requires IPv6"

Disable libcore test that requires IPv6

Test: N/A
Bug: 25178637
Bug: 34038136
Change-Id: I14c7451d269ee8f1163369d8c0d2209498466b66

Merge "Clean up VerifiedMethod."
am: b1c41c2431

Change-Id: Ibfb35a9eb02b17f77371a573658ada399bf43b0a

Merge "Clean up VerifiedMethod."

Clean up VerifiedMethod.

Remove unused methods, and devirt map.

Results on a large well-known app in speed compile with -j4:

Before:
 dex2oat took [...] native alloc=149MB [...] swap=208MB (218103808B)
After:
 dex2oat took [...] native alloc=84MB [...] swap=208MB (218103808B)

Bug: 34053922
Test: m clean-oat-host && m test-art-host
Change-Id: I0791c1e4ec7a98ae3c98321c582974db8e26617d

Merge "Revert "Avoid scratch register exhaustion during ARM64 stack slot moves.""
am: db577b6911

Change-Id: If6d1d1c3bedfde8378425b519dd732cb494876ee

Merge "Revert "Avoid scratch register exhaustion during ARM64 stack slot moves.""

Revert "Avoid scratch register exhaustion during ARM64 stack slot moves."

Checker test fails.
Bug: 32545705

This reverts commit 4ec76d28f0f808117272134347abf828eea80b91.

Change-Id: Ief14978596341399404c504ec4ca6b68c54fd63c

Merge "MIPS64: Align register spills on 8-byte boundaries in slow paths"
am: 595375fc44

Change-Id: Ia4e9e9a65f9d3c48532983bff3205acc6590e0e3

Merge "MIPS64: Align register spills on 8-byte boundaries in slow paths"

MIPS32: java.lang.String.getChars

Use memcpy(3) to copy characters under the assumption that memcpy()
has been hand optimized for best performance on the platform being
tested.

Test: run-test --optimizing 020-string
Test: run-test 020-string
Test: run-test --no-prebuild --optimizing 020-string
Test: run-test --no-prebuild 020-string
Test: run-test --optimizing 082-inline-execute
Test: run-test 082-inline-execute
Test: run-test --no-prebuild --optimizing 082-inline-execute
Test: run-test --no-prebuild 082-inline-execute
Test: mma -j2 ART_TEST_OPTIMIZING=true test-art-target-run-test
Test: booted MIPS32R2 emulator.

Note: Tested against both the MIPS32R2, and MIPS64R6 emulators.

Change-Id: I4192cf6244db120c8de5cc4932d4132acfc9740d

Merge "Avoid scratch register exhaustion during ARM64 stack slot moves."
am: 00797355fa

Change-Id: I6edabfc4aa252d8308f6e0baa6b81887fa4357e3

Merge "Avoid scratch register exhaustion during ARM64 stack slot moves."

Avoid scratch register exhaustion during ARM64 stack slot moves.

On ARM64, do not limit the selection of a scratch register
used in a move between two stack slots (or between two
double stack slots) to VIXL's floating-point register pool,
as it contains only one register (D31) and can be exhausted
in some rare cases. Instead, query both the core and the FP
register pools.

Add a regression run-test (626-checker-arm64-scratch-register).

Test: m test-art-target (on ARM64)
Bug: 32545705
Change-Id: I6203a4340e3c8b4f4879c07ed1be4c433c311c0f

Merge "MIPS64: java.lang.String.getChars"
am: 66e3919bc4

Change-Id: I1439b994fbd909923c02e16936dc5a60dfb88ec3

Merge "MIPS64: java.lang.String.getChars"

MIPS64: Implement branchless HCondition for floats

Test: mma test-art-target-run-test64 in QEMU

Change-Id: I595b5b7ddf9ebb19e872ed85f2e4098a835d9214