Merge "Iterative move coalescing for gc regalloc"
am: 592c9bd2ad

Change-Id: I0001020989745ff605d82a712bc57da88c0295b0

Merge "Iterative move coalescing for gc regalloc"

Iterative move coalescing for gc regalloc

Implement iterative move coalescing for graph coloring
register allocation. Based on Andrew Appel's implementation
in "Modern Compiler Implementation in Java", modified to
support constraints such as pair intervals.

Test: ART_TEST_OPTIMIZING_GRAPH_COLOR=true m test-art-host

Change-Id: I8642297d3bd798a4fc4de4b356ac3304098471a5

Merge "x86/x86-64: Shorter fast-path for read barrier field load."
am: 9d32586b52

Change-Id: I817da2e93d1cc5e0198c1ed2edab7b21951227e8

Merge "ARM64: Use libvixld when compiling for debug mode."
am: 0c7972c6c2

Change-Id: I8853f50c9c542104183ddbcf4bf36c2baa1d9e77

Merge "x86/x86-64: Shorter fast-path for read barrier field load."

Merge "ARM64: Use libvixld when compiling for debug mode."

Merge "Full enable new round implementation on x86/x86_64"
am: c531c21794

Change-Id: I8f0c2115bf64d733a9d548060a21449dbf57d18f

Merge "Full enable new round implementation on x86/x86_64"

ARM64: Use libvixld when compiling for debug mode.

VIXL debug mode checks are valuable to catch dangerous code that can
lead to bugs.

This patch includes a couple of fixes for issues spotted by VIXL in debug mode.

Change-Id: I388ae1ffd9256ad74d0b6ce06f79cc7927a5f28a

x86/x86-64: Shorter fast-path for read barrier field load.

Test: Tested with ART_USE_READ_BARRIER=true on host.
Bug: 29966877
Bug: 12687968
Change-Id: I73359495910dacb2cc28f1a21ef9e610bab5a476

Merge "ARM/ARM64: Improve assembler macros for kSaveEverything frame."
am: 33699c9529

Change-Id: Idf569faf84dd37c71b5a75622142e76f933b79df

Merge "ARM/ARM64: Improve assembler macros for kSaveEverything frame."

Merge "ART: Check for more low-space situations in zygote"
am: 8cf44dbedd

Change-Id: I11ebd60a509a49034a44479f74f29d9d640ff0ca

Merge "ART: Check for more low-space situations in zygote"

ART: Check for more low-space situations in zygote

Even if a boot image exists and can be loaded, the rest of the
platform may not be able to boot. We previously only checked
after full image generation and missed this case.

Also prune the cache if there wasn't even space to put the boot
marker.

(cherry picked from commit f242f66cf7137ef681e44a494758976e11579260)

Bug: 30765660
Test: m test-art-host
Change-Id: Icd43746e681c62ce2e4e8745fb17bda65ca60372

Merge "Fix pending exception failure for encoded statics"
am: 34110807ea

Change-Id: Ie0be2f627c540bd22888e3a8e327c8efc23c80f3

Merge "Fix pending exception failure for encoded statics"

Fix pending exception failure for encoded statics

ReadValueToField may allocate a string and cause OOME, we were
not checking this each loop iteration. Throwing an exception
with a pending exception causes an abort.

Bug: 30690988

Test: test-art-host

Change-Id: I2c9751bf0544cf115d9d8e914e58ebccc81bcbf4

Full enable new round implementation on x86/x86_64

Rationale:
Running JIT on Fugu does not always provide a constant area.
In such cases, we need to construct FP constants through stack.
This only applies to x86.

Test: 580-checker-round

BUG=26327751

Change-Id: I7e2c80dafbafbe647cfe9ecb039920bb534c666a

ARM/ARM64: Improve assembler macros for kSaveEverything frame.

On ARM, use vpush/vpop {d0-d15} instead of {s0-s31}.
On ARM64, use 16-byte aligned stp/ldp for FP registers.

Test: Run ART test suite on Nexus 9.
Bug: 30212852
Change-Id: I36c04d3f1f7e03661c501977c3c9ffa7d2942d2f

Merge "Fix negative array size checking"
am: 38dd0e8f0a

Change-Id: Icd6eba521a54f33d5c2c422aacca20923b8e0967

Merge "Fix negative array size checking"

Fix negative array size checking

Mask out the alignment after the size check. Was broken in previous
CL.

Test: target test 412 --64 with CC + baker

Bug: 30162165
Change-Id: Ic4eb7229fb742490cd9193baf0faa2be6b454f38

Merge "ART: Fix JIT profile saver"
am: f2b1529b09

Change-Id: Iadbd6f1101de48515396b26cb0bafc2087dbd631

Merge "ART: Fix JIT profile saver"

ART: Fix JIT profile saver

We miss the case when jit activity notification count exceeds
the threshold. Fix it.

Bug: 30583550
Test: manual test in master

Change-Id: Ib10a34d3c2f5d3cf674b1c4d7a3c1204784865d3
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>

Merge "Rename callee save enumerators."
am: 066187de25

Change-Id: I3172a5e9dd37dc75746a0a63d4758558d0c00b7b

Merge "Rename callee save enumerators."

Merge "Instruction: Add new formats 45cc and 4rcc."
am: 3de6f3f9c9

Change-Id: Icae8eb59fd0b65477411b3da965a9b6fc2464203

Merge "Instruction: Add new formats 45cc and 4rcc."

Merge "ARM: Embed constants in add/sub-long."
am: 4997d14c8d

Change-Id: I0adf61fab5b5b17e29839542c6406d4e1fa5cc73

Merge "ARM: Embed constants in add/sub-long."

Rename callee save enumerators.

And related image method enumerators, macros, etc.
Clean up some entrypoint assembly comments.

This is a follow-up to
    https://android-review.googlesource.com/252348

Test: Run ART test suite on host and Nexus 9.
Bug: 30212852
Change-Id: I2707342d4255c88c547655be83ed97a67e12ae9e

Instruction: Add new formats 45cc and 4rcc.

These are new 4 byte formats with the following properties.

- The first three (16 bit) words of these instructions have the same
  format as 35c and 3rc respectively.
- The fourth 16 bit word encodes an additional constant index reference.

This change includes placeholder opcodes for invoke-polymorphic and
invoke-polymorphic/range which will be the first dex instructions that
use this new format. In addition to a method_idx that gives the invoked
method, these instructions also provide a proto_idx which gives the
(static) type signature of the call site.

The only reason these are included in this change is because we need
an instruction with a given format to write a unit_test using the Instruction
API.

bug: 30550796
test: make test-art-host

Change-Id: I02612ddee47169757175a8079d82f811f6545945

Merge "Revert experimental lambda feature."
am: c218427ab9

Change-Id: Id73fbf20e9f119552f1a0b623575d6caf8b414bf

Merge "Revert experimental lambda feature."

Merge "JDWP: fix method argument count"
am: 45b1718769

Change-Id: Ifeaec3aced92c63c81eff1b795d19a3e58def03c

Merge "JDWP: fix method argument count"

Merge "Add a graph coloring art test option"
am: e4520c185d

Change-Id: Iaa861f3c5d314f28e54b9ca0a11bdaaf8bbb75c6

Merge "Add a graph coloring art test option"

Merge "ART: Extract JNI macro assembler for arm"
am: dfdf1e5343

Change-Id: Ide2cdde3c6b2ec58ee89710184f2369667bd728e

Merge "ART: Extract JNI macro assembler for arm"

Add a graph coloring art test option

Test: ART_TEST_OPTIMIZING_GRAPH_COLOR=true m test-art-host

Change-Id: I1bd7c20cdd372ed227204c990484d123c67bdfa8

Merge "ART: Extract JNI macro assembler for arm64"
am: c21f1c5b29

Change-Id: I1b943b3efd7ff12730d0bb4a20ec2083b1a9d064

ART: Extract JNI macro assembler for arm

Extract the JNI assembler parts from the regular assembler.

Change-Id: Ice39ffeb854f8651e93443dd69f4c9e0e802975c
Test: m test-art-target (N6)

Merge "ART: Extract JNI macro assembler for arm64"

Merge "Card mark holding class instead of declaring class"
am: 0ed44f9595

Change-Id: I6b4eeb9b956a6b5def8b28366822af4b0d44c686

Merge "Revert "Revert "ARM64 asm for region space array allocation"""
am: 4c3cc6ca2a

Change-Id: Ib650df895187d92141998106c7044720db7de4f7

Merge "Card mark holding class instead of declaring class"

Merge "Revert "Revert "ARM64 asm for region space array allocation"""

Card mark holding class instead of declaring class

For profiling info, we need to mark the card of the holding class
instead of declaring class. This is required for GC correctness since
the GC relies on the card table to track cross space references.

Test: test-art-host ART_TEST_JIT=true

Bug: 30655270

Change-Id: Ia4690219ded0df38032b644440273e06bc303956

Revert "Revert "ARM64 asm for region space array allocation""

Also added missing large object check. No regression from the check
N6P CC EAAC time at 1313 for 10 samples vs 1314 before reverts.

Bug: 30162165
Bug: 12687968

Test: test-art-target with CC + heap poisoning

This reverts commit 6ae7f3a4541e70f04243a6fe469aa3bd51e16d79.

Change-Id: Ie28f652f619898d7d37eeebf3f31a88af8fac949

Merge "Clean up VLOG(oat) in OatFileAssistant."
am: 3f3201a89e

Change-Id: Iff3cd700f34ec55d3162746d213bcf5f914d441c

Merge "MIPS64: Properly preserve $gp in art_quick_test_suspend"
am: c1f9241b98

Change-Id: Ibba332862ae5c625cc647884161cf2d5d52b9bee

Merge "Clean up VLOG(oat) in OatFileAssistant."

Merge "MIPS64: Properly preserve $gp in art_quick_test_suspend"

JDWP: fix method argument count

The arguments count must include 'this' for instance method.

Bug: 30673627
Test: make test-art-host
Test: art/tools/run-jdwp-tests.sh --mode=host --variant=X64
Change-Id: Ie582e3bb3e27a2572ce5d3c66990f02650eee61b

Merge "Fix setting FdFile::ReadOnlyMode() flag"
am: d59b196d6d

Change-Id: Ibf16a9bfd0d4070a1658edabf5cfd1db3e3bb173

Merge "Fix setting FdFile::ReadOnlyMode() flag"

MIPS64: Properly preserve $gp in art_quick_test_suspend

$gp register was restored in RESTORE_SAVE_EVERYTHING_CALLEE_SAVE_FRAME
with value from $t8, but $t8 has been clobbered in the meantime.

Calculation of $gp is done as a part of the macro
SETUP_SAVE_EVERYTHING_CALLEE_SAVE_FRAME. This allows us to use version
of .cpsetup where $gp is preserved on the stack, instead in another
register.

This fixes over 60 ART tests.

Additionally this suppresses assembler warnings for MIPS32 and MIPS64:
    Warning: used $at without ".set noat"

Test: mma test-art-target-run-test on QEMU

Change-Id: Ic92680c3cb2a4405353b3640c836e92c0231bd15

Fix setting FdFile::ReadOnlyMode() flag

The Unix flag O_RDONLY is defined as zero and hence its presence
cannot be tested with '(flags & O_RDONLY) != 0'. This used to be
broken in FdFlag when setting its internal `read_only_mode_` flag.

Test: m test-art-host-gtest-fd_file_test
Change-Id: Ib48abfc908c7032f031450a1574130e06f6c3bab

Revert experimental lambda feature.

This is a revert of the following changes :

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

6918bf13eb855b3aa8ccdddda2d27ae8c60cec56.
lambda: Experimental support for capture-variable and liberate-variable

fc1ccd740b7c8e96dfac675cfc580122cd1b40a6.
lambda: Infrastructure to support capture/liberate-variable dex opcodes

e2facc5b18cd756a8b5500fb3d90da69c9ee0fb7.
runtime: Add lambda box/unbox object equality

2ee54e249ad21c74f29a161e248bebe7d22fddf1.
runtime: Partially implement box-lambda and unbox-lambda experimental opcodes

158f35c98e2ec0d40d2c032b8cdce5fb60944a7f.
interpreter: Add experimental lambda opcodes for invoke/create-lambda

a3bb72036f5454e410467f7151dc89f725ae1151.
Added format 25x to dexdump(2).

Plus surrounding cleanups.

Test: make test-art
Change-Id: Ic6f999ad17385ef933f763641049cf721510b202

Merge "Use Thread.dispatchUncaughtException() rather than duplicating logic."
am: c8ba678762

Change-Id: Ifdd9c4b4e8ee45eb36bb763703f3e73de25262c1

Merge "Use Thread.dispatchUncaughtException() rather than duplicating logic."

ART: Extract JNI macro assembler for arm64

Extract the JNI assembler parts from the regular assembler.

Change-Id: I0b0ad32e18f585b75e9da0237afe082c25a1d291
Test: m test-art-target (N9)

Merge "Fix missing spaces in checker tests"
am: 5c6233c00f

Change-Id: I7c2b7c776a8a400286f17c92ea561f470d93f00c

Clean up VLOG(oat) in OatFileAssistant.

Refactor GetOatDexFile to have an error message out parameter.

Change-Id: I6b933f1fcfc9726c051a9d9678d92587cc02501e
Test: OatFileAssistantTest with and without -verbose:oat logging turned on.
Test: m test-art-host

Merge "Fix missing spaces in checker tests"

Merge "ART: Improve addr2line"
am: 7ab52cd6c0

Change-Id: I7813a75eb3ef4f0cf080e7f773d8c7b711ae2044

Fix missing spaces in checker tests

This fixes missing spaces in few checker tests.

Change-Id: Id2e7ee4da0077dfd6676526cbd203a98e2bfd7ee

Merge "ART: Improve addr2line"

Merge "ART: Extract JNI macro assembler for x86"
am: c0362b5f93

Change-Id: Iea4afe81d5a52315c968947d5081973b80424e1c

Merge "ART: Extract JNI macro assembler for x86-64"
am: af2962caff

Change-Id: I447c33ff5875c04735074685ad7e7e57e90a3ccd

Merge "ART: Extract JNI macro assembler for x86"

Merge "ART: Extract JNI macro assembler for x86-64"

Merge "Implement running user defined list of passes"
am: 2f8d19f299

Change-Id: I6e5a30f02e077d52955d595acfb720126e0572c7

Merge "Implement running user defined list of passes"

ART: Extract JNI macro assembler for x86-64

Extract the JNI assembler parts from the regular assembler.

Test: m test-art-host
Change-Id: I291fb76ad9232123b4c1992488ee81fec3c1db47

ART: Extract JNI macro assembler for x86

Extract the JNI assembler parts from the regular assembler.

Change-Id: I0b47af03ca12798f58dafec716a529eb0edf9649
Test: m test-art-host

Merge "ARM/ARM64: Improve Mterp */lit8 assembly."
am: 61cd79bdea

Change-Id: I40e98474597ead6c5f7aa641a82b0787290ba1fb

Implement running user defined list of passes

This change introduces new dex2oat switch --run-passes=. This switch
accepts path to a text file with names of passes to run.
Compiler will run optimization passes specified in the file rather
then the default ones.

There is no verification implemented on the compiler side. It is user's
responsibility to provide a list of passes that leads to successful
generation of correct code. Care should be taken to prepare a list
that satisfies all dependencies between optimizations.

We only take control of the optional optimizations. Codegen (builder),
and all passes required for register allocation will run unaffected
by this mechanism.

Change-Id: Ic3694e53515fefcc5ce6f28d9371776b5afcbb4f

Merge "ARM/ARM64: Improve Mterp */lit8 assembly."

Merge "ahat: Target Java 1.7."
am: 9cff32df75

Change-Id: I982e0b6bd686d0327ed8c8b49c9c8d1c8b3cd45d

Merge "ahat: Target Java 1.7."

Merge "Revert "ARM64 asm for region space array allocation""
am: 61d4850eba

Change-Id: I168b85fde8f11ae143429f7f76bdebc211ff1b04

Merge "Revert "ARM64 asm for region space array allocation""

Merge "Support Mockito in art buildbot tests"
am: 1f69674a36

Change-Id: If950283797508087df7306e0f6b87bf4d32c90a4

Revert "ARM64 asm for region space array allocation"

This change breaks many tests on the ARM64 concurrent
collector configuration.

Bug: 30162165
Bug: 12687968

This reverts commit f686c3feabe3519bedd1f3001e5dd598f46946ef.

Change-Id: I5d7ef5fa2ffb6a8d9a4d3adbcc14854efa257313

Merge "Support Mockito in art buildbot tests"

Support Mockito in art buildbot tests

Test: art/tools/buildbot-build.sh --host -j60 && \
art/tools/run-libcore-tests.sh --mode=host --variant=X32

Change-Id: Ifa32927251fa944df7bf9704f8164083aeb557a1

Merge "Remove OldSystemTest#test_load from known failures."
am: 542c6df669

Change-Id: Ida34775a0032cd5342292c83bce46a9589eacb6e

Merge "Remove OldSystemTest#test_load from known failures."

ARM/ARM64: Improve Mterp */lit8 assembly.

Embed shifts in operations when possible. Use UBFX for shift
distance extraction - for ARM this saves an instruction, for
ARM64 this just improves readability as the reader doesn't
need to know that the higher bits are unused by shifts.

Test: Run ART test suite on Nexus 9.
Change-Id: I5666f3eaded618d3f6cb754c4a431fd9548e746d

Remove OldSystemTest#test_load from known failures.

Test: libcore.java.lang.OldSystemTest
Bug: 26326992
Change-Id: Ic77dab37c05044790278407867339a198e8be9a8

Merge "Enable IntermediateAddress for primitive arrays with read barriers."
am: 51616fb233

Change-Id: I976378f1bf81740d651a89d983b82f184e3a3276

Merge "Enable IntermediateAddress for primitive arrays with read barriers."

Enable IntermediateAddress for primitive arrays with read barriers.

Test: ART host and target (ARM, ARM64) tests.
Bug: 26601270
Bug: 12687968
Change-Id: I6736ba7b1809bece1bf3cd82c69e4f42a0d3c4a7

Merge "ART: Factor out native stack dumping"
am: c10ad42302

Change-Id: I6d19b60d96f6a795f728f9ee2be20f8f459c4421

Merge "ART: Factor out native stack dumping"