Also compile libnetd_client for target testing.

The runtime linker will load the one on /system otherwise.

Change-Id: Id57e84d1b34e807437818d39bf6ef5f727a7d210

Merge "Disable 149-suspend-all-stress, its output is flaky."

Disable 149-suspend-all-stress, its output is flaky.

bug:28988206
Change-Id: I13cce14b72d753c9ba2ca9b4e256eb571620caa9

Merge "Revert "ART: Disable check under memory tools""

Revert "ART: Disable check under memory tools"

This reverts commit 820c13bea5795e04986bf617319517562de76514.

The check is valid, the whole stack setup is broken.

Merge "ART: Suppress memcpy overlap in jemalloc for valgrind"

ART: Suppress memcpy overlap in jemalloc for valgrind

Clang/LLVM uses memcpy for a self-assign, which is detected by
valgrind. Ignore.

Bug: 29279679
Change-Id: I47483e1a2a982b9f3642f98e4389633989885bb8

Merge "Use ScopedObjectAccess in ThreadList::Dump"

Use ScopedObjectAccess in ThreadList::Dump

Prevent deadlocks if two different threads are calling Dump at the
same time:

Thread 1: Requests thread 2 to suspend
Thread 2: Requests thread 1 to suspend
Both threads are suspended and blocked on ScopedObjectAccess in
thread dumping.

Added to suspend all stress, reduced thread count to 4 temporarily
since this is the maximum number of simultaneous checkpoints.

Bug: 28988206

Change-Id: I9e4b8391ebad0bca0e42a03819c58c99ddba6b35

Merge "ART: Disable check under memory tools"

Merge "Fix the region space name for dumpsys meminfo."

Merge "Disable the CC collector / read barrier checks in non-debug build."

ART: Disable check under memory tools

Disable a stack check under memory tools. It assumes a single
common stack, which seems violated at least under valgrind on
ARM.

Change-Id: Ida05adb0728cc46f3bbda4d34be7a1623bb3bc82

Fix the region space name for dumpsys meminfo.

Bug: 12687968
Change-Id: I42e3d4cd9afb9892eb1d173b18f8b3e9491429dd

Disable the CC collector / read barrier checks in non-debug build.

Bug: 12687968

Change-Id: Ia8295354b705018ffa864eb8101aa5c09528af13

Merge "ART: Fix swap usage determination in dex2oat"

ART: Fix swap usage determination in dex2oat

The decision needs to be postponed until we have access to the
dex files.

Bug: 27878043

(cherry picked from commit ce55f0d4da1db20676273032b3c79624f665b35b)

Change-Id: I55833c6ea827ec7a322711bd182fc723d1727f08

Merge "Fix dex file leak in oat file manager"

Fix dex file leak in oat file manager

Simplified ownership by having a vector of unique pointers own
all newly opened dex files.

Bug: 29246280

(cherry picked from commit fed316715faeec7bf34e3c4a878288c1342cb0e8)

Change-Id: I97db09ced76db8ffdbae371ff72977c4276a0494

Merge "ART: Add ATRACE section for collision check"

ART: Add ATRACE section for collision check

The collision check for duplicate classes is potentially expensive.
Add a ScopedTrace to mark it.

Bug: 26880306
Change-Id: Icd75bd0ccc15f0d1f321d85382a4d038fa8ab049

Merge "Add Valgrind target tests"

Add Valgrind target tests

Change-Id: I1251ac2122c60ed5fcabf184704bc439243ee85b

Merge "Fix some "possible" divide by 0"

Fix some "possible" divide by 0

Bug: 28529431
Change-Id: I61f638926b2ae63c5f883fc2cfdce19b00ce79c8

Merge "Add thread suspend stress"

Merge "Re-introduce an old version of 600-verifier-fails."

Re-introduce an old version of 600-verifier-fails.

The fugu buildbot had a crash on this version:
https://build.chromium.org/p/client.art/builders/fugu-ndebug/builds/3844

It got lost after updates to the tests.

Change-Id: Ib3748e6d6772e90bbc56b8139e374c7cf8953ce4

Merge "Ensure we have a profiling info object before trying to compile."

Ensure we have a profiling info object before trying to compile.

Otherwise we will never succeed compiling and infinite loop.

Change-Id: I7ff09c69e67f0df4cb450ffa10a529625eac67fd

Merge "Also catch OOME from the new byte[]."

Also catch OOME from the new byte[].

Change-Id: Ic756070ed4a3cb3760760fd5a2ebcad1ea0a8ca0

Merge "Fix FOUR_ARG_DOWNCALL assembly stubs on arm and x86."

Merge "Use usleep instead of sleep(0)."

Fix FOUR_ARG_DOWNCALL assembly stubs on arm and x86.

They were creating a stack that the runtime did not understand.

bug:28348339
Change-Id: Ic03663552209beda8ff1e79db58bedc8f34d9a0e

Use usleep instead of sleep(0).

sleep(0) is unfortunately not specified.

Change-Id: I5e9c626d21a562ca04329c0cc039a1f3a71dfc84

Merge "Bump frame size limit 8 bytes to fix x86_64 build with latest external/libcxx"

Bump frame size limit 8 bytes to fix x86_64 build with latest external/libcxx

Change-Id: Ib8964d18aa509ee40a91d92eea1011447d429ed7

Merge "Revert "Revert "Revert "Revert "Revert some flaky unloading"""""

Add thread suspend stress

Seems to be passing, aim to catch bugs related to thread suspension.

Bug: 28988206

Change-Id: I48190900ccd7a2545c6f09cf730e995043b0d9b7

Revert "Revert "Revert "Revert "Revert some flaky unloading""""

Fugu is still unhappy

Bug: 28406866

This reverts commit 340f486aa0126facb67494449b5c2ee46a1a75e6.

Change-Id: I45fc77f924d991669d27b99c1458b2def8692664

Merge "Temporarily disable test E"

Merge "Add a few more pending exception checks"

Add a few more pending exception checks

Bug: 28371539

Change-Id: Ibd2f7ccf7bd8cf7ef6c24534be1efbcc9b30014a

Temporarily disable test E

Rationale:
Interpreter does not seem to perform access checks
prior to null check, a feature that was just fixed
in the AOT compiler. Disabling the test until we
figure out who is right.....

BUG=29068831

Change-Id: I2cdce568727e631f48648f401806e7ecd6985038

Merge "Revert "Revert "Revert "Revert some flaky unloading""""

Revert "Revert "Revert "Revert some flaky unloading"""

Ensure that all threads are in the JNI code before destroying the
runtime.

Bug: 28406866

This reverts commit b2716bbf321293ddbd0b96f0ea999794b0e9b09b.

Change-Id: I7f6189a9d0d8a1bf7af5535026804bc2183466c9

Merge "Do not place null check on unresolved method calls."

Do not place null check on unresolved method calls.

Rationale:
These invokes drop through the runtime anyway where various
checks are done, including null check. A few of these
checks need to occur before the null check.

With fail-before/pass-after smali test.

BUG=29068831

Change-Id: I260715e742365433a323598d97f7fdab321e8512

Merge "Use sleep instead of sched_yield."

Use sleep instead of sched_yield.

The compiler thread has a different priority than the main thread,
so there's no guarantee sched_yield will actually pass the CPU to
the compiler thread.

Change-Id: I7e8d3cbf478c482a9ba9502f7bba7fb1397cd28e

Merge "Be pro-active into ensuring a method is JITted in test."

Be pro-active into ensuring a method is JITted in test.

Change-Id: I27d7bb76178478e6153e87a076bb39a4dc61f50b

Merge "Wrap certain exception types when loading an erroneous class."

Wrap certain exception types when loading an erroneous class.

Bug: 28787733

(cherry-picked from commit 7c8aa8357196781c811a73d2eb66aaaa1681ce36)

Change-Id: Iea55486c4b95ee16e1f19c8ba2d24c18b9100c97

Merge "Enable profman pretty printing"

Enable profman pretty printing

Bug: 28748264
Change-Id: I4dc366aa710c9ef05aa5d1b164d558232fe57a3d
(cherry picked from commit 45968e76fffdde6a99a462056a52e91c12f4172a)

Merge "Initial profman support for verbose dump"

Merge "Do not hide instance field hard failure with soft failure"

Initial profman support for verbose dump

Bug: 28748264
Change-Id: Ibc59e7f31afa4646fcf048eececb87437dd83de9
(cherry picked from commit 07f468f117c4c30a032b7884c08cb1dd2964444e)

Do not hide instance field hard failure with soft failure

Rationale:
Yet another verifier inaccuracy found with fuzz testing.
Instance field verification should proceed testing instance
field access after soft failures in cases where hard failures
could still follow. Failure to do so resulted in a compiler
crash (now made bit friendly with DCHECK as well).

With crash-before/pass-after test.

BUG=29126870

Change-Id: I8674d6171158eaa2aeb0492b35dfafea76416cac

Merge "Revert "Revert "Fix ArtMethod::GetInvokeType for static methods on interfaces."""

Merge "ART: Remove redundant MoveInstructionBefore method"

Revert "Revert "Fix ArtMethod::GetInvokeType for static methods on interfaces.""

bug:29089267
bug:27521545

We were hitting a compiler DCHECK that a class would never require to
do access checks on itself. The reason was that the compiler driver
was not trying to resolve a type, but instead relied on the verifier
for pre-populating the dex cache. However, the verifier doesn't
necessarily run in JIT mode.

This reverts commit 12abcbd950bd0ff4528e2e0d27ca5e881c7b0467.

Change-Id: I59204c16927084f6605a2a3f999ca529f949e1ad

ART: Remove redundant MoveInstructionBefore method

Change-Id: If53d7011197cc6b9c1702a3d98ef11b59eb76f0c

Merge "MIPS32: Improve method entry/exit code"

Merge "Revert "Revert "Revert some flaky unloading"""

Revert "Revert "Revert some flaky unloading""

Still some issues with 136-daemon-jni-shutdown

bug:28406866

This reverts commit 9e726e874580f1e8b2e0bfa63d2e9a3ac911cd1c.

Change-Id: I76e7ba47e30cf97ecd074d5c2aff41cda8edac59

Merge "ART: ArrayGet hoisting restriction added."

Merge changes Iba55937d,Ib5d82bde,Id0c921bd

* changes:
  Revert "Revert "Revert "Temporarily disable dex cache array emptyness DCHECK"""
  Revert "Revert "Hold dex caches live in class table""
  Revert "Hold dex caches live in class table"

Revert "Revert "Revert "Temporarily disable dex cache array emptyness DCHECK"""

Bug: 29083330

This reverts commit 46d46993cc0f11c2ac1b7ac8ffd55d78111faf48.

Change-Id: Iba55937de9c4f963ed084093c047d1550f994742

Revert "Revert "Hold dex caches live in class table""

Bug: 29083330

This reverts commit f102faf1bcbdb2149e3e7bf27b1819f621b7894b.

(cherry picked from commit c9dbb1df3b5c06ba122cacaf35b17cb53c6be3c6)

Change-Id: Ib5d82bde2e4a031c1cab0ea6116925ef99d9f07f

Revert "Hold dex caches live in class table"

This reverts commit d6d49e56c2b7b11f474acb80cb02bb1fe9b7861e.

Bug: 29083330

(cherry picked from commit f102faf1bcbdb2149e3e7bf27b1819f621b7894b)

Change-Id: Id0c921bd6fe422159f8daa598dc8e7c9ed6eca47

Merge "Revert "Revert some flaky unloading""

ART: ArrayGet hoisting restriction added.

Currently if we hoist ArrayGet from loop there is no guarantee
that insn will be executed at runtime. Because of that we could
face issues like crashes in generated code.

This patch introduces restriction for ArrayGet hoisting. We say
that ArrayGet execution is guaranteed at least one time if its bb
dominates all exit blocks.

Change-Id: I9f72c0f4c33b358341109238bea46cb5a82f490f
Signed-off-by: Anton Shamin <anton.shamin@intel.com>

Revert "Revert some flaky unloading"

This reverts commit 73ad16e0193866d36dbd4088ac77fa5d4ceec334.

With this code it is possible that during testing we will race the
final shutdown of some threads (i.e. returning to thread_start after
signaling thread has been finished) with the unloading of libart.so by
libnativehelper. This could cause crashes. By ensuring that libart.so
code is not unloaded in libnativehelper we fix the issue with this
patch.

Bug: 28406866

Change-Id: Ic67118c2a4e743941402b17b6bbfe4cce441f68f

Merge "ART: Improve JitProfiling perf in mips/mips64 mterp."

Merge "Revert "Fix ArtMethod::GetInvokeType for static methods on interfaces.""

Revert "Fix ArtMethod::GetInvokeType for static methods on interfaces."

bug:29089267
bug:27521545

Fails some DHECKs.

This reverts commit 808067335b228d7b50ad84123d3c8ecb7aeeb200.

Change-Id: I0e768ce85be593e3f50fd02abc29aa34f2be3562

Merge "Fix ArtMethod::GetInvokeType for static methods on interfaces."

Fix ArtMethod::GetInvokeType for static methods on interfaces.

And avoid calling ResolveMethod in the JIT, since it already
knows that method.

bug:29089267
bug:27521545

(cherry picked from commit 2dc77ecf375882f51ff7c09712c05b80e58abb6b)

Change-Id: I36084b1f207317452c42fdfc8ffa4d8c721d2f76

Merge "Revert "Revert "Temporarily disable dex cache array emptyness DCHECK"""

Revert "Revert "Temporarily disable dex cache array emptyness DCHECK""

This fix has been reverted.

Bug: 29083330

This reverts commit 06cd763b7d2a7511ea4a54b52b2931e57f3522d3.

Change-Id: I5382c872caeb7e699de41acaa54ac3ee9f0543e0

MIPS32: Improve method entry/exit code

Improvements:
- the stack frame is (de)allocated in one step instead of two
- callee-saved FPU registers are 8-byte aligned within the frame,
  allowing a single ldc1/sdc1 instruction to load/store an FPU
  register without causing exceptions due to misaligned accesses
- the return address register, RA, is restored early for better
  instruction scheduling

Change-Id: I556b139c62839490a9fdbce8c5e6e3e2d1cc7bb7

ART: Improve JitProfiling perf in mips/mips64 mterp.

Change-Id: I4e1a214d92bd17ebd0a9b595e2eca2d7dcc13758

Merge "Do not place null check from unresolved field access."

Do not place null check from unresolved field access.

Rationale:
These accesses go though the runtime anyway where various
checks are done, including null check. Since particular
checks, like access checks, need to occur prior to the
null check (to ensure link errors are not masked by
a null reference), the explicit null check should not
occur in the HIR.

BUG=29068831

Change-Id: I30fc9cb8cf4993e4176e235ceba3a38aef98d503

Merge "Revert "Temporarily disable dex cache array emptyness DCHECK""

Merge "Fix an issue of partial fragment deoptimization"

Revert "Temporarily disable dex cache array emptyness DCHECK"

Bug: 29083330

This reverts commit 7b1541f4aecdc4c2878437ac3d46f2e74eacf0f1.

Merge "Hold dex caches live in class table"

Merge "Revert "Revert "Make the boot image non-debuggable."""

Revert "Revert "Make the boot image non-debuggable.""

The fix for b/29043547 has been submitted.

Bug: 28769520

This reverts commit cc98f634847914d306d9c1ae16c46b95e0645270.

Change-Id: I6ed511c0024ecaee75740e2847d1be20c4fb07dd

Merge "Delay dex-to-dex compilation until Optimizing is done."

Merge "Refactor handling of input records."

Delay dex-to-dex compilation until Optimizing is done.

This fixes a race between inlining in the Optimizing
backend and dex-to-dex quickening where the Optimizing can
read the non-quickened opcode and then the quickened field
index or vtable index and look up the wrong field or method.
Even if we such tearing of the dex instruction does not
happen, the possible reordering of dex-to-dex and Optimizing
compilation makes the final oat file non-deterministic.

Also, remove VerificationResults::RemoveVerifiedMethod() as
we have only the Optimizing backend now and as such it was
dead code and would have interfered with this change.

Bug: 29043547
Bug: 29089975
Change-Id: I8389927d35dcacaf2f99c2153f055857036c8129

Fix an issue of partial fragment deoptimization

If the interpreter bridge returns to the instrumentation exit stub,
we need to let the stub makes the decision whether the calling code
is deoptimizeable or not since the stub knows the real return pc.

Bug: 28769520
Change-Id: I262d1222e50a1ccbcb3675d05dcab414dc242a28

Hold dex caches live in class table

Prevents temporary dex caches being unloaded for the same dex file.
Usually this is OK, but if someone resolved a string in that dex
cache, it could leave stale pointers in BSS. Also it can use extra
memory in linear alloc if we allocate dex cache arrays multiple
times.

Bug: 29083330
Change-Id: Ia44668f013ceef1f5eb80f653a48d0f8004548c9

Merge "Exit 0 on success even if --never-clean is passed"

Merge "Stop creating $ANDROID_BUILD_TOP/{&&,mkdir} directories."