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"

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

Merge "MIPS64: Implement virtual intrinsic slow path"

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

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"

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

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

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

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

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

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

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

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"

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"

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"

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"

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

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

Fix test in case we run the ART_TEST_TRACE tests.

Change-Id: Ic998a29f2d5c573f34def798eb4f2fe5cf30e8a5

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

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

Merge "Fix mips32oc ArraySet null constant assignment"

Fix TODO on instrumentation and add some more DCHECKs.

bug:25343683
bug:25438583

Change-Id: I232deb1b6761466b514c687ce304f61928755cdc

Fix mips32oc ArraySet null constant assignment

Fix assigning a null constant to an array index. Previously this
would unnecessarily invoke pAputObject.

This commit fixes previous test failure: 521-checker-array-set-null.

Change-Id: I02ff5296fa3a48b021c0dfd9bb10628ba6e5d5e5

Merge "lambda: Minor capture-variable/liberate-variable clean-up after post-merge reviews."

ahat: limit default number of results shown.

Previously, ahat had performance issues rendering large pages. This
change causes ahat to limit the number results shown in large pages by
default, requiring the user to explicitly request more information if
they care about it.

Bug: 25114227
Change-Id: Ief67396be254be4c84e6971f5b903a701206e17b

Merge "Annotate References with their referent."

Annotate References with their referent.

Also, don't include annotations in an object's link. That way we can
distinguish between the object link and any links in the annotations.

Change-Id: I3fef3f5b2c343699f527cad9efa7c93a4b35a3a4

Merge "Fix broken handling of instrumentation lists."

Merge "Make things generally quieter."

Make things generally quieter.

* Give run-test a --quiet flag that causes it to only print on errors.
* Give cpplint a --quiet flag that causes it to not print anything
  when there are no errors.
* Add a ART_TEST_QUIET flag to build/Android.common_test.mk which
  makes run-test targets much quieter when true (the default). With
  this flag only failures will be printed.
* Make build/Android.cpplint.mk pass the new cpplint --quiet flag so
  that only failures will be printed.

Before:
    [ 96% 5715/5906] build test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-461-get-reference-vreg32
    test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-448-multiple-returns32 RUNNING
    /usr/local/google/buildbot/src/googleplex-android/master-art-host/art/test/448-multiple-returns: building...
    /usr/local/google/buildbot/src/googleplex-android/master-art-host/art/test/448-multiple-returns: running...
    /usr/local/google/buildbot/src/googleplex-android/master-art-host/art/test/448-multiple-returns: succeeded!
    test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-448-multiple-returns32 PASSED

After:
    [ 96% 5715/5906] build test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-461-get-reference-vreg32

Change-Id: Idf6fce7f48a619f83254b48861dbd7f8eb4ebdbf

Merge remote-tracking branch 'goog/stage-aosp-master' into HEAD

Merge "Limit summary string lengths to 200 characters."

Fix broken handling of instrumentation lists.

- We cannot copy before iterating, as entries might be deleted.
- We cannot remove entries in the list, as mutators could be
  currently iterating over it.

Solution in this change is to never remove list entries, but
put null when a listener is removed. When adding a listener, we
will either put it where there is a null slot, or at the end
of the list if there is no null slot.

Change-Id: Id94582fd971cd56bcb445caff64270d21987f700

JDWP: fix java.lang.String creation

Bug: 25439464
Change-Id: I56f11ed942585e8110dbbba1178cf11ec76e032f

Merge "Use --oat-fd instead of --oat-file when calling dex2oat."
am: ea834bbed4

* commit 'ea834bbed4f39d23cb2aafb1aa5cf73f71bcf21a':
  Use --oat-fd instead of --oat-file when calling dex2oat.

Merge "Use --oat-fd instead of --oat-file when calling dex2oat."

Merge "Fix deadlock with the JIT code cache."
am: df6dc42ba2

* commit 'df6dc42ba2ca0fa43ba970ba2e60977422105f7e':
  Fix deadlock with the JIT code cache.

Merge "Fix deadlock with the JIT code cache."

Fix deadlock with the JIT code cache.

Also remove hack done for ThreadStress.

Change-Id: Ie25c3bca08d9f2b8919706fa3fc26c5ab213f4a3

Merge "Minor improvements for the CC collector."
am: 4bbe7807f3

* commit '4bbe7807f313bb8e59131812c31bf31513094f8f':
  Minor improvements for the CC collector.

Merge "Minor improvements for the CC collector."

lambda: Minor capture-variable/liberate-variable clean-up after post-merge reviews.

Change-Id: I64f867d4ed5a5efcac138097f38efe4bb7f1281d

Merge "Fix minor problems with interface verification and default methods."
am: 3482c27127

* commit '3482c271278b30ca7a5c6206b7f80332616f6a84':
  Fix minor problems with interface verification and default methods.

Merge "Fix minor problems with interface verification and default methods."

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

* commit '86a8e247e3cd873c03bb79f79bf88a1b4c4e0597':
  Revert "ART: Update DCE to work with try/catch"

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

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

This reverts commit ce52901e2c8377fc1c331ae0faf7fbcb46b9da97.

Change-Id: I6b3a1f2a3dc036030b089b0df2005ecefa66b949

Minor improvements for the CC collector.

- Split Mark() and inline its first part.
- Make sure some other routines are inlined.
- Add some UNLIKELY's.
- Use VisitConcurrentRoots().

Ritz EAAC GC time decreased from 28.9 -> 27.6s (-4.5%) on N5.

Bug: 12687968

Change-Id: I7bd13f162e7daa2a5853000fb22c5fefc318994f

Merge "ART: Update DCE to work with try/catch"
am: 808e752491

* commit '808e7524918da5f7f41d2ceaa58dbb381e5708ff':
  ART: Update DCE to work with try/catch

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

ART: Update DCE to work with try/catch

Dead block elimination was previously disabled because it needed
to be updated. With this patch, try/catch blocks can be removed
as a result of a dead if/switch branch.

Change-Id: I3261060bf24fd5fe7bb0f989247f0ef62ec5fd7b

Merge "Add application image dex2oat options"
am: 49560fcce5

* commit '49560fcce5b9a07108f7cc75a3fb0fa265e5fa0e':
  Add application image dex2oat options

Merge "Add application image dex2oat options"

Add application image dex2oat options

Bug: 22858531
Change-Id: Ia6a2a832b20b5bb70672521cd06f679335efbd92

Merge "Fix tests flakiness with jit when using Proxy classes."
am: e00197d605

* commit 'e00197d60579838974c02853fd3371ac761922ae':
  Fix tests flakiness with jit when using Proxy classes.

Merge "Fix tests flakiness with jit when using Proxy classes."

Fix tests flakiness with jit when using Proxy classes.

We cannot copy the entry point between ArtMethod when
the entry point has been JITted. We put the interpreter
bridge instead.

bug:25334878

Change-Id: I65a50cc1f10a5a152733807f8c85fb3ed81c5829

Merge "Fix stalled comment from last commit."
am: 78ef3fee3f

* commit '78ef3fee3fcf0b2cfb68b3e943cd144d389313fd':
  Fix stalled comment from last commit.

Merge "Fix stalled comment from last commit."

Fix stalled comment from last commit.

Change-Id: Ic54821c80a8c6ab76d44e2a81a6f821b5741dc53

Merge "Fix inlining and lse bugs with unresolved access."
am: 816b2b3f70

* commit '816b2b3f704d61ba608ad2c3513d4697e7fb5dfe':
  Fix inlining and lse bugs with unresolved access.

Merge "Fix inlining and lse bugs with unresolved access."

Fix inlining and lse bugs with unresolved access.

bug:25414532

Change-Id: I48b6660754774ea3e8a62a74175b1aa3728e0151

Merge "Remove extra space at end of line in output."
am: 43c6d31c43

* commit '43c6d31c43c05d97511bb9658964fd3d2f32b915':
  Remove extra space at end of line in output.

Merge "Remove extra space at end of line in output."

Merge "Remove package on which we cannot revert."
am: f0b308bdba

* commit 'f0b308bdba546081d64d1e593395f8957b3a2e89':
  Remove package on which we cannot revert.

Merge "Remove package on which we cannot revert."