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

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."

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

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

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

Merge "Optimizing/ARM: Improve shifts of long values by a constant."
am: ea30daef25

* commit 'ea30daef253f4b616ec7c9d4c7e134f33454d19d':
  Optimizing/ARM: Improve shifts of long values by a constant.

Merge "Optimizing/ARM: Improve shifts of long values by a constant."

Optimizing/ARM: Improve shifts of long values by a constant.

Change-Id: Id66ef8cdb9e64306f2be547370b90cc100a3e086

Correct exception behavior for default methods

Default methods are defined to throw an IncompatibleClassChangeError
(ICCE) when they are called and there is no "best" implementation.
Previously we would simply throw an ICCE during class loading as soon
as we noticed that this would happen if called. This makes us wait
until we actually attempt to execute the method. Furthermore, this
allows us to use other, non-conflicting, methods on the object as
normal.

Furthermore, this makes us correctly throw AbstractMethodErrors in
cases where all default implementations of a method are overridden by
abstract declarations.

Adds 3 tests for this new behavior.

Bug: 24618811

Change-Id: Id891958a81f9b3862b2ce5919636aabef7d3422e

Merge "Don\'t use fixed addresses in mem map test"
am: d7c76bd7e8

* commit 'd7c76bd7e867b151ac81ef41810614c7b4cc898e':
  Don't use fixed addresses in mem map test

Merge "Don't use fixed addresses in mem map test"

Merge "ART: Fix leak in Heap"
am: 5b1157bf01

* commit '5b1157bf01fa1f9ca3dca57fc874d857d4083aca':
  ART: Fix leak in Heap

Don't use fixed addresses in mem map test

We obtain a valid address instead of assuming there won't be a map
there.

These checks used to be OK when we didn't start the runtime.

Change-Id: Ie636ed3773d2c0c62d87e57623ee4ce380c64db0

Merge "ART: Fix leak in Heap"

ART: Fix leak in Heap

Delete a mutex in the destructor.

Change-Id: I7fb8f3f9e7a30baf352f384f38520eb0dd4906eb

Merge "Optimizing: Clean up Equal/NotEqual constant folding for nulls."
am: 972c5a6205

* commit '972c5a62051ccd4558bc8956262183fe3ec466a4':
  Optimizing: Clean up Equal/NotEqual constant folding for nulls.

Merge "Optimizing: Clean up Equal/NotEqual constant folding for nulls."

Merge "Rosalloc fast path in assembly for x86_64."
am: 688567c06f

* commit '688567c06f9da553eb3c766a13a4cd32ca31ddce':
  Rosalloc fast path in assembly for x86_64.

Merge "Rosalloc fast path in assembly for x86_64."

Optimizing: Clean up Equal/NotEqual constant folding for nulls.

Change-Id: I17766395092ec61df61ef0b9ae4c37fd38164a3b

Merge "ART: Remove unnecessary helper method from GraphChecker"
am: 5c7c8248e4

* commit '5c7c8248e48857f3a7c98a02d93c6971834d7241':
  ART: Remove unnecessary helper method from GraphChecker

Merge "ART: Remove unnecessary helper method from GraphChecker"

ART: Remove unnecessary helper method from GraphChecker

Change-Id: I59f50a559da560033f4c9407d83b68fc5d64bd3c

Merge "Optimizing: Improve constant folding + DCE for inlining."
am: feaf26db63

* commit 'feaf26db63a62bce5a80550f500efa67ff64424f':
  Optimizing: Improve constant folding + DCE for inlining.

Merge "ART: Refactor iteration over normal/exceptional successors"
am: 9696702985

* commit '969670298516b9316b36270fcc4c21bfcd499595':
  ART: Refactor iteration over normal/exceptional successors

Merge "Optimizing: Improve constant folding + DCE for inlining."

Merge "ART: Refactor iteration over normal/exceptional successors"

ART: Refactor iteration over normal/exceptional successors

Add helper methods on HBasicBlock which return ArrayRef with the
suitable sub-array of the `successors_` list.

Change-Id: I66c83bb56f2984d7550bf77c48110af4087515a8

Optimizing: Improve constant folding + DCE for inlining.

Run constant folding before DCE in inliner to eliminate more
code that can prevent inlining. Improve the constant folding
to evaluate Equals and NotEquals for null inputs.

Change-Id: I876ffb903ef39484370b6c8793f0f8467a977362

Merge "ART: Improve comment about inlining into try/catch"
am: 8a6463a705

* commit '8a6463a7052ec69e7c0b94a65b26807f570e6359':
  ART: Improve comment about inlining into try/catch

Merge "ART: Improve comment about inlining into try/catch"

ART: Improve comment about inlining into try/catch

Change-Id: I66a4fd3206847c8d5bb57b1678d9d3dc94331294

Merge "ART: Fix critical edge checks in GraphChecker"
am: ee57104951

* commit 'ee57104951acf3dc5f6343f1d3022e3cd73016f5':
  ART: Fix critical edge checks in GraphChecker

Merge "Revert "Revert "ART: Update DCE to work with try/catch"""
am: 83dcccb96f

* commit '83dcccb96ff8424fb4ab46bcda7c422d59a8c135':
  Revert "Revert "ART: Update DCE to work with try/catch""

Merge "ART: Fix critical edge checks in GraphChecker"

Merge "Revert "Revert "ART: Update DCE to work with try/catch"""

ART: Fix critical edge checks in GraphChecker

Previous CL I5a13b8bb74509b48f5d628906f7158af007f99ae fixed logic for
splitting critical edges. The same logic is used in GraphChecker when
testing critical edges weren't introduced by a pass. This patch updates
it too.

Bug: 25493695
Bug: 25454012
Change-Id: I56226c82324ee55ae2fbe7262608dd2868d930f1

Merge "ART: Add clarifying comment"
am: 1963759e84

* commit '1963759e84664b51b617b00e15728f439895033b':
  ART: Add clarifying comment

Merge "ART: Add clarifying comment"

ART: Add clarifying comment

Change-Id: I189ec4cae0aa1a5245a79e86d1ec0592e38eac4a

Revert "Revert "ART: Update DCE to work with try/catch""

The previous CL failed because it did not update inputs of catch phis.
Since phi input indices cannot be easily mapped back to throwing
instructions, this new implementation at least removes catch phi uses
of values defined in the removed blocks to preserve graph consistency.

This reverts commit fb552d7061746f7a90fdd5002696e255e2e15c35.

Change-Id: I63d95915d1ef50e71d3bcf0cd10aaded554035b4

Merge "Delete flaky DCHECK"
am: 1b87033ed8

* commit '1b87033ed83ba03983b49237da89ce36d319b68e':
  Delete flaky DCHECK

Merge "Delete flaky DCHECK"

Delete flaky DCHECK

Fixes mips build

Change-Id: I738b41f09f3c926f21640e358cf0bd5557167b0e

Merge "Fix mac build"
am: a5c3e44663

* commit 'a5c3e44663c3b9c97fe350b4191cec079738fca3':
  Fix mac build

Merge "Fix mac build"

Fix mac build

Change-Id: I22bf169f15483e54b97d77ff0887cd1fde428af9

Merge "Add low_4gb support to MapFile and MapFileAtAddress"
am: ff12923229

* commit 'ff12923229e713c27b9a9588a3fdf8d59475d756':
  Add low_4gb support to MapFile and MapFileAtAddress

Rosalloc fast path in assembly for x86_64.

Measurements (host, ms)
    BinaryTrees:   324 ->  360 (+11%)
    BinaryTrees with 64 MB alloc stack + 1 GB heap:
                   299 ->  275  (-8%)
    MemAllocTest:  414 ->  368 (-11%)

Interestingly, BinaryTrees gets slower with default settings due to more
blocking (gc-for-alloc) collections. It seems because allocations are
faster, the allocation stack size and the heap size become the
bottleneck (note both an allocation stack overflow as well as heap
exhaustion cause gc-for-alloc collections). With a larger allocation
stack and heap where no blocking collections are observed, BinaryTrees
gets faster.

Bug: 9986565
Change-Id: I642b9fecd0a583cc133998c2f3932de815c4a757

Merge "Add low_4gb support to MapFile and MapFileAtAddress"

Merge "Some cleanup for tests 960-965 build scripts."
am: 02c330f6af

* commit '02c330f6afdb2a528fc5d6c54a5fd683ddaa219f':
  Some cleanup for tests 960-965 build scripts.

Merge "Some cleanup for tests 960-965 build scripts."

Add low_4gb support to MapFile and MapFileAtAddress

Motivation is to use this for loading app images in low 4GB at a
non fixed address.

Added test.

Bug: 22858531
Change-Id: I0f79a4a7bfbfbdfc112e41b25c8682b1fb932ab7

Some cleanup for tests 960-965 build scripts.

Make all 96* build scripts consistent and update 965 to build and run
with jack.

Change-Id: If27ee9c92880c75307a18b054ba926670877073a

Merge "ART: Rerun the verifier for compile-time failures"
am: 78232f2482

* commit '78232f2482e71fef255796633da68bb769baa93a':
  ART: Rerun the verifier for compile-time failures

Merge "ART: Rerun the verifier for compile-time failures"

Merge "Class Linker: Implement default interface verification ordering."
am: f1584a6f52

* commit 'f1584a6f5251892509097c3bb30bd91d36634d0a':
  Class Linker: Implement default interface verification ordering.

Merge "Class Linker: Implement default interface verification ordering."

Class Linker: Implement default interface verification ordering.

We ensure that default interface verification failure is propagated to
the implementing class so that the compiler is able to properly handle
verification failures. Furthermore we ensure that default interfaces
are verified at the same time their implementing classes are.

This does not add additional support for default methods to the verifier.

This adds a basic test of this functionality.

Previously assumptions made about the order of verification in our
code could cause a check failure during compilation. Our old code
assumed that all code that would be run during a classes
initialization had been verified before any of it was run. The default
interfaces broke this assumption. Now we will simply verify default
interfaces with any classes that implement them.

Bug: 24618811

Change-Id: Ia11a50a28889dbd7ca22ffa730ef443e39f3cb5b

ART: Rerun the verifier for compile-time failures

To aid app failure diagnosis, by default re-run the verifier at
runtime to compute a better VerifyError message.

Rewrite the verifier driver code to pass the last actual low-level
verifier message.

Bug: 25432718
Change-Id: Ib8e6dd1ce8121045c0d38f54969100094c3dde6e

Merge "Add more image writer comments and clean up"
am: 3c58937626

* commit '3c5893762675d90697d9205b76dc24e2d04e27ca':
  Add more image writer comments and clean up

Merge "Add more image writer comments and clean up"