Merge "ART: add kryo to a53 #835769 & #843419 erratum exception list"
am: 9013bb031c

* commit '9013bb031c1c3244e70ae246437604f2c094a671':
  ART: add kryo to a53 #835769 & #843419 erratum exception list

Merge "ART: add kryo to a53 #835769 & #843419 erratum exception list"

Merge "ART: Refactor GenerateTestAndBranch"
am: 3ce45f4e6a

* commit '3ce45f4e6abd725407a539521f69959a6ac5a9c1':
  ART: Refactor GenerateTestAndBranch

Merge "ART: Refactor GenerateTestAndBranch"

Merge "Change roots view to "rooted"."
am: 886d1040c9

* commit '886d1040c99b521ca5a2d363242163c17a5776b1':
  Change roots view to "rooted".

Merge "Change roots view to "rooted"."

ART: Refactor GenerateTestAndBranch

Each code generator implements a method for generating condition
evaluation and branching to arbitrary labels. This patch refactors
it for better clarity but also to generate fewer jumps when the true
branch is the fallthrough successor.

This is preliminary work for implementing HSelect.

Change-Id: Iaa545a5ecbacb761c5aa241fa69140cf6eb5952f

Merge "Add ability to skip run-tests with ART_TEST_RUN_TEST_SKIP."
am: 991842a527

* commit '991842a5273d20695dd5f35eb9a9e28c386c5b4b':
  Add ability to skip run-tests with ART_TEST_RUN_TEST_SKIP.

Merge "Add ability to skip run-tests with ART_TEST_RUN_TEST_SKIP."

Merge "Ignore NullConstants when computing the type for Phis."
am: a663d67500

* commit 'a663d67500be03df7708afc059f135d2be76ef6e':
  Ignore NullConstants when computing the type for Phis.

Merge "Ignore NullConstants when computing the type for Phis."

Merge "Implemented control of "export only" feature in dexdump."
am: a445f7afbe

* commit 'a445f7afbe113186de97007891307065109a6b84':
  Implemented control of "export only" feature in dexdump.

Merge "Implemented control of "export only" feature in dexdump."

Merge "Mark kQuickReadBarrierForRootSlow as direct entry point on MIPS."
am: 72cc929186

* commit '72cc92918665acb262f69e56be984e24bd15a223':
  Mark kQuickReadBarrierForRootSlow as direct entry point on MIPS.

Merge "Mark kQuickReadBarrierForRootSlow as direct entry point on MIPS."

Mark kQuickReadBarrierForRootSlow as direct entry point on MIPS.

Change-Id: I35caa32c9773646692706642188ab522237a68ac

Merge "x86/x86-64 read barrier support for concurrent GC in Optimizing."
am: 3eb242c0c0

* commit '3eb242c0c0d7e8327d2b1f515b463337e37c64ba':
  x86/x86-64 read barrier support for concurrent GC in Optimizing.

Merge "x86/x86-64 read barrier support for concurrent GC in Optimizing."

Merge "Move the trace file out of the profiles folder."
am: 3610abea2f

* commit '3610abea2fe09f519e20fa4212f116d9167b5d51':
  Move the trace file out of the profiles folder.

Merge "Move the trace file out of the profiles folder."

x86/x86-64 read barrier support for concurrent GC in Optimizing.

This first implementation uses slow paths to instrument heap
reference loads and GC root loads for the concurrent copying
collector, respectively calling the artReadBarrierSlow and
artReadBarrierForRootSlow (new) runtime entry points.

Notes:
- This implementation does not instrument HInvokeVirtual
  nor HInvokeInterface instructions (for class reference
  loads), as the corresponding read barriers are not stricly
  required with the current concurrent copying collector.
- Intrinsics which may eventually call (on slow path) are
  disabled when read barriers are enabled, as the current
  slow path infrastructure does not support this case.
- When read barriers are enabled, the code generated for a
  HArraySet instruction always go into the array set slow
  path for object arrays (delegating the operation to the
  runtime), as we are lacking a mechanism to keep a
  temporary register live accross a runtime call (needed for
  the instrumentation of type checking code, which requires
  two successive read barriers).

Bug: 12687968
Change-Id: I14cd6107233c326389120336f93955b28ffbb329

Merge "Fix class unloading with the CC collector."
am: dd4cbcc924

* commit 'dd4cbcc924c8ba2a578914a4a366996693bdcd74':
  Fix class unloading with the CC collector.

Merge "Fix class unloading with the CC collector."

Add ability to skip run-tests with ART_TEST_RUN_TEST_SKIP.

One can skip running specific run-tests under any configuration by
setting the ART_TEST_RUN_TEST_SKIP variable to a list of the run-test
names to skip.

    $ # None of the variants of 022-interfaces will be run.
    $ ART_TEST_RUN_TEST_SKIP=022-interface mm test-art-host-run-test

Change-Id: I33f24dd7f1eb9f4d6dc407a4abc3c5919437d4fa

Fix class unloading with the CC collector.

Avoid unnecessarily decoding dex cache and class loader weak roots,
which would trigger read barriers.

Re-enable 141-class-unload with the CC collector.

Bug: 12687968
Bug: 24468364
Change-Id: Ib4c19f25000873cab0e06047040442d135285745

Merge "Disable test 964 with gcstress since it often times out."
am: 0e06a0989f

* commit '0e06a0989f4fc53e0d281a5a2cb5b5a17feaea0c':
  Disable test 964 with gcstress since it often times out.

Merge "Disable test 964 with gcstress since it often times out."

Merge "Revert "Revert "Add missing null check to String::ToCharArray"""
am: f6ebcd099a

* commit 'f6ebcd099aeaa694fada7ce4a77b1698488d8121':
  Revert "Revert "Add missing null check to String::ToCharArray""

Disable test 964 with gcstress since it often times out.

Change-Id: Ice9b25d373b849d4415f9189fe94d8ccd6d05cbe

ART: add kryo to a53 #835769 & #843419 erratum exception list

Kryo is custom-designed 64-bit ARM CPU and does not need this A53
erratum handling.

https://www.qualcomm.com/news/snapdragon/2015/09/02/snapdragon-820-and-kryo-cpu-heterogeneous-computing-and-role-custom

Change-Id: Id8f6c616ab2ed19056fd1e3829b85b94a6e6eb84

Merge "Revert "Revert "Add missing null check to String::ToCharArray"""

Implemented control of "export only" feature in dexdump.

Rationale:
Export only is now under explicit control of a new -e flag, and
can be used to control both -l xml and -l plain. The result is
that, by default, xml output now shows all items (old behavior
can be forced with -e -l xml). This solution feels a bit cleaner
than adding a flag to override export only when -l xml is given
(old dexdump forced this tight relation).

Bug: 25673756 (feature request).

Change-Id: I7cfd48bf91195d2a81d134936719d267fda399f5

Revert "Revert "Add missing null check to String::ToCharArray""

Fix flaky test.

Bug: 25641543

This reverts commit b52a39c76e0cb0638c92cdcf6b5c5aa551420e1a.

Change-Id: I6b0d3087dba3680252e65af554530e2769cde55c

Merge "Stream DWARF debug section directly to disk."
am: 2f49b0a907

* commit '2f49b0a90720fe9f4cab040c376aab5f387967e3':
  Stream DWARF debug section directly to disk.

Merge "Stream DWARF debug section directly to disk."

Merge "Revert "Add missing null check to String::ToCharArray""
am: 6aa0735c7b

* commit '6aa0735c7b84061901a844270d22249cbb01004e':
  Revert "Add missing null check to String::ToCharArray"

Merge "Revert "Add missing null check to String::ToCharArray""

Revert "Add missing null check to String::ToCharArray"

Does not work with the interpreter.

Bug: 25641543

This reverts commit 952d608062eec2d7f47f9b45dba935ad8b4d23e5.

Change-Id: Ic112fa69c7ddd119cbccc5b65007b5ee4dfccd09

Merge "Add missing null check to String::ToCharArray"
am: dc9706dcea

* commit 'dc9706dcea7c1384d2b2137485f9579db36ca328':
  Add missing null check to String::ToCharArray

Merge "Add missing null check to String::ToCharArray"

Add missing null check to String::ToCharArray

Added test.

Bug: 25641543
Change-Id: Ic9a21ce8bc530dbedf14334ad47f5faa90ae4ddc

Merge "Fix bug with test 964."
am: 1227d627e0

* commit '1227d627e0e0ef8b34a66f94714ed737953098ce':
  Fix bug with test 964.

Merge "Fix bug with test 964."

Fix bug with test 964.

An indentation error meant that some test cases would not be
generated. Also fixed run-test script since fixing this causes the
output to be larger than the default ulimit.

Change-Id: I454f89078f8307f400b66627fdab13feaf492512

Merge "Add class loader argument to openDexFileNative"
am: efca362e8c

* commit 'efca362e8c67d5b330dd4ebc312cd45cf2585964':
  Add class loader argument to openDexFileNative

Merge "Add class loader argument to openDexFileNative"

Add class loader argument to openDexFileNative

Bug: 22858531
Change-Id: I315919d91822db0c73cf16b21d660d5870d5746f

Merge "Optimizing/X86: PC-relative dex cache array addressing."
am: cff81076cb

* commit 'cff81076cbb4bbe3841942f14326f4401fa3c8df':
  Optimizing/X86: PC-relative dex cache array addressing.

Merge "Optimizing/X86: PC-relative dex cache array addressing."

Merge "Optimizing: Clean up constant location handling."
am: fe0ec35c68

* commit 'fe0ec35c68d57205bd8fe13bd195ae0b5a3ed180':
  Optimizing: Clean up constant location handling.

Merge "Optimizing: Clean up constant location handling."

Merge "Revert "Mark __jit_debug_register_code as a unique symbol""
am: 7a2f5c9459

* commit '7a2f5c9459b492bd8550e437087191e52ca6324f':
  Revert "Mark __jit_debug_register_code as a unique symbol"

Merge "Revert "Mark __jit_debug_register_code as a unique symbol""

Revert "Mark __jit_debug_register_code as a unique symbol"

This reverts commit e7132a9a5d98103fa52a750287476b063fa88863.

We are still using ld on some targets instead of gold when compiling for ararch64. Revert until we figure out why.

Change-Id: If09fac8d72485dc8adbc81e44a2af84ed91dcd91

Merge "JDWP: default implementation of Method.IsObsolete"
am: 190c5f9f3b

* commit '190c5f9f3b8aba5c8c7a26d4cfe0f45297bd875c':
  JDWP: default implementation of Method.IsObsolete

Merge "JDWP: default implementation of Method.IsObsolete"

Merge "MIPS64: fix calling conventions in LoadClass and field accesses"
am: e542fd0c04

* commit 'e542fd0c04426c45ae6a04bc4ae79ac77b99d8c6':
  MIPS64: fix calling conventions in LoadClass and field accesses

Optimizing/X86: PC-relative dex cache array addressing.

Add PC-relative dex cache array addressing for X86 and use
it for better invoke-static/-direct dispatch. Also delay
the initialization to the PC-relative base until needed.

Change-Id: Ib8634d5edce4920cd70172fd13211809cf6948d1

Merge "MIPS64: fix calling conventions in LoadClass and field accesses"

Merge "Mark __jit_debug_register_code as a unique symbol"
am: 772e3ab102

* commit '772e3ab10292ad3ce4ca5e217c552ded4e26651a':
  Mark __jit_debug_register_code as a unique symbol

Merge "Mark __jit_debug_register_code as a unique symbol"

Merge "ART: Add comments to Scoped-/ArenaAllocator."
am: 9b71e9c702

* commit '9b71e9c702df1b65ea658cd5fc856e5a6d80669c':
  ART: Add comments to Scoped-/ArenaAllocator.

Merge "ART: Add comments to Scoped-/ArenaAllocator."

Merge "Image space cleanup for app images"
am: 11a4c96a7f

* commit '11a4c96a7f8268a94cbdcbc2771d03bc4f2ce065':
  Image space cleanup for app images

Merge "Image space cleanup for app images"

Image space cleanup for app images

Removed Heap::GetImageSpace, added Heap::GetBootImageSpace.

Generalized some logic in the class linker for image spaces.

Bug: 22858531

Change-Id: Ib3e12bb061a247e232d3dc93b0d6b35eb3a34d25

Merge "Get class loader to remember interrupts"
am: 2abe1f4a8d

* commit '2abe1f4a8d6b56ed95da0671091e4fe15f651bd6':
  Get class loader to remember interrupts

Merge "Get class loader to remember interrupts"

Get class loader to remember interrupts

Bug: 25606036

Monitor::Wait should not reset the interrupt flag unless it actually
throws the exception.

Change-Id: Id6a35c888160043831f30055f9bd39e7f0440439

MIPS64: fix calling conventions in LoadClass and field accesses

This fixes failures in test 529-checker-unresolved

Change-Id: I6170c22059e9711b2fcc965b00d6e34edd839539

Merge "Make test 968\'s name shorter"
am: 5165b6ba1f

* commit '5165b6ba1fda85d769c5092a65f07bcfffb0b3e7':
  Make test 968's name shorter

Merge "Make test 968's name shorter"

Make test 968's name shorter

It was causing intermittent CI breaks due to the long name of the
generated oat file.

Change-Id: I71b4b3aa1a13df2696b276998cafe40006d54b66

Merge "Revert "lambda: unit test for lambda expressions at Java Language source level.""
am: 63e310c5c3

* commit '63e310c5c3cd2341c1b2a60d25764c44ee066318':
  Revert "lambda: unit test for lambda expressions at Java Language source level."

Merge "Revert "lambda: unit test for lambda expressions at Java Language source level.""

Revert "lambda: unit test for lambda expressions at Java Language source level."

REASON: Buildbots are having jack issues

This reverts commit 152ee552652691ecaec2db420d468cb178647952.

Change-Id: I650fe1a94f1b97c14a689599703498506dd70b23

Merge "Move .debug_line before .debug_info."
am: c4b2410947

* commit 'c4b24109473d50e4cc829be5d1e7c77fd21527f3':
  Move .debug_line before .debug_info.

Merge "Move .debug_line before .debug_info."

Stream DWARF debug section directly to disk.

Change the structure so that the debug sections are written to
disk as we go.  There are still some temporary buffers, however,
we no longer hold all of the data in memory before writing it.
We can not avoid buffering of some things (e.g. .debug_str).

Change-Id: Id4940cf10ae5b6f3cac7fb8d20197f0304079b92

Move .debug_line before .debug_info.

Swap the order since .debug_info depends on .debug_line, but
not vice versa.  This is for disk streaming convenience.

Change-Id: Iabe18fea97aa5527aedfc55c9bdf66e5587568bc

Merge "ART: Fix arena allocation for valgrind."
am: 00bcbcfdc3

* commit '00bcbcfdc3ba28b4b985d8a8029d677579e902ff':
  ART: Fix arena allocation for valgrind.

ART: Add comments to Scoped-/ArenaAllocator.

Change-Id: Iba7350114fb998ac9ee1466ef12f4a5024a18e8e

Merge "ART: Fix arena allocation for valgrind."

Merge "lambda: unit test for lambda expressions at Java Language source level."
am: a828aadd6d

* commit 'a828aadd6dd9a5e73daf9a167b71c88bc2bdfbd7':
  lambda: unit test for lambda expressions at Java Language source level.

Merge "lambda: unit test for lambda expressions at Java Language source level."

lambda: unit test for lambda expressions at Java Language source level.

Rationale: having a checkers-flavored unit test will enable testing
           both if the correct HIR is generated using the checker
           as well as the correctness of execution (interpreter for now,
           but native code later).

Change-Id: I588e9955efaaf53e5c5e3398399e009041cd65f0

Move the trace file out of the profiles folder.

/dalvik-cache/profiles/ is going away. Create the trace file under
/data/misc/trace.

This enable tracing without forcing SElinux into permissive mode.

Bug: 24698874
Bug: 25612377
Change-Id: I78d09ef629bebe635c3baabaf88f01996d3333e5

Merge "Correct exception behavior for default methods"
am: d15cd44c90

* commit 'd15cd44c905e055d2d60b847ba4b221edda5e119':
  Correct exception behavior for default methods

Merge "Correct exception behavior for default methods"

ART: Fix arena allocation for valgrind.

Move the zero-initialization check after marking the newly
allocated chunk as defined and check only the allocated
space without the red zone. Also mark unallocated space as
inaccessible instead of just undefined.

Change-Id: I74fc65f5b53acb74cec4e5a0146f41dacf4a1470

Change roots view to "rooted".

Instead of just showing roots in the roots view, show all objects
whose immediate dominator is the SENTINEL_ROOT.

Bug: 24613815
Change-Id: I96429d75395edfe109222e88d31cdc0bd87a7767

Ignore NullConstants when computing the type for Phis.

NullConstants are always typed as Object and they widen the
type of Phis unnecessarily.

Change-Id: I2b29e67617a22120642cb23d2bc6c1ff75b4e7c2

Optimizing: Clean up constant location handling.

Locations builder should use ConstantLocation() when the
code generator relies on a location to be constant. Code
generator should interrogate locations, not inputs, about
being const.

Change-Id: Ic35bb84aa9f83e0977b151a0430aca6c88f19cf0

Mark __jit_debug_register_code as a unique symbol

This symbol is used to notify the debugger about JIT library load/unload
events. Because of ICF, several functions with the same body put to the
same adress by the linker what causes the debugger to get notifictaion
about JIT load/unload events in several case where it isn't really happened.

This change disable ICF for the __jit_debug_register_code symbol to prevent
this behavior with keeping the symbol unique in the object file.

This is the recommit of 5caf652d7bfb43bdd2f3addfe8021a5fb8e3f796 after
fixing the compilation issue on sdk_phone_armv7-sdk (mac) target.

Change-Id: I027d2c7964098d47ed350ee6b7c74d82c88a5d1f

Merge "Revert "Revert "Run type propagation after inliner only when needed."""
am: 54af9bd45a

* commit '54af9bd45a482376b5fe8d4ac291afb5bc950065':
  Revert "Revert "Run type propagation after inliner only when needed.""

Merge "Revert "Revert "Run type propagation after inliner only when needed."""

Revert "Revert "Run type propagation after inliner only when needed.""

This reverts commit 271743601650308c7ac5c7a3ec35025d8130a298.

Change-Id: I173e27a0a4d7d54f90ca459eb48d280d1d40ab70

Merge "Ignore failures of WeakHashMapTest#test_keySet in libcore tests."
am: c85f88b4c1

* commit 'c85f88b4c145015401d6849c83d9061bd998861f':
  Ignore failures of WeakHashMapTest#test_keySet in libcore tests.

Merge "Ignore failures of WeakHashMapTest#test_keySet in libcore tests."

Ignore failures of WeakHashMapTest#test_keySet in libcore tests.

The outcome of this test depends on a weak reference being
collected before a timeout, which sometimes fails.  This
test used to pass, but recent changes have made this
non-determinism more frequent.  Until we rewrite this
properly (if ever), just ignore these failures.

Bug: 25437292
Change-Id: I1f041f6dfc1318026bce966575477486207f5351