Merge "Correct another RegStorage::Solo32"

Merge "ART: Fix an error in proxy_invoke stub on ARM64"

Correct another RegStorage::Solo32

It should be RegStorage::FloatSolo32

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

Merge "Modified functions to return pointers where appropriate"

Merge "AArch64: split list of opcodes in compiler filter."

Merge "Quick compiler: reference cleanup"

AArch64: split list of opcodes in compiler filter.

Some of the opcodes that are currently enabled in the compiler
filter are generating assertion failures when using libartd.so.
In this patch, these opcodes are marked as experimental
via #if ... #endif and are disabled. This allows using libartd.so
to run the tests and for development. Further commits will follow
to fix the assertion issues.

Change-Id: I9493fddc281542cb15ced38b262f37f4ef4f0870

Quick compiler: reference cleanup

For 32-bit targets, object references are 32 bits wide both in
Dalvik virtual registers and in core physical registers.  Because of
this, object references and non-floating point values were both
handled as if they had the same register class (kCoreReg).

However, for 64-bit systems, references are 32 bits in Dalvik vregs, but
64 bits in physical registers.  Although the same underlying physical
core registers will still be used for object reference and non-float
values, different register class views will be used to represent them.
For example, an object reference in arm64 might be held in x3 at some
point, while the same underlying physical register, w3, would be used
to hold a 32-bit int.

This CL breaks apart the handling of object reference and non-float values
to allow the proper register class (or register view) to be used.  A
new register class, kRefReg, is introduced which will map to a 32-bit
core register on 32-bit targets, and 64-bit core registers on 64-bit
targets.  From this point on, object references should be allocated
registers in the kRefReg class rather than kCoreReg.

Change-Id: I6166827daa8a0ea3af326940d56a6a14874f5810

Merge "x86_64: Fix art_quick_aput_obj"

Merge "Don't use Dump for printing a bitset in liveness_test."

Don't use Dump for printing a bitset in liveness_test.

Change-Id: Ib33e975cee838da97a3bebe04176b5ae8b62f9d5

x86_64: Fix art_quick_aput_obj

The ebx register is not scratch for x86_64 and it leads to
its corruption (seen on art/test 201). Replacing ebx by ecx.

Change-Id: I7f5eeba47688ada5afba82a9303fa736f823d77e
Signed-off-by: Dmitry Petrochenko <dmitry.petrochenko@intel.com>
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>

Merge "ART: BitVector not calculating number_of_bits correctly"

ART: Fix an error in proxy_invoke stub on ARM64

Make sure that d0 is filled with the right value, as the return
from C code is always in x0.

Bug: 15386201
Change-Id: I6c048b3d92272d769f1de3bdc9ce30733b9e1935

Modified functions to return pointers where appropriate

GetStringChars will return a pointer to the underlying char array iff
it is not movable. Otherwise, it will return a copy of the char array.
For consistency, the null terminating character has been removed as the
specification for a jchar strings are not null terminated:

http://developer.android.com/training/articles/perf-jni.html

GetStringCritical will now always return a pointer to the char array.
The char array is pinned and moving gc is disabled until the pointer is
released.

Change-Id: I19c8cbaecc1f3f723d80acec074fb8c5e2d489c3

am 0f821c00: Merge "Make class status volatile."

* commit '0f821c00aeca7b6fca789e5613a91aef786b45af':
  Make class status volatile.

am 7a33c4b8: Merge "Fix interpreter entrypoint initialization in the image"

* commit '7a33c4b8260e2fafc9ca94e51334d4f18c8e4324':
  Fix interpreter entrypoint initialization in the image

Merge "Make class status volatile."

Merge "Fix interpreter entrypoint initialization in the image"

Fix interpreter entrypoint initialization in the image

Sets the interpreter_to_interpreter_bridge entrypoint if the method is not
compiled. This avoids the slow path through Quick compiler's world.i

Bumps oat version to force recompilation of image.

Bug: 15098601
Change-Id: I08f7dfce26ecc5ca88bdf21187b2909c0c069596

Make class status volatile.

Discourage loads and stores from reordering around the status being updated.
Bug: 15347354

Change-Id: Ice805cb834617747c8209e98a142d3e5c7585719

ART: BitVector not calculating number_of_bits correctly

The number_of_bits_ field has an unclear intent. Instead, using
  storage_size_ * kWordBits when relevant.

Change-Id: I8c13be0d6643de37813fb154296d451f22c298c8
Signed-off-by: Jean Christophe Beyler <jean.christophe.beyler@intel.com>

am 502a1975: Merge "Switch on implicit checks and bump oat version to force recompilation"

* commit '502a1975e146b288356e49a3af82629637da5ece':
  Switch on implicit checks and bump oat version to force recompilation

Merge "Switch on implicit checks and bump oat version to force recompilation"

am a3209fae: Merge "Added workaround passing empty arg array for proxy invocation."

* commit 'a3209fae94bf7db935b235cc97a4852721accd3e':
  Added workaround passing empty arg array for proxy invocation.

Merge "Added workaround passing empty arg array for proxy invocation."

Added workaround passing empty arg array for proxy invocation.

This recreates old Dalvik behavior for older target sdk versions,
but will still pass null for newer ones.

Bug: 13247236

(cherry picked from commit 01d5a146e20660bd06f026c16f19ec080f8fdd7b)

Change-Id: I911889cf559ad8d9f37ea9be3929387c86446851

am f6b5378b: Merge "ART: Fix comments and link before MIR to next in Remove method"

* commit 'f6b5378ba182c5c26c556ae34638afc38adef519':
  ART: Fix comments and link before MIR to next in Remove method

Merge "ART: Fix comments and link before MIR to next in Remove method"

am 042cbb60: Merge "x86: art_quick_instrumentation_entry should not use offset address"

* commit '042cbb60dc25c16806b02d41dfad040861f727ad':
  x86: art_quick_instrumentation_entry should not use offset address

am 8928d477: Merge "Fix GC race condition in visit roots"

* commit '8928d4779c5cb4844e213cc483777c745d0749be':
  Fix GC race condition in visit roots

Merge "x86: art_quick_instrumentation_entry should not use offset address"

Merge "Fix GC race condition in visit roots"

ART: Fix comments and link before MIR to next in Remove method

There are some comments without a period at the end and the remove does not
link the MIR before the one we want to remove to the one after.

Change-Id: Ia1263d3191e39b4f8f76ef62b28f7fbb1c6c0403
Signed-off-by: Jean Christophe Beyler jean.christophe.beyler@intel.com

x86: art_quick_instrumentation_entry should not use offset address

art_quick_instrumentation_entry uses offset of the address
instead of full adress but this address is used in ret instruction
which does not expect it.

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

Fix GC race condition in visit roots

While GC visits roots other code can change the set pending exception
due to lack of synchronization.

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

am e4283be9: Merge "Quick compiler: fix array overrun."

* commit 'e4283be97047a26d3476acd3863dcc386498be17':
  Quick compiler: fix array overrun.

Merge "Quick compiler: fix array overrun."

Quick compiler: fix array overrun.

MIRGraph::InlineCalls() was using the MIR opcode to recover
Dalvik instruction flags - something that is only valid for
Dalvik opcodes and not the set of extended MIR opcodes.

This is probably the 3rd or 4th time we've had a bug using
the MIR opcode in situations that are only valid for the Dalvik
opcode subset.  I took the opportunity to scan the code for
other cases of this (didn't find any), and did some cleanup while
I was in the neighborhood.

We should probably rework the DalvikOpcode/MirOpcode model whenver we
get around to removing DalvikInstruction from MIR.

Internal bug b/15352667: out-of-bound access in mir_optimization.cc

Change-Id: I75f06780468880892151e3cdd313e14bfbbaa489

am fbc3e0ba: Merge "ART: Add thread safety test for LargeObjectSpace"

* commit 'fbc3e0baa16f265a5dbc99a38383b4dbc41b04a8':
  ART: Add thread safety test for LargeObjectSpace

Merge "ART: Add thread safety test for LargeObjectSpace"

ART: Add thread safety test for LargeObjectSpace

Add a test that just calls Alloc and Free repeatedly with multiple
threads, checking whether things are really race-free.

Change-Id: I39ea9e7449e3354b49c3aa11c27fcfeb07acff9f

am b20c373e: Merge "AArch64: Enable extended MIR"

* commit 'b20c373e45200102eaf2d4aac35d366067e0074b':
  AArch64: Enable extended MIR

Merge "AArch64: Enable extended MIR"

am b379e8d1: Merge "Compaction cleanup for FindArrayClass."

* commit 'b379e8d107b9131d9b6c7b48aea01f9866e4047b':
  Compaction cleanup for FindArrayClass.

Merge "Compaction cleanup for FindArrayClass."

Compaction cleanup for FindArrayClass.

We now pass double pointer in to signify that it can cause thread
suspension, this double pointer gets wrapped by a handle if don't
find the array class in the cache.

Change-Id: I43436b6c35597c7252b65d1180baddb5ac4caabb

am e4e04bbd: Merge "Support disassembly of 16-bit immediates"

* commit 'e4e04bbd86d1131b9f380a52a46b1a3b83335d5f':
  Support disassembly of 16-bit immediates

am afa8e909: Merge "Fix race condition in MemMap::MapAnonymous."

* commit 'afa8e909b7fbf53cb45573c458c031d3174069c9':
  Fix race condition in MemMap::MapAnonymous.

Merge "Support disassembly of 16-bit immediates"

Merge "Fix race condition in MemMap::MapAnonymous."

Fix race condition in MemMap::MapAnonymous.

Previously we were using MAP_FIXED which introduced a serious race
condition if MAP_32BIT was set since it would possibly overwrite
an existing map at the address which we determined was free with
msync. There was a window of time after we had msynced a page where
another thread could map something at that page. The new method
avoids using MAP_FIXED and unmaps allocations which succeed but
aren't in the low 4GB when MAP_32BIT is set.

Bug: 15338094
Bug: 14974497
Change-Id: I57f00baf4143e9fa17fb1d4c2be04b30705a2bfd

Support disassembly of 16-bit immediates

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

am b413cd79: Merge "Don\'t report down-calls as unhandled exceptions."

* commit 'b413cd79c46b7c48ac763cb8152a55a4ed60fe9f':
  Don't report down-calls as unhandled exceptions.

Merge "Don't report down-calls as unhandled exceptions."

am 22460ecb: Merge "Quick compiler: Method* as a reference"

* commit '22460ecbaae7e489a3bc17a2adb2e7e9d0028f9c':
  Quick compiler: Method* as a reference

Merge "Quick compiler: Method* as a reference"

Don't report down-calls as unhandled exceptions.

Bug: 15310540
Also, narrow scope of catch/deoptimize stack visitors that are specific to
quick exception delivery.

Change-Id: Ib13a006ce1347acb93a36b0186550d4c3ec2034b

Switch on implicit checks and bump oat version to force recompilation

This switches on the compiler's implicit null pointer and stack
overflow checks.  These use signals

Change-Id: I7b536784bbca5f07e30379881a1eea6ab4c258ac

am c24234e9: Merge "Another workaround for stack overflow issue with clang"

* commit 'c24234e9d6fea3d6d5451aa4fa14ca0c6e63f8bd':
  Another workaround for stack overflow issue with clang

Merge "Another workaround for stack overflow issue with clang"

am 3326027d: Merge "Quick compiler: x86_64 workaround"

* commit '3326027d2cc52463db3786a860cf238e8feb349e':
  Quick compiler: x86_64 workaround

Merge "Quick compiler: x86_64 workaround"

am b7f02280: Merge "ART: Use StackReference in Quick Stack Frame"

* commit 'b7f02280f7f56ae94fe7f01e161be0b725b6e4a9':
  ART: Use StackReference in Quick Stack Frame

am ac70051a: Merge "Revert "Revert "Clean up image writer patching."""

* commit 'ac70051a6f97025b8a93390b2b5a62e491fc77ca':
  Revert "Revert "Clean up image writer patching.""

am 72fc6356: Merge "Revert "Revert "Add support for relative patching to quick offsets."""

* commit '72fc63563661b4b954acd5962ec3ae30b7666569':
  Revert "Revert "Add support for relative patching to quick offsets.""

Merge "ART: Use StackReference in Quick Stack Frame"

Merge "Revert "Revert "Clean up image writer patching."""

Merge "Revert "Revert "Add support for relative patching to quick offsets."""

Revert "Revert "Clean up image writer patching.""

This reverts commit fb241b6e93f8b54122b978ad0df23a9e514eef0f.

Change-Id: I99da82fd549bcdc3c2fdfe496147897335c50b8e

Revert "Revert "Add support for relative patching to quick offsets.""

This reverts commit 3e2e99484043aa25b8ed41509ac1b3ccd7d6a959.

Change-Id: Ifbe604330cd3a1dd0a8afa7c500868e2fd0fc500

am 6a7f45ee: Merge "Revert "Add support for relative patching to quick offsets.""

* commit '6a7f45ee5401d3d0a5af8ce68bc3b22ae6d85555':
  Revert "Add support for relative patching to quick offsets."

am 31273d0c: Merge "Revert "Clean up image writer patching.""

* commit '31273d0c1e3c29498cc2360584b8935bf35f4376':
  Revert "Clean up image writer patching."

am 5e5af81e: Merge "Change GetFreeMemory to use the growth limit."

* commit '5e5af81e97b602c7ccb71b6145a9637c944954a5':
  Change GetFreeMemory to use the growth limit.

Merge "Revert "Add support for relative patching to quick offsets.""

Revert "Add support for relative patching to quick offsets."

This reverts commit 57d27334bcb9ea95c969be70d8d93eb1a9c72b41.

Change-Id: I5df09ee342c337291388a48e4975a6bf5c1f9944

Merge "Revert "Clean up image writer patching.""

Revert "Clean up image writer patching."

This reverts commit fc3b871ca37593561e3b67006771710bf4d174ea.

Change-Id: I4d1250f3adcd17fcd1b63496816e098db4d5be3c

Merge "Change GetFreeMemory to use the growth limit."

am d87ca14e: Merge "Fix build."

* commit 'd87ca14e850d6cc029419fef41f9212907bda2bf':
  Fix build.

Merge "Fix build."

Fix build.

Lint error due to a wrong #include guard.

Change-Id: I3b5d56c412580ec14f5c09b859a1bd6f9eb667ee

am b827c44f: Merge "ART: Setting up cleanup"

* commit 'b827c44f39f66fce600408f82988d6e154627ba7':
  ART: Setting up cleanup

Merge "ART: Setting up cleanup"

AArch64: Enable extended MIR

This patch enables all the extended MIR opcodes for ARM64. Please note
that currently the compiler will never generate these opcodes since the
BB optimisations are not enabled.

Change-Id: Ia712b071f62301db868297d37567795128b5bf2e
Signed-off-by: Serban Constantinescu <serban.constantinescu@arm.com>

ART: Setting up cleanup

- Moved code around to actually have the clean-up code in a PassDriver format.
    This allows us to better control what is being called after an optimization
    It also allows the use of a centralized pass system for both optimizations
      and cleanup.

Change-Id: I9d21e9bb9ee663739722f440d82adf04f73e380c
Signed-off-by: Jean Christophe Beyler <jean.christophe.beyler@intel.com>
Signed-off-by: Razvan A Lupusoru <razvan.a.lupusoru@intel.com>
Signed-off-by: Yixin Shou <yixin.shou@intel.com>
Signed-off-by: Chao-ying Fu <chao-ying.fu@intel.com>
Signed-off-by: Udayan Banerji <udayan.banerji@intel.com>

ART: Use StackReference in Quick Stack Frame

The method reference at the bottom of a quick frame is a stack
reference and not a native pointer. This is important for 64b
architectures, where the notions do not coincide.

Change key methods to have StackReference<mirror::ArtMethod>*
parameter instead of mirror::ArtMethod**. Make changes to
invoke stubs for 64b archs, change the frame setup for JNI code
(both generic JNI and compilers), tie up loose ends.

Tested on x86 and x86-64 with host tests. On x86-64, tests succeed
with jni compiler activated. x86-64 QCG was not tested.

Tested on ARM32 with device tests.

Fix ARM64 not saving x19 (used for wSUSPEND) on upcalls.

Tested on ARM64 in interpreter-only + generic-jni mode.

Fix ARM64 JNI Compiler to work with the CL.

Tested on ARM64 in interpreter-only + jni compiler.

Change-Id: I77931a0cbadd04d163b3eb8d6f6a6f8740578f13

am 29b53d3d: Merge "Moved a BufferedOutputStream onto the heap to reduce stack usage."

* commit '29b53d3d715b1ec19349e8cbf7c5e4ff529bd5fe':
  Moved a BufferedOutputStream onto the heap to reduce stack usage.

Merge "Moved a BufferedOutputStream onto the heap to reduce stack usage."

Moved a BufferedOutputStream onto the heap to reduce stack usage.

Bug: 15278350
Change-Id: I6aa7cf3979453e555686610e90520b762b812f8c

Change GetFreeMemory to use the growth limit.

This makes more sense since it is what the allocator uses. Also fixed
somewhere where we were not properly passing in whether or not it was
a large object allocation.

Bug: 15327879
Change-Id: Ieab7af5427f5cdc2760390186b67e2c96d4bafa7

am e4707ae7: Merge "Ignore catch blocks whose type can\'t be resolved."

* commit 'e4707ae7985a4f855b97f3b20d99fb55368f44ad':
  Ignore catch blocks whose type can't be resolved.

Merge "Ignore catch blocks whose type can't be resolved."

Ignore catch blocks whose type can't be resolved.

Reverts change 72b3e430d880ef57eaa6a34a0822165994052202 but keeps unit test and
missing delete that would fail assertions on long jump context recycling.

Change-Id: I926755e8b831b208aa7e1ce46421bef3793a1441

am 2b2c0cf9: Merge "Clean up image writer patching."

* commit '2b2c0cf947a4b228c9f378a3b4f281eaa2bff19a':
  Clean up image writer patching.

Merge "Clean up image writer patching."

Quick compiler: x86_64 workaround

A recent CL changed the allocation of temp registers destined to
hold a reference such that on 64-bit systems a 64-bit register
would be allocated.  However, for bring-up purposes, the x86_64
backend wants the ability to continue holding long values in a
register pair.

Change-Id: I25d9f755fafbe96144226677f85c6d5cad1ffa76

Clean up image writer patching.

Also fixed an issue related to incorrect patching of non relative
patches.

Change-Id: I55b0b3814b65b7fa8749a934dfaaf07b97735304

am 31e7fcb9: Merge "Add support for relative patching to quick offsets."

* commit '31e7fcb904f03a504f082d25814ac4644b5073e4':
  Add support for relative patching to quick offsets.

Merge "Add support for relative patching to quick offsets."