Merge "Fix duplicate checks in JNI macro assemblers."
am: 2af1aa066e

Change-Id: I658e24a12a0086257acfa5ec88ffb91d2d25f03d

Merge "Fix duplicate checks in JNI macro assemblers."

Merge "Adjust spacing before NOLINT comments in ART."
am: c17d21d442

Change-Id: Ic44e4396616b9f8e692b6c530553e9f98305e479

Merge "Adjust spacing before NOLINT comments in ART."

Fix duplicate checks in JNI macro assemblers.

Instead of checking one register twice, check the other
register passed to the function.

Test: Run ART test suite on host and Nexus 5.
Bug: 30739460
Change-Id: If868ea14f5c192982488ed065b279a019b5b7cf4

Adjust spacing before NOLINT comments in ART.

Note that neither clang-tidy nor cpplint.py complain about
these style "issues", precisely because of the NOLINT
comments.

Test: WITH_TIDY=1 WITH_TIDY_CHECKS='-*,misc-macro-parentheses' mmma art
Change-Id: Id692fd394ffbd4fe208cbbe4407b4d5e208462bb

Merge "ARM: Add vldm/vstm assembler support."
am: dce74be0c4

Change-Id: Ibb492f8477f8552b30967ab89f0b633b29b52959

Merge "ARM: Add vldm/vstm assembler support."

Merge "Initialize Heap's collector pointers to null."
am: 3d1d18d74d

Change-Id: I043cf61f87dc57402e836433340084b3fec03058

Merge "Initialize Heap's collector pointers to null."

Merge "Revert "Iterative move coalescing for gc regalloc""
am: 0c283627c0

Change-Id: I25700c95e5c7e35325636192bb8a494d6d425830

Merge "Revert "Iterative move coalescing for gc regalloc""

Revert "Iterative move coalescing for gc regalloc"

There are lifetime issues with allocators and coloring
iterations that got flagged by valgrind.

This reverts commit 465ed699e810868fe5bb39730e6d149a4734372d.

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

Merge "Remove kRestrictCompilationFiltersToImage"
am: eef560e257

Change-Id: Ib645687b2f18dc58a8f9a0891baeff8a653acb45

Merge "Remove kRestrictCompilationFiltersToImage"

Merge "Enable compiled-methods= dex2oat switch"
am: 8be74495ac

Change-Id: I64c6fb97997642108337463054428c48ff9a4d28

Merge "Enable compiled-methods= dex2oat switch"

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

Initialize Heap's collector pointers to null.

Test: Run ART test suite on host.
Bug: 30807718
Change-Id: I9b83b44df250b00af48dc1aa653d78aea84d0874

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"

ARM: Add vldm/vstm assembler support.

Change-Id: Id97417de8ebd07fc1fbee5d17e4b1c620a2c44cc

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"

Enable compiled-methods= dex2oat switch

Pass compiled_methods instead of nullptr from dex2oat to
compiler driver.

Test: Run dex2oat with compiled-methods and dump-passes
set. Look at the dump-passes output and verify that only
methods provided in compiled-methods are compiled.

Change-Id: Ic7249878f7bde5fdb18f9cc348b7ba82de486df8

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

Remove kRestrictCompilationFiltersToImage

Removed compiler driver kRestrictCompilationFiltersToImage flag,
impact is in the noise.

Change-Id: I50be7a201f677fa0cb6db38d9b5f039754e1f98c

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