Merge "Revert "Mark __jit_debug_register_code as a unique symbol""
am: 5729f5b897

* commit '5729f5b897ffba17dacf5bff105f4ae2d101209b':
  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 5caf652d7bfb43bdd2f3addfe8021a5fb8e3f796.

Something's going wrong on the Mac host side, even though this should be target only.

Change-Id: I165f23ee25cb1c4b0732e1d1afcae75271abcb09

Merge "ART: Fix simplification of catch blocks in the presence of dead code"
am: b203aad7a0

* commit 'b203aad7a0db904efa8429d48b53e56583f61ec2':
  ART: Fix simplification of catch blocks in the presence of dead code

Merge "ART: Fix simplification of catch blocks in the presence of dead code"

ART: Fix simplification of catch blocks in the presence of dead code

Simplification of catch blocks transforms the code so that catch
blocks have only exceptional predecessors. However, it is invoked
before trivially dead code is eliminated which breaks simple
assumptions such as the fact that a catch block cannot start with
move-exception if it has non-exceptional predecessors. This patch
fixes the algorithm to work under these relaxed conditions.

Bug: 25494450
Bug: 25492628
Change-Id: Idc8d010102a4b8b9a6cd918b98d6e11d1838db0c

Merge "Mark __jit_debug_register_code as a unique symbol"
am: 52960cd1ae

* commit '52960cd1ae1992522c8bae82fa872be49417187e':
  Mark __jit_debug_register_code as a unique symbol

Merge "Mark __jit_debug_register_code as a unique symbol"

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.

Change-Id: I392be25d8a07f47f8217bf693c8e5919d29cac7b

Merge "ART: Change behavior for rethrowing init failures (2)"
am: ef7ef4ce04

* commit 'ef7ef4ce04a46405e15cd91115c6e657235adf83':
  ART: Change behavior for rethrowing init failures (2)

Merge "ART: Change behavior for rethrowing init failures (2)"

Merge "ART: Fix potential integer overflow in JNI."
am: f66f05ddfc

* commit 'f66f05ddfceaa8a2f613d3ed8316a21c95e1fbca':
  ART: Fix potential integer overflow in JNI.

Merge "ART: Fix potential integer overflow in JNI."

ART: Fix potential integer overflow in JNI.

Change-Id: I06fe2035f911cfc8537e27961c2dc2c7e4d1e20d

Merge "MIPS32: java.lang.String.equals"
am: 17656eb32a

* commit '17656eb32a08f81384d4afda8b19b71f0736abd4':
  MIPS32: java.lang.String.equals

Merge "MIPS32: java.lang.String.equals"

MIPS32: java.lang.String.equals

Add intrinsic support for String.equals on MIPS32.

Change-Id: I2d184aa4d5dae7cdd4a89c2c902535692c9e7393

Merge "Improve X86 handling of constants to Field/Array Set"
am: 97e740166a

* commit '97e740166a0687ddca6a51deee64b61c1bd17868':
  Improve X86 handling of constants to Field/Array Set

Merge "Improve X86 handling of constants to Field/Array Set"

Improve X86 handling of constants to Field/Array Set

AnTuTu 5.6 f.v.b() function on X86 showed several places where FP
constants were created using the stack, one in an inner loop.

Change ArraySet and *FieldSet to allow constants, and then generate
integer constant moves, rather than forcing the constant to be
materialized into a register. This will also help with register
pressure.

Change-Id: Ibebe2fed748a1bbeca47c36e9536b9b2357fc3cb
Signed-off-by: Mark Mendell <mark.p.mendell@intel.com>

Merge "JDWP: fix returned tag for ClassType.NewInstance"
am: 5348137d15

* commit '5348137d15483f7aeaf03f5116b8d6e01b5c1bac':
  JDWP: fix returned tag for ClassType.NewInstance

Merge "JDWP: fix returned tag for ClassType.NewInstance"

Merge "ART: Fix ignoring duplicate fields when loading class."
am: 2fa684206b

* commit '2fa684206b0a8bd203d07e1edf74a839b498f74f':
  ART: Fix ignoring duplicate fields when loading class.

Merge "ART: Fix ignoring duplicate fields when loading class."

Merge "MIPS64: Implement virtual intrinsic slow path"
am: 2704896d3a

* commit '2704896d3aa2902f46eebd722095b06331f548be':
  MIPS64: Implement virtual intrinsic slow path

Merge "MIPS64: Implement virtual intrinsic slow path"

Merge "Change test .dex.art to .art"
am: 9c3cd19d85

* commit '9c3cd19d85a1366fdd9a617562d4966d6844d9f7':
  Change test .dex.art to .art

Merge "Change test .dex.art to .art"

Change test .dex.art to .art

Fixes some file names that were too long and causing tests to fail.

Change-Id: Id4929b2ba98e7c0bc7daa33ec3e783738f5e4719

MIPS64: Implement virtual intrinsic slow path

This fixes a crash in dex2oat while compiling boot.oat.

Change-Id: I44fc92809902d7fc226c88b3e3f081b72cc19ce5

Merge "Revert "Revert "Add basic image writer support for app images"""
am: ae04419075

* commit 'ae0441907596ff5d7d3ebba6e12ed25e3ec11e21':
  Revert "Revert "Add basic image writer support for app images""

Merge "Revert "Revert "Add basic image writer support for app images"""

Revert "Revert "Add basic image writer support for app images""

No changes, bug fixed in:
https://android-review.googlesource.com/#/c/180886/

Bug: 22858531

This reverts commit 4b018565e57c3349a3c1b5ec8ac9dae261c5e00b.

Change-Id: I86d9c2b55d535d803c6e1b3b8b4836bf6ff077e5

Merge "Fix elf writer to properly handle empty text sections"
am: 51c1b63fc7

* commit '51c1b63fc71bea1b22b9719bc24ffd130aa2e420':
  Fix elf writer to properly handle empty text sections

Merge "Fix elf writer to properly handle empty text sections"

Fix elf writer to properly handle empty text sections

Previously, empty text sections caused an underflow in the oat file
size.

Added regression test.

Bug: 22858531
Change-Id: I9391a3666ab04e32472a9776d5217fe1743417a9

Merge "ahat: limit default number of results shown."
am: 1a143287f2

* commit '1a143287f23d8ed7f69ccc336733f605f44cf5da':
  ahat: limit default number of results shown.

Merge "ahat: limit default number of results shown."

Merge "Minor cleanup of Monitor::VisitLocks"
am: 4ba3766449

* commit '4ba3766449e4c36478ed896ee6d2ad9e9ee43f74':
  Minor cleanup of Monitor::VisitLocks

Merge "Minor cleanup of Monitor::VisitLocks"

ART: Fix ignoring duplicate fields when loading class.

Bug: 25493974
Change-Id: I59e7ee6ee5b991b327553d022d7d212db415c9c3

Minor cleanup of Monitor::VisitLocks

Uses the Instruction class to not bother with the decoding logic.

Change-Id: Id1501b57129084014f05e41513319639a630361d

Merge "Finalized all components of range analysis needed for dynamic bce."
am: d93223d796

* commit 'd93223d7966bee1ae91a9e224c1dd56de1aa3f50':
  Finalized all components of range analysis needed for dynamic bce.

Merge "Finalized all components of range analysis needed for dynamic bce."

Merge "X86-64: Split long/double constant array/field set"
am: f2a93f4d82

* commit 'f2a93f4d824d6ea0f5858c774ba03f1c002a6aaa':
  X86-64: Split long/double constant array/field set

Merge "X86-64: Split long/double constant array/field set"

X86-64: Split long/double constant array/field set

A long constant needs to be in a register to store to memory.
By allowing stores of constants that are outside of the range of
int32_t, we reduce register usage.

Also support sets of float/double constants by using integer stores.

Rename RegisterOrInt32LongConstant to RegisterOrInt32Constant as it
now handles any type of constant.

Change-Id: I025d9ef889a5a433e45aa03b376bae40f14197d2
Signed-off-by: Mark Mendell <mark.p.mendell@intel.com>

Merge "Fix conditional jump over jmp (X86/X86-64/ARM32)"
am: 3f96f42b37

* commit '3f96f42b3788f9b9b243822212ec686e599279ef':
  Fix conditional jump over jmp (X86/X86-64/ARM32)

Merge "Fix conditional jump over jmp (X86/X86-64/ARM32)"

Merge "Move .dynsym ELF section after .text"
am: f941a32401

* commit 'f941a32401f554a1713aedba44757f6edb664aad':
  Move .dynsym ELF section after .text

Fix conditional jump over jmp (X86/X86-64/ARM32)

Optimize the code generation for 'if' statements to jump to the
'false' block if the next block to be generated is the 'true' block.

Add an X86-64 test for this case.

Note that ARM64 & MIPS64 have not been updated.

Change-Id: Iebb1352feb9d3bd0142d8b0621a2e3069a708ea7
Signed-off-by: Mark Mendell <mark.p.mendell@intel.com>

Merge "Move .dynsym ELF section after .text"

JDWP: fix returned tag for ClassType.NewInstance

Bug: 25490662
Change-Id: I0c369f04342a60caef4f7462005804307a410d18

Merge "Fix flake around instrumentation."
am: 55a77ef5f9

* commit '55a77ef5f99c2b056eb5f2820c1ff3b29c0ec270':
  Fix flake around instrumentation.

Merge "Fix flake around instrumentation."

Move .dynsym ELF section after .text

The main purpose of the dynamic section is to describe where
.text is and how large it is.  It makes more sense to write
this section once we actually know this information.

Also write string sections (.dynstr and .strtab) before the
corresponding symbol sections because the strings take more space.

Neither of these changes is important now since we build the
sections in memory first, but it is move towards streaming of
the ELF file directly do disk.

Due to alignment requirements, the move increases the oat file
size by one page (4K) which is insignificant.

Change-Id: I61cc9f934b031d26a3a1f6007b51315e1538e3eb

Fix flake around instrumentation.

bug:25492507
Change-Id: Ia4de07ecaf33c3153f5d93473671d2c93593f52d

Merge "Code cleanup to avoid CompilerDriver abstractions in JIT."
am: b24301b06b

* commit 'b24301b06b31b463f7e92ebc9a8f75839e54b746':
  Code cleanup to avoid CompilerDriver abstractions in JIT.

Merge "Code cleanup to avoid CompilerDriver abstractions in JIT."

Merge "Properly delete the jit thread pool."
am: 2b13eaa500

* commit '2b13eaa50065346d4a1345b05fe2e1455df6d71d':
  Properly delete the jit thread pool.

Merge "Properly delete the jit thread pool."

Properly delete the jit thread pool.

bug:25461989
bug:25462600
Change-Id: I273cf256285d01c085e4dea1d997955d029361b9

Code cleanup to avoid CompilerDriver abstractions in JIT.

Avoids allocating a CompiledMethod.

Change-Id: I35b4aa0d7c74daba68e827a01e71c300fce3b3bf

Merge "Revert "Use Jack lang dev version for compiling art tests""
am: ad15539f5a

* commit 'ad15539f5a1135d82439c5eb3ed4bb80c422a734':
  Revert "Use Jack lang dev version for compiling art tests"

Merge "Revert "Use Jack lang dev version for compiling art tests""

Revert "Use Jack lang dev version for compiling art tests"

Tests fail.

This reverts commit 55fed87ff8b60d55589ee7df75b5080b17bce541.

Change-Id: I02c064c6774d72bc1349dbaab4d05ef69c537952

Merge "Revert "Add basic image writer support for app images""
am: c09b334456

* commit 'c09b3344568f9533a0954f5dfba20d65d05ec4c1':
  Revert "Add basic image writer support for app images"

Merge "Revert "Add basic image writer support for app images""

Revert "Add basic image writer support for app images"

interpreter and jit tests fail.

Bug: 22858531

This reverts commit c033474cfbfe1e963c07fa5c38aed02e35ed6f91.

Change-Id: Ic12a3e2a1908ac0db52d21a0b44b2508c88b2585

Merge "Use Jack lang dev version for compiling art tests"
am: f97ad8115d

* commit 'f97ad8115d00d5bd9bbd9c485a7a4b8107f620a2':
  Use Jack lang dev version for compiling art tests

Merge "Use Jack lang dev version for compiling art tests"

Merge "Fix continuous build"
am: e597d59771

* commit 'e597d59771ac85a18744587eebc8711286aae5d4':
  Fix continuous build

Merge "Fix continuous build"

Fix continuous build

Stop building vogar.jar target since it no longer exists.

Change-Id: I4b834abd519736199c5ee1e5881b3531dbbf2c6f

Merge "Add basic image writer support for app images"
am: a930832a4e

* commit 'a930832a4ef0a2ef0835c5db4650750018ab0765':
  Add basic image writer support for app images

Merge "Add basic image writer support for app images"

ART: Change behavior for rethrowing init failures (2)

Always store the pending exception when making a class erroneous.
Instead of filtering by ExceptionInInitializerError, add an option
to the rethrow that enforces a NoClassDefFoundError, which is required
by the specification.

Use the libcore companion change to add the stored error (if any) as
a cause to the NoClassDefFoundError, which should significantly help
tracking down issues.

Fix run-test 008 to expect spec-compliant behavior. Test that a cause
has been set.

Bug: 25445103
Change-Id: I6a0dc54e78312283faf23415887eff387531407f

Add basic image writer support for app images

Needed to handle references from app image -> boot image.

Generate app images for tests to enable some testing.

Bug: 22858531

Change-Id: I1af98b6c4dfcb3a147fb5b0dea64aa4946c7ce57

Merge "ART: Change behavior for rethrowing init failures"
am: accc24985c

* commit 'accc24985c08e98a27f43bc856fba4c3c48e79e9':
  ART: Change behavior for rethrowing init failures

Merge "ART: Change behavior for rethrowing init failures"

Merge "Revert "ART: Implement DeadPhiHandling in PrimitiveTypePropagation""
am: 3c60103948

* commit '3c60103948a8357497765965234eff7a56164061':
  Revert "ART: Implement DeadPhiHandling in PrimitiveTypePropagation"

Merge "Revert "ART: Implement DeadPhiHandling in PrimitiveTypePropagation""

Revert "ART: Implement DeadPhiHandling in PrimitiveTypePropagation"

Crashes on YouTube, need to investigate

This reverts commit 1749e2cfb5c5ed4d6970a09aecf898ca9cdfcb75.

Change-Id: If5f133d55dcc26b8db79a670a48fbd4af7807556

ART: Change behavior for rethrowing init failures

Allow to store a Throwable instance or a throwable class. Handle
rethrow accordingly.

Bug: 25444180
Change-Id: I703c2c6eaf34ad0e3bc0f5a104d65f2ff1b212ca

Merge "ART: Implement DeadPhiHandling in PrimitiveTypePropagation"
am: 26bfcec526

* commit '26bfcec526860857410944f0d437957d109ef7ae':
  ART: Implement DeadPhiHandling in PrimitiveTypePropagation

Merge "ART: Implement DeadPhiHandling in PrimitiveTypePropagation"

ART: Implement DeadPhiHandling in PrimitiveTypePropagation

DeadPhiHandling revives non-conflicting phis with environment uses
but does not properly merge types. To not duplicate code, this patch
modifies PrimitiveTypePropagation to deal with conflicts and thus
replaces DeadPhiHandling altogether.

Bug: 24252151
Bug: 24252100

Change-Id: I198c71d1b8167fc05783a5a24aa9f1e3804acafe

Finalized all components of range analysis needed for dynamic bce.

Rationale: added ability to generate taken-test, prompt back need
           for finite-test; cleaned up the API now that bounds
           check needs are all known.

Change-Id: I3d09b249965d1a980c09381240de175ca4b2e455

Use Jack lang dev version for compiling art tests

Change-Id: Iada2d3919d5c5b5b4d32a607d9cddb89dad49648

Merge "Delay emitting CFI PC adjustments until after Thumb2/Mips fixup."
am: c8894ab502

* commit 'c8894ab5021aecd0fa5eba94af47f732914af33b':
  Delay emitting CFI PC adjustments until after Thumb2/Mips fixup.

Merge "Delay emitting CFI PC adjustments until after Thumb2/Mips fixup."

Delay emitting CFI PC adjustments until after Thumb2/Mips fixup.

On Mips also take into account out-of-order CFI data emitted
from EmitBranches().

Change-Id: I03b0b0b4c2b1ea31a02699ef5fa1c55aa42c23c3

Merge "Fix test in case we run the ART_TEST_TRACE tests."
am: b178e11d9e

* commit 'b178e11d9eb44fc0ac666049f3aa14f7f14ffd64':
  Fix test in case we run the ART_TEST_TRACE tests.

Merge "Fix test in case we run the ART_TEST_TRACE tests."

Fix test in case we run the ART_TEST_TRACE tests.

Change-Id: Ic998a29f2d5c573f34def798eb4f2fe5cf30e8a5

Merge "JDWP: fix java.lang.String creation"
am: 6bf6e438cd

* commit '6bf6e438cd4746effce0b26d504c54100191c988':
  JDWP: fix java.lang.String creation

Merge "JDWP: fix java.lang.String creation"

Merge "Fix TODO on instrumentation and add some more DCHECKs."
am: 12cbc87247

* commit '12cbc872477dd278a27599d8714381657211b184':
  Fix TODO on instrumentation and add some more DCHECKs.

Merge "Fix mips32oc ArraySet null constant assignment"
am: 147bc832ca

* commit '147bc832ca81c4297c38c14eb7d5a8e281d4483d':
  Fix mips32oc ArraySet null constant assignment

Merge "Fix TODO on instrumentation and add some more DCHECKs."