x86_64: Turn on 64-bit core registers initialization.

This enables 64-bit core registers initialization for x86_64.
The backend update with 64-bit temp support is in progress.

Change-Id: If7c9a62c1145f81050adda86f2beed427220baa2
Signed-off-by: Chao-ying Fu <chao-ying.fu@intel.com>
Signed-off-by: Dmitry Petrochenko <dmitry.petrochenko@intel.com>
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>
Signed-off-by: Mark Mendell <mark.p.mendell@intel.com>

Merge "Avoid register pool allocations on the heap."

Avoid register pool allocations on the heap.

Create a helper template class ArrayRef and use it instead
of std::vector<> for register pools in target_<arch>.cc to
avoid these heap allocations during program startup.

Change-Id: I4ab0205af9c1d28a239c0a105fcdc60ba800a70a

Merge "ART: Add a test checking TLS offsets and entrypoints"

Merge "Fix reflection_test failures."

Merge "ART: Allocate ostreams on the heap in Oatdump::DumpOatMethod"

ART: Allocate ostreams on the heap in Oatdump::DumpOatMethod

The stack frame size for this method is above our limits. Allocate
ostreams on the heap instead of on the stack.

Bug: 15278350
Change-Id: Ia94a1f7190b7c2ac096e100682e747334f1bf6cc

Merge "Do not force unwind info to debug_frame."

ART: Add a test checking TLS offsets and entrypoints

Add a test (runtime for now, as compile-time checks seem not
possible with current infrastructure) that checks the structure
of the thread tls and entrypoint structs. A failure of the test
indicates a structural change, which hints at a necessary oat
version bump.

Change-Id: I500b5cd940c916ebfc1580e9dd2c287c4d263956

Do not force unwind info to debug_frame.

The .cfi_sections directive was forcing unwind information into the
debug_frame, which is stripped out. This causes any unwinds done by
libunwind/debuggerd through these art functions to terminate prematurely.

Bug: 15350524
Change-Id: Id6abe7c7106132083bc894a77564b652a08566cb

Merge "Fix GetOutVROffset to use correct Method* size."

Fix GetOutVROffset to use correct Method* size.

[Tip 'o the hat to Mark Mendell for noticing this one]

Use StackReference to find Method* size in GetOutVROffset.

Change-Id: Ie1b25eb839c59305cadc5fdc7cbb5f41f1a85686

Merge "Enable -Wframe-larger-than=1728 compilation error"

Enable -Wframe-larger-than=1728 compilation error

Bug: 15278350
Change-Id: I142dc9de0312ac1f8aa55a4bce86b5bb6722535f

Merge "ART: Heap-allocate buffer in space_bitmap when on 64b arch"

ART: Heap-allocate buffer in space_bitmap when on 64b arch

The stack frame size will be above our limit if we stack-allocate
a buffer.

Bug: 15278350
Change-Id: Ie1a31370538410d47c177f0a6e51879a1fc0752c

Merge "art: -Wsystem-header warning"

Merge "Print mutexes held in Thread::Dump."

Print mutexes held in Thread::Dump.

Should help finding and fixing thread suspend timeout issues.

Bug: 15328839
Change-Id: I30a10529cec0716c7571a0318e9f66be54734fd8

Merge "Fix compiler warnings."

Merge "ART: Fix forbidden thread state change in interpreter"

ART: Fix forbidden thread state change in interpreter

While loading a type for assignability, it might happen that it's
not available yet locally and must be resolved. Formerly, we
disallowed a state change to ensure no GC taking place while a new
shadow frame has not been pushed on the stack yet.

As a fix, allow a "shadow frame under construction" in the thread,
which is visited during GC.

Change-Id: I973487a46b0e9e21fd6d49099d713b58f06d3b45

Merge "Use sendfile when copying profile file."

Fix compiler warnings.

Added GetReference, GetHandle to StackHandleScope to prevent the
compiler from optimizing away these loads/stores from inline
functions.

Change-Id: I4db02dd3194665d844292e74e3a7d7c80e730e06

Use sendfile when copying profile file.

This removes the 4k stack allocation and allow to activate
-Wframe-larger-than=1728 compiler opion.

Bug: 15278350
Change-Id: I389ffe06feb3c1c1bf620e20164cca04a0594788

Merge "Add missing preamble in switch-based interpreter"

Merge "AArch64: Fix cmp-long and method with long arguments."

Merge "ART: x86_64 disassembler improvements"

Merge "ART: Fix stub_test inline assembly for ARM64"

Merge "ART: Make LOCAL_CLANG architecture dependent for the target"

ART: Make LOCAL_CLANG architecture dependent for the target

Be selective for which target we compile with Clang. Currently we
only want to compile with Clang for ARM64, which means we need to
be careful about ARM, which is the second architecture for that.

Bug: 15014252

(cherry picked from commit 9689e3768621130b2536564f4e00fcb6b3d25df4)

Change-Id: I312e1caea08f2f3a20304b27f979d3c7b72b0a04

Merge "Fix a bug in SSA construction."

Fix a bug in SSA construction.

If a join block does not have an incoming value for a local
from a predecessor block, we should not create a phi. The
verifier has made sure the local is updated before any
following reads after this block.

Change-Id: Id2785efc73c9fb3224826fff2f4b4ad215905ff4

Merge "Fix a possible file descriptor leakage"

Fix a possible file descriptor leakage

Bug: 15279918
Change-Id: I7909a53f9028d2f445fb97a0a4293f36b3c012dd

AArch64: Fix cmp-long and method with long arguments.

1. Fix cmp-long.
2. Use single register to pass long argument.
3. Flush StackReference<ArtMethod> on arm64 the same as in common code.
3. Fix the mismatch in calculate reg offset.

Change-Id: Ie2723260fb143512e4da6ee88d4f3aded80d3d5e

Add missing preamble in switch-based interpreter

Adds missing preamble for CONST_WIDE_HIGH16 instruction.

Change-Id: I80371d6f5e5d1b60c88837706dc139782776eefa

ART: x86_64 disassembler improvements

This patch
 (a) enables full support of 64bit extended regs r8-r15,
     including 8bit r8l-r15l, 16bit r8w-r15w and also
     32bit r8d-r15d
 (b) fixes an issue with decoding reg from ModRM byte
     (REX.B should be used)
 (c) fixes an issue with decoding regs from SIB byte
     (regs that contain addr are target-specific)

Change-Id: I6bf3d7102780907b1cbe2a46927352ac0b506295
Signed-off-by: Vladimir Kostyukov <vladimir.kostyukov@intel.com>

Merge "ART: Add another proxy test"

Fix reflection_test failures.

This change also allows the Mips emulator to boot in interpreter mode.

Change-Id: Id1d5d65df79f65a747d37ddce6dd79906d1f4fcf
Signed-off-by: Douglas Leung <douglas@mips.com>

ART: Fix stub_test inline assembly for ARM64

The rebased Clang resolves register constraints differently from
before. Change the inline assembly.

Bug: 15393246
Change-Id: Iea2c2693f0a035db541ff91f796ee5f997a2ced7

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>

ART: Add another proxy test

Change-Id: Ie5beda276a40067b7246001720cc5eea2fa7585a

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