Merge "ART: Do not build Checker tests with `dx --no-optimize`"
am: 4dd25bf0f8

* commit '4dd25bf0f83d36ea7ce570ac4680ce4870808d91':
  ART: Do not build Checker tests with `dx --no-optimize`

Merge "ART: Do not build Checker tests with `dx --no-optimize`"

Merge "Revert "Disable test while investigating.""
am: 57e6fb09c0

* commit '57e6fb09c09f5fe755703657cacfb39681b09577':
  Revert "Disable test while investigating."

Merge "Revert "Disable test while investigating.""

ART: Do not build Checker tests with `dx --no-optimize`

Change-Id: Ice9298b92cde5486db40ccdef43cca6d36179d2c

Revert "Disable test while investigating."

Change #195560 fixes the test.
This reverts commit 1c58d6980541eb469f4860f00a77ba8c729efef9.

Change-Id: Idbfb0c5d2fbc1796f7dd1056181b5fc912a065a3

Merge "Suppress jsr166.LinkedTransferQueueTest#testWaitingConsumer."
am: a9656fad4e

* commit 'a9656fad4e14ab2cc73f8c739322f875d02be2b2':
  Suppress jsr166.LinkedTransferQueueTest#testWaitingConsumer.

Merge "Suppress jsr166.LinkedTransferQueueTest#testWaitingConsumer."

Suppress jsr166.LinkedTransferQueueTest#testWaitingConsumer.

To make the buildbot read barrier config greener for now.

Bug: 25883050
Change-Id: I5ce837e7783dd2c32825993d8dd754741839261d

Merge "Revert "Revert "Reduce sleep duration in SuspendAllDaemonThreads"""
am: c1c8e73671

* commit 'c1c8e736719cbbd47ff836e84cfc1803f424462e':
  Revert "Revert "Reduce sleep duration in SuspendAllDaemonThreads""

Merge "Improve handling of daemon threads after runtime shutdown"
am: 66521058c8

* commit '66521058c80ee4058995d28f512dd6d173610e3a':
  Improve handling of daemon threads after runtime shutdown

Merge "Revert "Revert "Reduce sleep duration in SuspendAllDaemonThreads"""

Revert "Revert "Reduce sleep duration in SuspendAllDaemonThreads""

Fixed the error where I reduced the time until the warning by 10x.

Bug: 26351700

This reverts commit bc34a7ed232cb3d384c250578e2b4ede45f2b167.

Change-Id: I6b30a0711fe077421665b78c39ded88899407700

Merge "Improve handling of daemon threads after runtime shutdown"

Improve handling of daemon threads after runtime shutdown

The main issue comes from the fact that user daemon threads are
allowed to continue running after the runtime has shutdown. They may
still have a JNI env pointer. To prevent crashing if they call into
the env, we replace the function pointers with functions that sleep
forever.

The other issue is that user daemon threads that are blocked in an
ART condition variable may get woken up by another user daemon inside
of Monitor::Notify or by a spurious wakeup (i.e. SIGQUIT). To deal
with this issue, we check the JNI env for shutdown runtime when we
are woken up from a condition variable wait. This check fixes test
132 with --host --gdb --interpreter. Previously this test crashed
since dlclose was somehow causing a spurious futex wakeup.

TODO: Investigate adding a unit test.

Bug: 18577101
Change-Id: I479b38968ee9fbc4ee4b252ee2528787279972cc

Merge "Fix general multi-image TODOs."
am: 37a5abcf5e

* commit '37a5abcf5e7644ae1fd1a85e865c8a71e38a9af2':
  Fix general multi-image TODOs.

Merge "Fix general multi-image TODOs."

Fix general multi-image TODOs.

- Removed some unnecessary comments.

Bug: 26317072

(cherry-picked from commit 509f2ab308032534f14d8f4f374f60da8a615fa2)

Change-Id: Id542262ca1182fb77f029f2136266e470bde9473

Re-disable tests broken for optimizing by multi-image.

Bug: 26317072
Change-Id: I8ce15cde51766073565f6e108c4e00accc7124da

Fix general multi-image TODOs.

- Removed some unnecessary comments.
- Reenabled tests that are working again.

Change-Id: Id542262ca1182fb77f029f2136266e470bde9473

Merge "Fix image writer for generation of app images"
am: 97f2ca08c3

* commit '97f2ca08c3d9a2b1694419aea07cd64f477c0af2':
  Fix image writer for generation of app images

Merge "Fix image writer for generation of app images"

Fix image writer for generation of app images

Fixed IsInBootImage and IsInBootOatFile. Also re-enabled generation
of app images for tests.

Bug: 26317072
Bug: 22858531
Change-Id: I19cbe7a8143b5ff7582eb27dfd9d2d8121c67b3c

Merge "Disable test while investigating."
am: e2e87002d9

* commit 'e2e87002d99b57f05cb4165bc7053e841807488e':
  Disable test while investigating.

Merge "Disable test while investigating."

Disable test while investigating.

Change-Id: Ide7fd1251e10fe93525fe3fccd9240318585f9e6

Merge "Add a missing implicit null check in the ARM codegen."
am: 363910e676

* commit '363910e676f388b87478e553c243157d395ffc3c':
  Add a missing implicit null check in the ARM codegen.

Merge "Add a missing implicit null check in the ARM codegen."

Merge "Small implicit null checks refactoring in the ARM codegen."
am: fbd9dfcf4f

* commit 'fbd9dfcf4ffc32f0bda831aa898c7ea5dbf8ed9a':
  Small implicit null checks refactoring in the ARM codegen.

Merge "Small implicit null checks refactoring in the ARM codegen."

Small implicit null checks refactoring in the ARM codegen.

Change-Id: I7dccb02cf7ac2f7d8fd1676b03e0b394701fbe3f

Add a missing implicit null check in the ARM codegen.

The code generated for object ArraySet on ARM used to
miss an implicit null check for the array when the assigned
value is `null`.  This has not been an actual issue so far,
as ArraySet instructions have never been using implicit null
checks.

Note: This CL comes without a regression test, as the code
path in question is not used (yet).

Change-Id: If3bc85e32802595e635513dfb83ccfcfd8f00d3d

Merge "ARM Baker\'s read barrier fast path implementation."
am: 74179b2082

* commit '74179b2082f8114db5154c75be643d9337cb3a63':
  ARM Baker's read barrier fast path implementation.

Merge "ARM Baker's read barrier fast path implementation."

ARM Baker's read barrier fast path implementation.

Introduce an ARM fast path implementation in Optimizing for
Baker's read barriers (for both heap reference loads and GC
root loads).  The marking phase of the read barrier is
performed by a slow path, invoking the runtime entry point
artReadBarrierMark.

Other read barrier algorithms continue to use the original
slow path based implementation, which has been renamed as
GenerateReadBarrierSlow/GenerateReadBarrierForRootSlow.

Bug: 12687968
Change-Id: Ie7ee85b1b4c0564148270cebdd3cbd4c3da51b3a

Merge "Add libstdc++ as a target for buildbot builds."
am: 5ee288c9dd

* commit '5ee288c9dd99614e3a238f5efceeec6456e3499d':
  Add libstdc++ as a target for buildbot builds.

Merge "Add libstdc++ as a target for buildbot builds."

Merge "Re-enable class table for image writer"
am: 2c22e8de76

* commit '2c22e8de763207cda10f035f4506e049e2aa2d2d':
  Re-enable class table for image writer

Merge "Re-enable class table for image writer"

Re-enable class table for image writer

Changes, create one temporary class table per ImageInfo. Don't keep
track of class loaders.

Bug: 26317072

Change-Id: I4f9121194f39031fb8629d60b6feeb2747a77870

Add libstdc++ as a target for buildbot builds.

It's a hidden dependency of libjavacore.

Change-Id: I7eca051502897c37293e22fbdda8a5924694ee76

Merge "Perform profile file analysis in dex2oat"
am: b7332f1c77

* commit 'b7332f1c774c675b5264a45890e72bf140664018':
  Perform profile file analysis in dex2oat

Merge "Perform profile file analysis in dex2oat"

Perform profile file analysis in dex2oat

Dex2oat can accept now multiple profile files to drive a profile based
compilation. --profile-file and --reference-profile-file speficy a pair
of profile files which will be evaluated for significant differences
before starting the compilation. If the difference is insignificant
(according to some internal metric) the compilation is skipped and a
message is logged.

Multiple pairs of --profile-file and --reference-profile-file can be
specified. This effectively enables multi user support since profiles
for different users will be kept separately.

--reference-profile-file can be left out, case in which the decision is
solely based on --profile-file. If both flags are present, then their
repetition should form unique pairs.

If the compilation is performed and --reference-profile-file is given
then its data is merged with the data from the corresponding --profile-
file and saved back to the file.

If no profile flags are given, dex2oat proceeds as before and compiles
the dex files unconditionally.

As part of this change
- merge ProfileCompilationInfo and OfflineProfilingInfo under the same
object. There was no use to keep them separate anymore.
- SaveProfilingInfo now merges the data with what was in
the file before instead of overwriting it.

Bug: 26080105

Change-Id: Ia8c8b55587d468bca5179f78941854285426234d

Merge "Force Jack to consider multidex annotations"
am: 7825791088

* commit '78257910887d58cfe9b7c90d93cedd7b1145271d':
  Force Jack to consider multidex annotations

Merge "Force Jack to consider multidex annotations"

Force Jack to consider multidex annotations

This is a workaround and should not be necessary.

Bug: 26441710
Change-Id: Ife879e46deb872ed48cff9abbe1088e2f1c86bee

Merge "The ArtMethod is a long on 64bits."
am: b0c1667a56

* commit 'b0c1667a56eeb594a50157db80a7e5078fd579db':
  The ArtMethod is a long on 64bits.

Merge "The ArtMethod is a long on 64bits."

The ArtMethod is a long on 64bits.

Change-Id: I8719d7061def3968d8f933e7649ec7bbdabbcf67

Merge "Fix a crash with unresolved classes."
am: ef8f7d0553

* commit 'ef8f7d0553de78e4b73d10d6a8b16e1b533836c5':
  Fix a crash with unresolved classes.

Merge "Fix a crash with unresolved classes."

Merge "Assume the profile file was created before saving."
am: 7ede3dd40e

* commit '7ede3dd40e2f7da3e3073cc355312021c31820b3':
  Assume the profile file was created before saving.

Merge "Assume the profile file was created before saving."

Fix a crash with unresolved classes.

We cannot pass the initialization duty to a HLoadClass that needs
access checks. The optimization was assuming the graph structure
could not lead to such a situation, but turns out with licm, it can.

Change-Id: I6c5299955e1f9221a2dd70976a55e0dea06882eb

Merge "Revert "Reduce sleep duration in SuspendAllDaemonThreads""
am: a90d8bc3a3

* commit 'a90d8bc3a3772877f4490d0509d53053069e5b55':
  Revert "Reduce sleep duration in SuspendAllDaemonThreads"

Merge "Revert "Reduce sleep duration in SuspendAllDaemonThreads""

Revert "Reduce sleep duration in SuspendAllDaemonThreads"

run-test 132 randomly chokes due to this change.

Bug: 26351700

This reverts commit adef5bde6c3392dca3dafa0f8d2c9428b59af3ef.

Change-Id: I18e007264e1e8b9027d7d241cb9f0053ad8d112a

Merge "Reduce sleep duration in SuspendAllDaemonThreads"
am: fcad0099ee

* commit 'fcad0099eeceb6f2c09a6de28d8e8f05dab69ed5':
  Reduce sleep duration in SuspendAllDaemonThreads

Merge "Reduce sleep duration in SuspendAllDaemonThreads"

Reduce sleep duration in SuspendAllDaemonThreads

Previously we unconditionally slept for 200ms which caused runtime
shutdown to take 200ms longer than required.

Reduces runtime shutdown time for "am" from ~200ms to ~10ms.

Bug: 26351700
Change-Id: I7fd0059cb3b04dba0acc4a1754e76c89ec867f85

Merge "Add systrace output to runtime shutdown"
am: b94a8906b8

* commit 'b94a8906b82ed05ccf67be68d9d329b85c514d46':
  Add systrace output to runtime shutdown

Merge "Add systrace output to runtime shutdown"

Add systrace output to runtime shutdown

Help debug why am command shutdown is slow.

Findings:
~200ms spent in deleting thread list on N5.

Bug: 26351700
Change-Id: Icb59390a5267ae8bade4ced12252743c48206bb1

Merge "Address some intern table comments"
am: bf157c3f50

* commit 'bf157c3f50ab4790bae5a763c677e174b80e7f9a':
  Address some intern table comments

Merge "Address some intern table comments"

Address some intern table comments

Change-Id: I7ffaa463272015c2924ba03e006041daee498ad4

Merge "Remove gaps between images"
am: 5af190cb6f

* commit '5af190cb6f11a651afe0f0abc5ba0844246970d6':
  Remove gaps between images

Merge "Remove gaps between images"

Remove gaps between images

We now use the same logic to calculate the size as we use to create
the image header. This should mean that it always lines up correctly.
Previously the alignment rounding could have caused page off by one
errors in some cases.

Re-enable no gap check in heap.cc.

Bug: 26317072
Change-Id: I558a4e60197ca4613fc4ec4c3a618c64a83d006a

Merge "Re-enable adding intern table to image"
am: 503a6fa15f

* commit '503a6fa15f22bd5fedf097f1ecd01fcbc43779f3':
  Re-enable adding intern table to image

Merge "Re-enable adding intern table to image"

Merge "Get rid of shadow maps for images"
am: ed1ae775ea

* commit 'ed1ae775ea8e411028e62b477069c60dab6dca03':
  Get rid of shadow maps for images

Merge "Get rid of shadow maps for images"

Get rid of shadow maps for images

Fixed the image bitmap creation to use the End of the object section
instead of using the map size. This means that the bitmap covers at
most one word of bits past the end of the object section.

Reverts shadow map logic introduced by multi image CL:
(commit dcdc85bbd569f0ee66c331b4219c19304a616214)

Bug: 26317072
Change-Id: I181cf3490a5b292c00c6b0f97536f3206adcd091

Re-enable adding intern table to image

Changed intern table to have a stack of tables similarily to
ClassTable. Adding an image intern table adds to the front of the
intern table stack. Also some cleanup.

Bug: 26317072

Change-Id: I7bbf9485b5dbbbf3707fed21e29de3beccfb8705

Merge "Disable tests while investigating."
am: 664d7cfba0

* commit '664d7cfba0e734ba8917c260e424ea559b7e8f9f':
  Disable tests while investigating.

Merge "Disable tests while investigating."

Disable tests while investigating.

Change-Id: I3f1e46a8bf27c962c96315d125ddd8430b7f9f0b

Merge "ART: Don\'t set initial RTI for BoundType if input untyped"
am: 5b7b5ddb51

* commit '5b7b5ddb515828c93f0c2aec67aa513c32d0de22':
  ART: Don't set initial RTI for BoundType if input untyped

Merge "ART: Resolve ambiguous ArraySets"
am: a6bfd5967b

* commit 'a6bfd5967b751715e15c4f22d8a6a03285bed9bc':
  ART: Resolve ambiguous ArraySets

Merge "ART: Don't set initial RTI for BoundType if input untyped"

Merge "ART: Resolve ambiguous ArraySets"

ART: Don't set initial RTI for BoundType if input untyped

ReferenceTypePropagation will create a BoundType with upper bound
[Object, inexact, not null] for each if-not-null branch. The logic
setting its initial RTI will, however, set it straight to Object if
the input is untyped (loop phi or its derivate). This patch changes
the logic to leave the BoundType untyped and set it during fix-point
iteration.

Bug: 26330326
Change-Id: Ic492e2179a4c51f577908e60fbcf70d728b98a6f

ART: Resolve ambiguous ArraySets

Just like aget(-wide), the value operand of aput(-wide) bytecode
instructions can be both int/long and float/double. This patch builds
on the previous mechanism for resolving type of ArrayGets to type the
values of ArraySets based on the reference type of the array.

Bug: 22538329

Change-Id: Ic86abbb58de146692de04476b555010b6fcdd8b6

Merge "ART: Create BoundType for CheckCast early"
am: 87bafaf563

* commit '87bafaf563d35616804dd0cc0dbde176aac87817':
  ART: Create BoundType for CheckCast early

Merge "ART: Create BoundType for CheckCast early"

ART: Create BoundType for CheckCast early

ReferenceTypePropagation creates a BoundType for each CheckCast and
replaces all dominated uses of the casted object with it. This does
not include Phi uses on the boundary of the dominated scope, reducing
typing precision. This patch creates the BoundType in Builder, causing
SsaBuilder to replace uses of the object automatically.

Bug: 26081304

Change-Id: I083979155cccb348071ff58cb9060a896ed7d2ac

Merge "Update expectations after push of apache-xml and other fixes."
am: c917d195d8

* commit 'c917d195d8d8d05f90796b1e0842883fc608346d':
  Update expectations after push of apache-xml and other fixes.

Merge "Update expectations after push of apache-xml and other fixes."

Update expectations after push of apache-xml and other fixes.

bug:26326992
Change-Id: I7d0c0706c53b4b04a3a3038b334cac6bcda4868c

Merge "Revert "Add new test expectations after enso move.""
am: 9c2bb85d5b

* commit '9c2bb85d5bbfd6a864e420c82f312def700436e5':
  Revert "Add new test expectations after enso move."

Merge "Revert "Add new test expectations after enso move.""

Revert "Add new test expectations after enso move."

Failures were due to a bogus date on devices.

Bug: 26353151
Bug: 26326992

This reverts commit 376756d0c9e23173c1d93fe32acf6b3e08f45b73.

Change-Id: I3ecdd152b27299efab37ded29b62c2f3369b5d96

Merge "Fix multi-image TODOs in class linker, runtime, and oat file."
am: a44396ee4b

* commit 'a44396ee4ba91fd146344337b031896a49928345':
  Fix multi-image TODOs in class linker, runtime, and oat file.

Merge "Fix multi-image TODOs in class linker, runtime, and oat file."

Merge "ART: Add large object allocation to ThreadStress"
am: bda4caa149

* commit 'bda4caa149da1a8fbe9c88c0963c34f3961be849':
  ART: Add large object allocation to ThreadStress

Merge "ART: Add large object allocation to ThreadStress"

Fix multi-image TODOs in class linker, runtime, and oat file.

- Modified SanityCheckArtMethodPointerArray in class linker
- Put back warnings in OatFile::GetOatDexFile
- Reinstated ImageSpace VerifyImageAllocations in Runtime::Init

Bug: 26317072

(cherry-picked from commit 0dfef949bb824accde27f8cfe1b233ec9e087355)

Change-Id: I3bdb8f87d885213795c82c41e5095fec6daf00c4

Merge "Fix multi-image TODOs in class linker, runtime, and oat file."