Merge "ARM64: Use the zero register in the parallel-move resolver."
am: 89bd8358c0

Change-Id: I17b9c72e22a9bfa60659d81994916adbf787deb1

Merge "ARM64: Use the zero register in the parallel-move resolver."

Merge "Fix constant type"
am: 01b7982a45

Change-Id: Ie4b0bb80c820e9483348ec75b932c6ef04ccfeeb

Merge "Fix an assertion in the non-Baker read barrier ARM64 slow path."
am: 399e332bd4

Change-Id: Ie2fb93b0f2cc9f7bac3e5f565f39f063bd551926

Merge "Fix constant type"

Merge "Fix an assertion in the non-Baker read barrier ARM64 slow path."

Fix constant type

Remove the unnecessary cast to long for DCHECK, a nit that was omitted
from the last CL, https://android-review.googlesource.com/#/c/248243/.

Bug: b/28856653
Test: test-art-host-gtest-dex_file_test
Change-Id: I8ca18d19cbac61e40ddfbe32729aef5c20347174

Merge "ART: Extend run-test timeout logging time"
am: ae9408f46a

Change-Id: I4d236e966624f3c2b5f9bb7b56b2f96c45cd0a6b

Merge "ART: Extend run-test timeout logging time"

Merge "Scope the compilation and Create the ProfilingInfo in the loop."
am: 53d9a24c77

Change-Id: I822c8da6630ec0dc77107e4d2cc55e35e6e929e4

Merge "Scope the compilation and Create the ProfilingInfo in the loop."

Fix an assertion in the non-Baker read barrier ARM64 slow path.

Test: m ART_USE_READ_BARRIER=true ART_READ_BARRIER_TYPE=TABLELOOKUP test-art-target (ARM64).
Bug: 30563701
Change-Id: Ib7969bd78e9c14cc2be9d5527d80385486b4f409

ARM64: Use the zero register in the parallel-move resolver.

When moving zero to a stack slot, this will change

    mov temp, #0
    str temp, [...]

into

    str zr, [...]

Change-Id: I2211b00d70f3fa0a02e781c90198757290b2bf89

Merge "Return error directly when uncompressed length is 0"
am: d16ae7fe70

Change-Id: I7b99f01d906faf1a599aefc3217c77735cc4ab1c

Merge "Return error directly when uncompressed length is 0"

Return error directly when uncompressed length is 0

If uncompressed length is 0, in function MemMap::MapAnonymous, it will generate a MemMap instance whose base_begin is nullptr.
Then, when OpenMemory, it will create a DexFile instance.
At DexFile::DexFile constructor, header_ is asigned to 0, then header_->string_ids_off_ will cause crash.

Bug: b/28856653
Test: test-art-host-gtest-dex_file_test
Signed-off-by: ganxiaolin <ganxiaolin@xiaomi.com>
Change-Id: Id37f7629f4646cbc385ef054cb83b15be4c59b00

Scope the compilation and Create the ProfilingInfo in the loop.

We need to release the mutator lock to avoid being in a deadlock.

Also allocating the profiling just before compiling in the unlickely
event that it got collected after releasing the mutator lock.

bug:30351473
Change-Id: I98dba646536d857152949b92f6c69ff9a03cf783

Merge "Improve the graph visualizer's output for constant locations."
am: b7e921c905

Change-Id: If4d52743fcf006ebd3b1e5f2e850b5e1a644b246

Merge "Improve the graph visualizer's output for constant locations."

Merge "ARM64: Make the VIXL macro assembler part of ART ARM64's assembler."
am: 524a76c933

Change-Id: Ibc00f0daa7bdda524e79c7bc0f9eac73fe2efe67

Merge "ARM64: Make the VIXL macro assembler part of ART ARM64's assembler."

Merge "Split art-run-tests between building and packaging"
am: 8055598c8d

Change-Id: I5487ba15f85e99a5a98258bb08ae06bdfb736f5c

ART: Extend run-test timeout logging time

Double the time a test is allowed to attempt logging after it
timed out to two minutes.

Also change from using nested timeouts to using "-k" in general,
not just for host dex2oat.

Bug: 30351473
Test: m test-art-host
Change-Id: I134c7b220f69eaab7ea430a5c7ce45d77b091996

Merge "Split art-run-tests between building and packaging"

Merge "Align method code rather than method header in oat files."
am: 35bb22f05c

Change-Id: I3b276be533e4d4841e3fbd8ac06aa6071c142d70

Merge "Align method code rather than method header in oat files."

Merge "ART: Fix Mips Thread offset checks"
am: aa21d0ecbb

Change-Id: I62d7910c1ff5baa0f541637b3ee07a7bfa70f93e

Merge "ART: Fix Mips Thread offset checks"

ARM64: Make the VIXL macro assembler part of ART ARM64's assembler.

This avoids a dynamic allocation of the VIXL macro assembler.

Change-Id: I4cd62678d0978f1ad6f32ea0ce7279e09152be38

ART: Fix Mips Thread offset checks

Follow-up to commit 542451cc546779f5c67840e105c51205a1b0a8fd.

Bug: 30373134
Test: m test-art-host
Change-Id: I1d0202896e683cb20db30e8aac1c5ebcacc96467

Improve the graph visualizer's output for constant locations.

Change-Id: I423fb378ee61fb53c3b328fc74f4e95cdef0992a

Merge "DexFile: Add basic support for v38 dex files."
am: c984725d00

Change-Id: Ie679587daf5836df3835aef1e2488921b1fbb000

Merge "ARM: Embed 0.0 in VCMP."
am: 3719016185

Change-Id: Id2f1ba6b265accac7feb72cec5b481b5c2858a8c

Merge "DexFile: Add basic support for v38 dex files."

Merge "ARM: Embed 0.0 in VCMP."

ARM: Embed 0.0 in VCMP.

Test: Run ART test suite on Nexus 5.
Change-Id: I5cbbd98c4d64a4d9213e27adcae929ead5099a39

Align method code rather than method header in oat files.

This has always been the intent. For example, the function
name CompiledMethod::AlignCode() shows this intent.

Test: Run ART test suite on host and Nexus 9.
Change-Id: I11ae8963fc537fee853fe82e3aca22e77907eae7

DexFile: Add basic support for v38 dex files.

Makes sure they aren't rejected, essentially. Also adds positive
and negative test cases in dex_file_test.

bug: 30550796
Change-Id: I8c95055a22f8a1435868c8bf3497f9641b7bb900
test: make test-art-host

Merge "dex_instruction: remove unused parameter from instruction definitions."
am: 3d6094531e

Change-Id: Ie3d13a22f79be1a02194b93eea35f080f4a8f934

Merge "dex_instruction: remove unused parameter from instruction definitions."

dex_instruction: remove unused parameter from instruction definitions.

bug: 30550796
test: make test-art-host
Change-Id: Ib80a2d5c548578cefb0552b597e1d50b813ce8ef

Split art-run-tests between building and packaging

We remove the 'obj/PACKAGING' directory during an installclean. Instead
of forcing a rebuild of all of these tests after an installclean, build
them in a different obj directory, then copy them into the PACKAGING
directory to be gathered together by LOCAL_PICKUP_FILES.

The built directory isn't used directly for LOCAL_PICKUP_FILES, since we
don't clean up tests that are no longer defined, so an installclean can
fix that. This matches the behavior of modules installed into the system
image, etc.

This brings my build time for "m -j55 tests" from 3m30s to 35s after
"m installclean".

The total size of these binaries are small (~10MB), so the extra copy is
insignificant.

Change-Id: I20f1c09df25b70df8c7f86167480b3a39e99642a

Merge "ART: Convert pointer size to enum"
am: ba65cc4a71

Change-Id: I223a62cf0e5b58c64400aff22109cad62ab278a0

Merge "ART: Convert pointer size to enum"

Merge "Revert "cpp-define-generator: "make art-update-cpp-defines" to regen asm_support""
am: 1a827a05af

Change-Id: I86b8efd42582dde2406bc18b490f461fc0570f27

Merge "Revert "cpp-define-generator: "make art-update-cpp-defines" to regen asm_support""

Revert "cpp-define-generator: "make art-update-cpp-defines" to regen asm_support"

Reverting to unbreak golem builds.

This reverts commit 3c92b129047b12f0887a2068b3800d7a7dcb7d78.

Change-Id: I78ef481f57fbcd3b3a9f4802dd6c21dabb186856

ART: Convert pointer size to enum

Move away from size_t to dedicated enum (class).

Bug: 30373134
Bug: 30419309
Test: m test-art-host
Change-Id: Id453c330f1065012e7d4f9fc24ac477cc9bb9269

Merge "Fixed bug in disassembly of roundss/roundsd"
am: 8bf927b5c2

Change-Id: I778093acc109aedb21c234dd4a40efea63695dc7

Merge "Fixed bug in disassembly of roundss/roundsd"

Fixed bug in disassembly of roundss/roundsd

Rationale:
These instructions should be marked as load, so that, using
Intel syntax, destination (xmm0) appears at left hand side, as in
   roundss xmm0, xmm1
and not the other way around. First I suspected a bug in the
encoding (hence the test) and even the register allocator, but
since the code behaved correctly, only disassembly was really wrong.

Test: disassembler_x86_test (but nothing for actual disassembly)

BUG=26327751

Change-Id: I060ef57f4d5a64cdc04b97ae8a799d1c0d22da05

Merge "Added direct memory operand support for comiss/sd on x86."
am: e304fc28c4

Change-Id: Iee9114dceeb90feccc670791982815c17136137a

Merge "libsigchain: intercept bsd_signal on lp32"
am: ba001a5577

Change-Id: I197ec9cf35e2c8cff45ca62e9ae501ee87da8701

Merge "Added direct memory operand support for comiss/sd on x86."

Merge "libsigchain: intercept bsd_signal on lp32"

libsigchain: intercept bsd_signal on lp32

bsd_signal is a deprecated synonym for the signal. Nevertheless
there are apps and libraries calling this function.

libsigchain needs to intercept these calls and handle them the same
way it handles signal() call.

Bug: http://b/30562229
Test: readelf --dyn-sym app_process32 and check that bsd_signal is exported
      readelf --dyn-sym app_process64 and check that bsd_signal is not exported
Change-Id: I3aa41632bb015568d8524a82f1c97e4443ec0d6c

Added direct memory operand support for comiss/sd on x86.

Rationale:
Memory operands allows for shorter instruction sequences.
Something that may be used e.g. the upcoming implementation
of the round intrinsic.

Bug=26327751

Change-Id: Ifd976ed86917de1879fa036cbbbff5edf4fb741d
Test: assembler_x86_test

Merge \"Put the deletion of profiling info under a GC critical section.\"
am: 1a7c33704a

Change-Id: I4952fd9f60b7b5c85b2acba7d808f008601029d0

Merge "Put the deletion of profiling info under a GC critical section."

Merge \"Revert \"Revert \"Scan immune spaces using mod union tables.\"\"\"
am: fcc3c6d26a

Change-Id: Id762c75f579d621cb67ca8ea7b7c16024732e23c

Merge "Revert "Revert "Scan immune spaces using mod union tables."""

Merge \"ART: remove gcc cruft from the makefiles\"
am: df638c66d1

Change-Id: I370c6a829207d657b6a67b259773ab28529ec373

Merge "ART: remove gcc cruft from the makefiles"

Merge \"cpp-define-generator: \"make art-update-cpp-defines\" to regen asm_support\"
am: 4130c3360e

Change-Id: I034b4ad64c8298cec6a998cf2d3ddf11c1812d71

Merge "cpp-define-generator: "make art-update-cpp-defines" to regen asm_support"

Merge \"Fix art::ArchTest::FinalizeSetup not being called.\"
am: 6e5e3b2e91

Change-Id: I2791958a7a77d62390805037e789be48fa58e23f

Merge "Fix art::ArchTest::FinalizeSetup not being called."

Fix art::ArchTest::FinalizeSetup not being called.

CommonRuntimeTestImpl::SetUp() is calling FinalizeSetup. Since
CommonRuntimeTestImpl::FinalizeSetup is not virtual (it's virtual
one step higher in class hierarchy in CommonRuntimeTestBase), overridden
version in ArchTest was never used. This is causing ArchTest to
fail with openjdk8u60 version of java.lang.reflect (it's calling
Class#getDeclaredField in clinit, which fails on 32 bit
version because of different pointer sizes [arch_test forces
64-bit ISA])

Bug: 28666126
Test:  make -j 32 test-art-host-gtest
Change-Id: Ief370f5b18ef787575ac0f88ecbe17ebbf037542

Merge \"ART: Fix RegTypeCache for instance field declaring class\"
am: bffecdbb50

Change-Id: I1c4beef69e286097a859d8855296a0c56826dcbb

Merge "ART: Fix RegTypeCache for instance field declaring class"

Put the deletion of profiling info under a GC critical section.

Otherwise the GC could see dangling pointers.

bug:30033802
Change-Id: I839c4f6e537aba98769e99e4d75ff869c9ff13eb

Merge \"Fix for interpreter crash on new instance of class\"
am: 0f290d764d

Change-Id: Ie237c079eee92df36143b32cdb175a4f52719e34

Merge \"Revert \"Revert \"Use try lock to fix class resolution race\"\"\"
am: 0af088d126

Change-Id: If34912dd115472f4fb2a84dd82a0b1d91c488a06

Merge \"Create OatFileAssistant::OatFileInfo inner class.\"
am: 3d733104c9

Change-Id: Iddfadebb0b90ea9ce8376a5a52ec8fc7c2e386f5

Merge "Fix for interpreter crash on new instance of class"

Merge "Revert "Revert "Use try lock to fix class resolution race"""

Revert "Revert "Use try lock to fix class resolution race""

Fix possible deadlock in EnsureResolved caused by interaction with
GC. Since we were sleeping while holding the mutator lock, it could
block thread suspension. This would deadlock if the thread that
had locked h_class is already suspended since we would spin forever
and not make progress.

Bug: 27417671
Bug: 30500547

Test: test-art-host ART_TEST_GC_STRESS=true

This reverts commit 69bf969c055c31a75d17ea92aeee756042678114.

Change-Id: Id8750df065dc3b9ef7dc874f2eb2cc2c58e5d1eb

Merge "Create OatFileAssistant::OatFileInfo inner class."

Fix for interpreter crash on new instance of class

Rationale:
Fuzz testing found divergences between the compiler and interpreter
which turned out to be caused by calling instance on java.lang.Class
(this worked for compiler but crashed interpeter). Since RI does not
allow this construct, solution is to force interpreter in this
unlikely case and throw run time exception like RI. This fixes
two cases found with fuzz testing. CL also includes
fail-before/pass-after test.

Test: 600-verifier-fails

BUG=29758098

Change-Id: Ie80f7758def44e6655d28fec4c10c34ffa0fd60b

Revert "Revert "Scan immune spaces using mod union tables.""

This reverts commit ccf7c8a68972f3b7bfa38af0c6b792f712f83765.

Change-Id: Iee618d9ae78e1dcdea4a7ecbe9253821d4d4d015
Test: N9 boot, art tests, Ritz EAAC.
Bug: 29516968
Bug: 29517059
Bug: 12687968

Merge \"Revert \"Use try lock to fix class resolution race\"\"
am: 7707777f06

Change-Id: I1cd0bea29e785d45fd9b140fc2209826ff316c54

Merge "Revert "Use try lock to fix class resolution race""

Revert "Use try lock to fix class resolution race"

This reverts commit a704eda0078989a73cac111ed309aca50d2e289b.

Bug: 27417671
Bug: 30500547
Change-Id: Ieea05236b9e61c722660cd9497c9d55d13ccd010

Merge \"Add arena stack and priority queue type aliases\"
am: ea159431cd

Change-Id: Ia4db08fe011ee7e1eaadc8f0f221f71f146dbdfb

Merge \"Revert \"Scan immune spaces using mod union tables.\"\"
am: b94ea458e7

Change-Id: I07be24b3ac8a58d041e8bf09857cb1209ab89c2c

Merge "Add arena stack and priority queue type aliases"

Merge "Revert "Scan immune spaces using mod union tables.""

Merge \"ART: remove ART_JIT makefile variable\"
am: 8481f6390a

Change-Id: Id4cf81cfe8547ea03f09a2d1e4d592d58e9725a7

Merge "ART: remove ART_JIT makefile variable"

Revert "Scan immune spaces using mod union tables."

This reverts commit 8256da35feac0ba7589fcd4b692a4b9a936679a4.

Test failing.

Change-Id: I391dcbdba19c6f6313e06075b0a69ea406c7c50e

Add arena stack and priority queue type aliases

These will be used in the graph coloring register allocator.

Test: m test-art-host

Change-Id: I4d3d063ae5a02e697f07fc8212f391b5347270e5

cpp-define-generator: "make art-update-cpp-defines" to regen asm_support

Change-Id: I1a550eda6c77f38e63bb1c8fb943541cbd3f9cdf

Merge \"Pass the right class loader when inlining.\"
am: 4c4d6f42cf

Change-Id: I8568aff8cf2323e43e0d15441827ded3edd417e1

Merge "Pass the right class loader when inlining."

ART: remove gcc cruft from the makefiles

ART only builds with clang now, delete all logic to determine gcc
vs. clang, and move all LOCAL_CLANG_CFLAGS into LOCAL_CFLAGS.

Test: mma, make test-art-host, make test-art-target
Change-Id: I578615233ad3e71af18afb47bb11f3be8417216c

ART: remove ART_JIT makefile variable

The ART_JIT variable isn't used (and was never used?).

Test: mma, make test-art-host, make test-art-target
Change-Id: Ie6463c3c13fab43504f3c60fd5a15f696e2cf481

Create OatFileAssistant::OatFileInfo inner class.

To share code reused for getting info about both the odex and oat files.

Test: oat file assistant tests.
Change-Id: Ifcc48e0ac481899285e39e3b02f3ade180e38728

Merge \"Avoid read barrier for IntArray::GetArrayClass\"
am: 533d684038

Change-Id: I6f2843bd1b2683fa4e120baa8abf80f6a377580c

Merge "Avoid read barrier for IntArray::GetArrayClass"

Avoid read barrier for IntArray::GetArrayClass

Changed the code use Mark instead of read barrier, this showed an
existing lock violation and possible deadlock which was fixed.
Prevent DCHECK failure from the assert.

Bug: 30469265
Test: test-art-host with CC

Change-Id: I275f953f06f6d13262043fc62eb88dca0356465a