Merge "Remove no thread suspension assert in AddImageSpace"
am: c71dc3852e

* commit 'c71dc3852edfa8e0cd2083d12845ff8607e189ff':
  Remove no thread suspension assert in AddImageSpace

Merge "Remove no thread suspension assert in AddImageSpace"

Remove no thread suspension assert in AddImageSpace

Heap::VisitObjects may cause thread suspension for the CC case.

Change-Id: Ibd495690b87dc8731919e1b1164be834c9da02b2

Merge "Use MarkIfNotInToSpace for SemiSpace::MarkObject"
am: cad8657a6c

* commit 'cad8657a6c30ec615df5bf8a59a4222f4849d0a2':
  Use MarkIfNotInToSpace for SemiSpace::MarkObject

Merge "Use MarkIfNotInToSpace for SemiSpace::MarkObject"

Use MarkIfNotInToSpace for SemiSpace::MarkObject

SemiSpace::MarkObject is called from ModUnionScanImageRootVisitor
which. These roots may be visited multiple times by the GC. One
example is the allocation stack traces and a class visiting the same
ArtMethod twice.

Bug: 27435111
Change-Id: I721b8bcf7fe43f4c351d902c2f3422d3bb585295

Merge "Only visit app image classes in class loader"
am: 42e84e007e

* commit '42e84e007ea69eefd871646a863cdae796ae3428':
  Only visit app image classes in class loader

Merge "Only visit app image classes in class loader"

Only visit app image classes in class loader

Only update dex cache arrays of added classes since the declaring
class is in image DCHECK fails for other classes in the class loader.

Also some cleanup to prevent app images leaving invalid state if
they get rejected.

Bug: 22858531
Bug: 27431418
Change-Id: Ib2a5692a1ad78b014a1bfc6b27fb1c12bc8565e6

Merge "ART: Fix UTF test and monitor pool old chunks"
am: 3040f4ae0b

* commit '3040f4ae0bf9d8ebe0e7e43f4843929f7a4da4d9':
  ART: Fix UTF test and monitor pool old chunks

Merge "ART: Fix UTF test and monitor pool old chunks"

Merge "Cache DexRegisterMaps when writing native debug info."
am: 1c6f902c28

* commit '1c6f902c28f7090cbc88be79119e2151edfea883':
  Cache DexRegisterMaps when writing native debug info.

Merge "Cache DexRegisterMaps when writing native debug info."

Merge "Query declaring class before querying IsNative."
am: b4d1d7217c

* commit 'b4d1d7217cf13e199650f1d8d18b3176714f6500':
  Query declaring class before querying IsNative.

Merge "ART: Update arm assembly to use current syntax"
am: fd409de54c

* commit 'fd409de54c59fda03453f4b1558aa82679b891af':
  ART: Update arm assembly to use current syntax

Merge "Query declaring class before querying IsNative."

Query declaring class before querying IsNative.

IsNative checks that the declaring class is not null.

Change-Id: I9b775858cdb685eb633b3d8a8a271a3d451bb8a9

Merge "ART: Update arm assembly to use current syntax"

Merge "Fix very rare bug around JIT code cache collection."
am: 03f9e6bbda

* commit '03f9e6bbdaf3cd6367176afe373ec6aab27b7c46':
  Fix very rare bug around JIT code cache collection.

Merge "Fix very rare bug around JIT code cache collection."

Merge "Ignore 145-alloc-tracking-stress failures in interpreter with CC."
am: dd6c73a3b5

* commit 'dd6c73a3b5a7eda84ede063832a5a12dff6f7ccd':
  Ignore 145-alloc-tracking-stress failures in interpreter with CC.

Merge "ARM64: Implement SystemArrayCopyChar intrinsic."
am: f51a235db3

* commit 'f51a235db3cc167db27a7c447a28655e7e37c320':
  ARM64: Implement SystemArrayCopyChar intrinsic.

Merge "Ignore 145-alloc-tracking-stress failures in interpreter with CC."

Merge "ARM64: Implement SystemArrayCopyChar intrinsic."

ART: Update arm assembly to use current syntax

Some of the ancient mterp code uses old-style arm opcode names
which are supported by gas, but not by clang's integrated
assembler.

Partial fix for internal b/27473367

Change-Id: Ic6128b98dfbf30f252e7487f802e9dfbe0a42b6a

Ignore 145-alloc-tracking-stress failures in interpreter with CC.

This run-test fails also with the interpreter on the
concurrent collector configuration; disable it for now.

Bug: 27467554
Change-Id: I54f76536b498ea2dedfd76d1bd77e1e26b415eb2

Merge "Make sure we keep adb logs on buildbots."
am: 867a7c7330

* commit '867a7c7330b74d258878894364c9493bf803aace':
  Make sure we keep adb logs on buildbots.

Cache DexRegisterMaps when writing native debug info.

I might make the function more expensive in the future so I want
to make sure it gets called only the minimum number of times.

Change-Id: I1d09ecf1db7b54d28aaa11a152226d469f514fe7

Merge "Make sure we keep adb logs on buildbots."

Merge "Re-enable CFI test for compressed symbols."
am: 04ca0a728d

* commit '04ca0a728db465311f1bcf43e87df39dcc104fcd':
  Re-enable CFI test for compressed symbols.

Merge "Re-enable CFI test for compressed symbols."

Re-enable CFI test for compressed symbols.

This enables the test for remote unwinding (modelling debuggerd),
but it does not test local unwinding since it is disabled for
performance reasons in libunwind.

Bug: 27391690
Change-Id: I28451576c421bbd70259d757332a178f29bf3646

Make sure we keep adb logs on buildbots.

Change-Id: I79239967a74eaef6d10b546407b223422741d157

Fix very rare bug around JIT code cache collection.

The bug is the following:
1) JIT thread: We start a code cache collection.
2) JIT thread: We mark all code that is in the call stack of all
   threads.
3) Mutator thread: after marking its stack, resumes and does call
   that pushes JIT compiled code to the call stack.
4) Mutator thread: deoptimizes compiled code of ArtMethod Foo,
   and therefore updates the entry point of Foo through
   JitCodeCache::InvalidateCompiledCodeFor.
   (Note that updating the entrypoint could also be done through
   instrumentation).
5) JIT thread: Call JitCodeCache::RemoveUnusedAndUnmarkedCode.
   The method used to remove entries that were not entrypoints.
   It sees the compiled code for Foo but that is not an entrypoint
   anymore, so deletes it.
6) Mutator thread problem: it now has compiled code in its call
   stack that is deleted.

If it's only one mutator thread, we only hit a DCHECK when walking
the stack, as we are now seeing an invalid pc. The deoptimization
will longjmp to the caller of that invalid entry anyway.

However, if multiple mutator threads are involved, one thread
might invalidate the compiled code while the other is still
running it. And we end up deleting code that is in the call
stack of a thread, and we will crash.

The fix is to mark entrypoints before marking call stacks,
so that anything a thread might jump to is marked and kept.

bug:27424509
bug:23128949
bug:26846185

Change-Id: I07cd08cedd96b9900629f7535e95404f622104ea

Merge "Suppress ArrayDequeTest#testForEachRemaining temporarily."
am: e8e56f26fb

* commit 'e8e56f26fbaf8c3cd5ab046f8e13d24de494ffda':
  Suppress ArrayDequeTest#testForEachRemaining temporarily.

Merge "Suppress ArrayDequeTest#testForEachRemaining temporarily."

Suppress ArrayDequeTest#testForEachRemaining temporarily.

Change-Id: I297a392d694b8143010084469c401f15a887f2f1

ARM64: Implement SystemArrayCopyChar intrinsic.

Change-Id: I33f559139a38ddf20cacb8c997e38fa7663a4066

Merge "Refine statistics around the JIT."
am: a3ed89f687

* commit 'a3ed89f687dbaa9b5321b2d38d41b487a4208b80':
  Refine statistics around the JIT.

Merge "Move back the boot image to debuggable."
am: 104f22ec88

* commit '104f22ec881a02b4df88d41e1b89e24151b66cc9':
  Move back the boot image to debuggable.

Merge "Refine statistics around the JIT."

Merge "Move back the boot image to debuggable."

Move back the boot image to debuggable.

Full frame deoptimization is broken with it.

Partial revert of https://android-review.googlesource.com/#/c/201383/

Change-Id: I7a402d79b0882f81987e56869551840da7d553e0

Merge "Fix lint issue."
am: b138a6ee7f

* commit 'b138a6ee7f1c1c172d9646547ad5def80db048e1':
  Fix lint issue.

Merge "Fix lint issue."

Fix lint issue.

Change-Id: I549cc641510a7f941d85f3a5f38127bc6701a0a3

Merge "Integer.bitCount and Long.bitCount intrinsics for ARM64"
am: dd11dde40a

* commit 'dd11dde40a9002eaebb977fc754f64d94ec4e60c':
  Integer.bitCount and Long.bitCount intrinsics for ARM64

Merge "Integer.bitCount and Long.bitCount intrinsics for ARM64"

Merge "ProfilingInfo roots should be visited by the declaring class."
am: 9ec0676c8e

* commit '9ec0676c8e17664926055e40a7283074dd9b3474':
  ProfilingInfo roots should be visited by the declaring class.

Merge "ProfilingInfo roots should be visited by the declaring class."

Merge "MIPS32: Highest/Lowest One Bit Set:"
am: 1028909c89

* commit '1028909c89640c89ab59570b24cb60a4342cdc31':
  MIPS32: Highest/Lowest One Bit Set:

Merge "MIPS32: Highest/Lowest One Bit Set:"

MIPS32: Highest/Lowest One Bit Set:

- int java.lang.Integer.highestOneBit(int)
- int java.lang.Integer.lowestOneBit(int)
- long java.lang.Long.highestOneBit(long)
- long java.lang.Long.lowestOneBit(long)

Change-Id: I575ec6f70cf5cac620cc5d9e49ec3f7b554b4622

ART: Fix UTF test and monitor pool old chunks

Zero-initialize buffers in utf_test. UTF conversion does not
append a zero terminator, but the test uses strlen to check the
byte count.

Move to unique_ptr for storing old monitor-pool chunk data to
fix a leak.

Bug: 27156726
Change-Id: I14424c6f98cf47beab6243f83a335bd6a682c638

Merge "Add ScopedGCCriticalSection to Trace::StopTracing."
am: e330de4226

* commit 'e330de4226a25f81678edb9b1a5140bf1d7b3b5c':
  Add ScopedGCCriticalSection to Trace::StopTracing.

Merge "MIPS32: Improve TrailingZeros"
am: 3b13d9408f

* commit '3b13d9408f22c15ca7773fade2e55c6e4271a810':
  MIPS32: Improve TrailingZeros

Merge "Add ScopedGCCriticalSection to Trace::StopTracing."

Merge "MIPS32: Improve TrailingZeros"

Merge "MIPS32: Implement bitCount intrinsics."
am: 7891581ad7

* commit '7891581ad7653f8d9d3447872a5495af7d17d076':
  MIPS32: Implement bitCount intrinsics.

MIPS32: Improve TrailingZeros

Change-Id: I1269ad5c58e1e3f39f3c2a04a3ed6dc1e44ff2fc

Merge "MIPS32: Implement bitCount intrinsics."

MIPS32: Implement bitCount intrinsics.

- int java.lang.Integer.bitCount(int)
- int java.lang.Long.bitCount(long)

Change-Id: Ibde089675064aa052cfcd9f53aa36c25a4cbfca9

Merge "MIPS32: Implement isInfinite intrinsics."
am: 00bc2d3413

* commit '00bc2d3413ed4b976dd9313c178fb6c86b66883c':
  MIPS32: Implement isInfinite intrinsics.

Merge "MIPS32: Implement isInfinite intrinsics."

Add ScopedGCCriticalSection to Trace::StopTracing.

This attempts to fix a deadlock in 545-tracing-and-jit with the read
barrier config. I think the cause is the same as what CL 174096
describes and it missed this one in Trace::StopTracing.

Bug: 26429931
Bug: 12687968
Change-Id: I02ca45e2667c9f6afc27d73919d6c31d204cc621

ProfilingInfo roots should be visited by the declaring class.

There seems to be an implicit assumption that only classes can
visit native roots (ie kVisitNativeRoots in Class::VisitReferences).
However, some places like Dbg::VisitRoots and
AllocRecordObjectMap::VisitRoots visit ArtMethod::VisitRoot
directly, and that breaks the assumptions.

bug:27435111
Change-Id: I5b476e614ba820394635d946cb562bf872a50e7e

Merge "Add systrace to VerifyClass"
am: 4a01be2b1a

* commit '4a01be2b1a7de5ce68cb5a453eb7cb4fc6c6658b':
  Add systrace to VerifyClass

Merge "Move some default-methods tests to Java from Smali."
am: 66bdb9e03e

* commit '66bdb9e03efa2055b6bfbe0a07ee873afe673fae':
  Move some default-methods tests to Java from Smali.

Merge "Add systrace to VerifyClass"

Add systrace to VerifyClass

Useful to see if verifier is preempting anything.

(cherry picked from commit 7f459f5859e5f01c92c3711e7d63c062f510b416)

Change-Id: I594cbdc5c2f8934fc6c4fc7ee47fc4ccfc3c627d

MIPS32: Implement isInfinite intrinsics.

- boolean java.lang.Float.isInfinite(float)
- boolean java.lang.Double.isInfinite(double)

Change-Id: I17dc2380ec864fd7612025ed400e29dd115ccab4

Merge "Move some default-methods tests to Java from Smali."

Move some default-methods tests to Java from Smali.

Move all smali tests for default method behavior in
non-source-incompatible contexts to java. Also move some of the
simpler tests for source and binary incompatibilities into java as
well when possible.

Bug: 27310767

Change-Id: I753196f19849494825953c1bf06f15b7132f459b

Merge "change image in run-jdwp-tests.sh"
am: 93504f2f89

* commit '93504f2f891959c1606bd033a86310613e60cb1b':
  change image in run-jdwp-tests.sh

Merge "Add a PassScope for PrepareForRegisterAllocation"
am: b204d9444d

* commit 'b204d9444d7c8bc6bc5e60d791060525b91b520c':
  Add a PassScope for PrepareForRegisterAllocation

Merge "change image in run-jdwp-tests.sh"

Merge "Add a PassScope for PrepareForRegisterAllocation"

Merge "Simplify LoadNativeLibrary()"
am: 9bab082adb

* commit '9bab082adb251327b806d537d6710c16b9897de9':
  Simplify LoadNativeLibrary()

Merge "Simplify LoadNativeLibrary()"

Refine statistics around the JIT.

- Better namings.
- Now also time the code cache collection time.
- Random cleanups.

bug:23128949
bug:27445008
bug:27442890

Change-Id: I1dd52544bea678af868e7c47907f7a0fc9a146c3

Merge "ART: cleanup exit_block_ in graph if exit block is removed"
am: 19a133b2e2

* commit '19a133b2e26770b900119aaac25e3e73bd287e3f':
  ART: cleanup exit_block_ in graph if exit block is removed

Merge "ART: cleanup exit_block_ in graph if exit block is removed"

Merge "Ignore 145-alloc-tracking-stress run-test failures with CC."
am: 982a1ac93d

* commit '982a1ac93d03379fc82f4f5b1a1b13eeea035d5b':
  Ignore 145-alloc-tracking-stress run-test failures with CC.

Integer.bitCount and Long.bitCount intrinsics for ARM64

Change-Id: If6180acc90239e52e5d33901b65e194d1ca7e248

Merge "Ignore 145-alloc-tracking-stress run-test failures with CC."

ART: cleanup exit_block_ in graph if exit block is removed

If we remove the exit block from the graph (for example method
contains infinite loop) we should also clean the field exit_block_
in graph. At least inliner expects it.

Change-Id: Icda668da2233cdd6cd673635a1949f5ed34cf270
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>

Ignore 145-alloc-tracking-stress run-test failures with CC.

This run-test sometimes fails with a timeout on ART
Builbot's x86-64 concurrent collector configuration. Disable
it to make the build turn green again, while we investigate
the failure.

Bug: 27467554
Change-Id: I0bac307ec85f197afb8763ca41a6ee50567c4d5f

Merge "Do a TryLock when allocating a ProfilingInfo from the interpreter."
am: e283588e56

* commit 'e283588e56df7eebabd795b98264003105b20ece':
  Do a TryLock when allocating a ProfilingInfo from the interpreter.

Merge "Do a TryLock when allocating a ProfilingInfo from the interpreter."

Add a PassScope for PrepareForRegisterAllocation

This pass does transform the graph so make it part of cfg-dumping.

Change-Id: I42e361382c85c97b974faad8bb0fcf2cb0750355

Merge "Delete alloc tracking map outside of critical section"
am: e4cec517fa

* commit 'e4cec517faf1c74d6895e0e5b512a6e2f83a0e06':
  Delete alloc tracking map outside of critical section

Merge "Delete alloc tracking map outside of critical section"

Delete alloc tracking map outside of critical section

There can be lock order violations otherwise due to runtime shutdown
lock that may get acquired in the condition variable destructor.

Change-Id: I23cb2dfe241f5cc6c42bf6766e89042cf06069b6

Merge "Revert "Disable test after libunwind change.""
am: 0b136ac861

* commit '0b136ac861fbf9e8b97ded8a7a3ae316f46078e4':
  Revert "Disable test after libunwind change."

Merge "Disable 577-profile-foreign-dex temporarily."
am: b93457153d

* commit 'b93457153d3ee0ba51d45819aa8ac34953ac5d1f':
  Disable 577-profile-foreign-dex temporarily.

change image in run-jdwp-tests.sh

Change-Id: I305fdd5b8b33c446529fe99f872af4fb8685bd7c

Merge "Revert "Disable test after libunwind change.""

Revert "Disable test after libunwind change."

This reverts commit 18047e4cd06387958d315d4de6d9c1753a08ee9e.

Moving the in-process lzma disable to libunwind allows to unwind not-zipped debug-info, again.

Bug: 27391690
Change-Id: I8c24dd0e73bc9d57ce83c6f5a8272d8252e5b809

Merge "Disable 577-profile-foreign-dex temporarily."

Merge "Fast ART MIPS64 interpreter"
am: b5e8ade3b6

* commit 'b5e8ade3b6bf2162a885133f2ac728c578c3c2b0':
  Fast ART MIPS64 interpreter