Merge "Make arm64 build with ART_TARGET_CLANG."

Merge "Add additional flags to make ART_TARGET_CLANG function with 3.5."

Merge "Assert dex-cache is populated following resolution in debug builds."

Merge "Add finalizer references from the entrypoints."

Merge "Replace the bool kDoReadBarrier template parameter with an enum."

Merge "Add GC cause to WaitForGcToComplete message."

Add GC cause to WaitForGcToComplete message.

Change-Id: I8fe107d90a84de065c407b8d29fd106267ac440d

Merge "ART: Move oat_data_flow_attributes_ to private and put an API"

Assert dex-cache is populated following resolution in debug builds.

Change-Id: I71bf6677935bbf6963c0d368fec3a28a454edbc6

Make arm64 build with ART_TARGET_CLANG.

Change-Id: If47ba8afbce3bbe9696cc7d6d08502e07aba5e7d

ART: Move oat_data_flow_attributes_ to private and put an API

The oat_data_flow_attributes had no checking mechanism to ensure bound
 correctness.

This fix handles this and also offers two functions to retrieve the
 attributes: using the MIR and DecodedInstruction.

Change-Id: Ib4f1f749efb923a803d364a4eea83a174527a644
Signed-Off-By: Jean Christophe Beyler <jean.christophe.beyler@intel.com>

Merge "Implement object lock and unlock entrypoints for x86-64"

Merge "Replace CountOneBits and __builtin_popcount with POPCOUNT."

Replace CountOneBits and __builtin_popcount with POPCOUNT.

Clean up utils.h, make some functions constexpr.

Change-Id: I2399100280cbce81c3c4f5765f0680c1ddcb5883

Merge "Don't leak oat file when we fail to open a dex file."

Implement object lock and unlock entrypoints for x86-64

This patch adds implementation for art_quick_lock_object and
art_quick_unlock_object stubs for x86-64 and enables their
testing in stub_test.

Change-Id: Ia373c9b0ebc7ebb959968464cf55607afd5384b0
Signed-off-by: Alexei Zavjalov <alexei.zavjalov@intel.com>

Merge "Revert "Revert "64bit changes to the stack walker for the Quick ABI."""

Revert "Revert "64bit changes to the stack walker for the Quick ABI.""

This reverts commit 8d5ead52a92675c258113d3dfa71bf8fceba5d9f.

Change-Id: I6b4774a9bd180de33551975e106322ba1192e6ab

Replace the bool kDoReadBarrier template parameter with an enum.

Fix one kDoReadBarrier/kIsVolatile mixup in an Object::GetFieldObject
call.

Bug: 12687968

Change-Id: I896b1137b21a20c0504abd2bf3fe6f83805f3300

Add finalizer references from the entrypoints.

We now have an invariant where we never allocate finalizable
objects with the Initialized or Resolved entrypoints. This speeds up
allocation by only doing the check in the slow path.

Before:
MemAllocTest: 3625, 3707, 3641
EvaluateAndApplyChanges: 3448, 3421, 3413

After:
MemAllocTest: 3164, 3109, 3135
EvaluateAndApplyChanges: 3272, 3299, 3353

Bug: 14078487

Change-Id: I2b0534af3e7c75ea5e5257cf3647744f7abfb74e

Merge "Implement art_quick_string_compareto entrypoint for x86-64"

Implement art_quick_string_compareto entrypoint for x86-64

This adds implementation of the art_quick_string_compareto
entrypoint for the x86-64 platform.

Add a test to stub_test, enabled for arm, x86 and x86-64.

Change-Id: I71b318b03d4c8920ccb3723b59c43542e219bf47
Signed-off-by: Alexei Zavjalov <alexei.zavjalov@intel.com>

Merge "Disable the ahead-of-time compilation for MIPS."

Merge "Fix the tools/art script about the ISA-specific dalvik cache."

Fix the tools/art script about the ISA-specific dalvik cache.

Bug: 14470552
Change-Id: I71367052d2a5f0dcf66cf2dfdbc260aff4ac952e

Merge "ART: aarch64 jni compiler needs to extend small return types"

ART: aarch64 jni compiler needs to extend small return types

As aarch64 calling convention does not mandate extension on return
values anymore and leaves the upper bits undefined, the jni compiler
needs to sign- or zero-extend the returned values when necessary.

As three architectures need extension now, refactor this fact into a
flag into a virtual method.

Add tests to JniTest that exercise the required extension.

Change-Id: Idebb7c4dedebb852e58ade63e1c2b1eeced23104

Merge "Update clean-oat targets to handle ISA directories"

Update clean-oat targets to handle ISA directories

Change-Id: Ia2f04058deff7b31ac458064723b05a6eb0e7f8d

Disable the ahead-of-time compilation for MIPS.

Bug: 14464004

Change-Id: I551c0dc4122853a1a9ab566d03bac74cc057cba5

Merge "Add thread unsafe allocation methods to spaces."

Merge "Clean up Add/Remove space."

Merge "ART: Fix target run-tests to build architecture directories"

ART: Fix target run-tests to build architecture directories

The recent dalvik-cache layout changes require the tests to create
the right directories, as ART does not auto-create outside of /tmp

(cherry picked from commit 335438a512d016bb0e582304c0a6a1ad510bcc47)

Change-Id: Ie9045cc34be8b2f106fdec4f3bf37ad830637756

Merge "Fix a few 64-bit compilation of 32-bit code issues."

Fix a few 64-bit compilation of 32-bit code issues.

Bug: 13423943

Change-Id: I939389413af0a68c0d95b23cd598b7c42afa4383

Don't leak oat file when we fail to open a dex file.

Change-Id: I929b34d240d2052d1eea8344b6fa79646920a4c1

Merge "Fix errno checking for dalvik-cache creation."

Merge "Use instruction specific dalvik cache dirs."

Fix errno checking for dalvik-cache creation.

On the host, it's possible for /tmp/foo/dalvik-cache/<isa> to
not exist, but for /tmp/foo/dalvik-cache to be already present.
In this scenario, the first mkdir() can fail with EEXIST.

(cherry picked from commit 6fa28aef44106eef4a7195adb5eb77f6e5531572)

Change-Id: I1da79ad23a4ff510a0f87eca4b52d9f52ab791ad

Use instruction specific dalvik cache dirs.

- All oat & art files are now placed under /data/dalvik-cache/<isa>/.
- GetDalvikCacheOrDie now requires a mandatory subdirectory argument,
  and is implicitly rooted under /data/.
- Added helper methods to convert InstructionSet enums into strings
  and vice versa.

(cherry picked from commit 2974bc3d8a5d161d449dd66826d668d87bdc3cbe)

Change-Id: Ic7986938e6a7091a2af675ebafec768f7b5fb8cd

Merge "Fix host build with clang 3.3"

Fix host build with clang 3.3

(cherry picked from commit d31ce7d053045f4c8509ae6ab61819287010ad92)

Change-Id: If4133d72a61b23347cb2da549308cf6fb0280850

Merge "x86_64 Assembler Test Infrastructure, fix x86_64 assembler"

x86_64 Assembler Test Infrastructure, fix x86_64 assembler

Some infrastructure to do real assembler testing. Need to extend to
other assemblers, and a lot more tests.

Fix some of the cases of the x86_64 assembler.

Change-Id: I15b5f3a094af469130db68a95a66602cf30d8fc4

Merge "Add heap reference size to the checks in arch_test"

Merge "Implement art_quick_aput_object stubs for X86-64 and ARM64"

Add heap reference size to the checks in arch_test

As we will potentially make heap references larger than 32b on 64b
systems at some point, add a constant to the .h files that describes
what the current assembly implementation assumes about the size of
heap references, and check whether that agrees with sizeof.

Change-Id: I75ffd7e2b59891ec36977fd831a8d5faac69c76d

Implement art_quick_aput_object stubs for X86-64 and ARM64

Implement the aput_object stubs for 64b architectures and enable
their testing in stub_test.

Fix missing @PLT for x86.

Add automatic _local labels in function definitions in x86-64 so we
can make local jumps (instead of PLT hoops).

Change-Id: I614b88fd5966acd8a564b87c47d4c50ee605320c

Clean up Add/Remove space.

Deleted the set_as_default parameter and added a new function
SetSpaceAsDefault instead.

Change-Id: Ic4c359854d08e64ac0d0df92f0105447adb9df36

Add thread unsafe allocation methods to spaces.

Used by SS/GSS collectors since these run with mutators suspended and
only allocate from a single thread. Added AllocThreadUnsafe to
BumpPointerSpace and RosAllocSpace. Added AllocThreadUnsafe which uses
current runs as thread local runs for a thread unsafe allocation.
Added code to revoke current runs which are the same idx as thread
local runs.

Changed:
The number of thread local runs in each thread is now the the number
of thread local runs in RosAlloc instead of the number of size
brackets.

Total GC time / time on EvaluateAndApplyChanges.
TLAB SS:
Before: 36.7s / 7254
After: 16.1s / 4837

TLAB GSS:
Before: 6.9s / 3973
After: 5.7s / 3778

Bug: 8981901

Change-Id: Id1d264ade3799f431bf7ebbdcca6146aefbeb632

Merge "Don't allow allocating finalizable objects during transactions."

Merge "Simplify GenConstString"

Add additional flags to make ART_TARGET_CLANG function with 3.5.

Change-Id: I5fbf29e5d08d905f7585fb6d0de9472e3f3ed218

Don't allow allocating finalizable objects during transactions.

It doesn't make sense to allocate finalizable objects during a
transcation since they will never get finalized without a started
runtime.

Before StatusInitialized in core.host.oatdump.txt: 3564
After StatusInitialized in core.host.oatdump.txt: 3564

Bug: 14078487

Change-Id: I7070536f7bb87bfc691d4268bd39a3eca492f48e

Simplify GenConstString

Now it's the same code for all platforms.

Bug: 13506069
Change-Id: I42c08a9dc99a3079caad01602de84296c9357dd8

Merge "Force inlining on trivial accessors."

Merge "Rewrite suspend test check with LIRSlowPath."

Force inlining on trivial accessors.

Make volatility for GetFieldObject a template parameter.
Move some trivial mirror::String routines to a -inl.h.

Bug: 14285442

Change-Id: Ie23b11d4f18cb15a62c3bbb42837a8aaf6b68f92

Rewrite suspend test check with LIRSlowPath.

Change-Id: I2dc17d079655586bfc588349c7a04afc2c6879af

Merge "Enable concurrent sweeping for non-concurrent GC."

Enable concurrent sweeping for non-concurrent GC.

Refactored the GarbageCollector to let all of the phases be run by
the collector's RunPhases virtual method. This lets the GC decide
which phases should be concurrent and reduces how much baked in GC
logic resides in GarbageCollector.

Enabled concurrent sweeping in the semi space and non concurrent
mark sweep GCs. Changed the semi-space collector to have a swap semi
spaces boolean which can be changed with a setter.

Fixed tests to pass with GSS collector, there was an error related to
the large object space limit.

Before (EvaluateAndApplyChanges):
GSS paused GC time 7.81s/7.81s, score: 3920

After (EvaluateAndApplyChanges):
GSS paused GC time 6.94s/7.71s, score: 3900

Benchmark score doesn't go up since the GC happens in the allocating
thread. There is a slight reduction in pause times experienced by
other threads (0.8s total).

Added options for pre sweeping GC heap verification and pre sweeping
rosalloc verification.

Bug: 14226004
Bug: 14250892
Bug: 14386356

Change-Id: Ib557d0590c1ed82a639d0f0281ba67cf8cae938c

Merge "Disable StubTest.APutObj if heap poisoning is enabled."

Merge "Add ARM64 & X86_64 Assembly, plus tests"

Add ARM64 & X86_64 Assembly, plus tests

This adds assembly code or removes UNTESTED annotation from
TWO_ARG_DOWNCALLand THREE_ARG_DOWNCALL macros and supporting code,
generating working allocation stubs.

Some object and array allocation tests are added to the stub_test.

Change-Id: I5e93b7543c1e6dbd33b0d4cf564c7cbd963e74ef

Merge "ART: Initialization of RegLocation and API for GrowableArray Iterator"

ART: Initialization of RegLocation and API for GrowableArray Iterator

Two things:

- Added a default initialization for the RegLocation.
- Added a default constructor and Reset for the GrowableArray's Iterator class.

Change-Id: I74d9c584304c77add42e0d66e4037ac45b890142
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>

Merge "Quick compiler: debugging assists"

Quick compiler: debugging assists

A few minor assists to ease A/B debugging in the Quick
compiler:
   1.  To save time, the assemblers for some targets only
update the object code offsets on instructions involved with
pc-relative fixups.  We add code to fix up all offsets when
doing a verbose codegen listing.
   2.  Temp registers are normally allocated in a round-robin
fashion.  When disabling liveness tracking, we now reset the
round-robin pool to 0 on each instruction boundary.  This makes
it easier to spot real codegen differences.
   3.  Self-register copies were previously emitted, but
marked as nops.  Minor change to avoid generating them in the
first place and reduce clutter.

Change-Id: I7954bba3b9f16ee690d663be510eac7034c93723

Merge "Setup policies for register allocation."

Setup policies for register allocation.

Change-Id: I857e77530fca3e2fb872fc142a916af1b48400dc

Merge "Fix DDM recent allocations"

Fix DDM recent allocations

A class may not have source file in which case ClassHelper::GetSourceFile
returns NULL. Adding the GetMethodSourceFile function helps us preventing
from dealing with null strings.

Bug: 14300208
Change-Id: I28707f883bacec4ee367ff703328d0f0240855f9

Merge "Visit deoptimized shadow frames as roots"

Visit deoptimized shadow frames as roots

During deoptimization, we create shadow frames but do not attach them to the
stack until we transition to interpreter mode. If a GC happens before that,
these shadow frames are not visited by GC so they may contain stale references.

This CL addresses this issue by visiting the shadow frames "under contruction"
in Thread::VisitRoots so we correctly update all references they hold.

To make them visible, we now save the top shadow frame (the first one created)
in the field Thread::tls_ptr_sized_values::deoptimization_shadow_frame. This
field will then be cleared when transitioning to interpreter mode.

Bug: 14324885
Change-Id: Ib213ddc80f19cfcdfcec6f20acaa7f1a0e9ce2c1

Disable StubTest.APutObj if heap poisoning is enabled.

The StubTest.APutObj test currently fails (crashes) as it runs code
that does not support heap poisoning yet. Disable it until such
support is added.

Bug: 12687968
Change-Id: I70d1f7f300e70ffae970ebeca9a29849aa7dffc6

Merge "Improve GSS reference processing."

Merge "Track package change."

Merge "Remove unreachable JNI workaround code."

Merge "Always log explicit GC."

Improve GSS reference processing.

Support the case where the reference object is in the free list space
and the referent object is in the bump pointer space at a bump pointer
space collection.

Bug: 11650816
Change-Id: If98b08edc9e37351c74ee07cb3f2d30c2b4d0056

Always log explicit GC.

People who use DDMS want to see that a GC actually occurs when they
press GC button.

Bug: 14325353
Change-Id: I44e0450c92abf7223d33552ed37f626fe63e1c28

Track package change.

art could keep using the "private" package, but it doesn't _need_ to
so it makes sense to switch it over to the [future] public package.

Change-Id: I715e0079fe64886ca9e4a9399bb525a7f8fadab0

Remove unreachable JNI workaround code.

Change-Id: I8d737ced8b553ed80a1fd72560e62abd78ff8b12

Merge "Fix racy DCHECKS."

Fix racy DCHECKS.

Added a size bracket lock to fix a race condition where we were
inserting into full_runs_ and non_full_runs_ holding a lock but
reading without holding a lock from a different thread.

Bug: 14326370
Change-Id: I5c492bddc4b9927e4a36603f3d787b046961675d

Merge "Avoid volatile 64-bit tearing on 32-bit architectures."

Avoid volatile 64-bit tearing on 32-bit architectures.

Change b122a4bbed34ab22b4c1541ee25e5cf22f12a926 removed inline assembly for
volatile 64bit read/writes. This isn't sound in the general case, reinstate.
Motivating change: https://android-review.googlesource.com/91250
Add optimizations for ARM in the case of LPAE support.

Change-Id: Ie86d8885d27c8f0da75f0c3bd50d4553a331282f

Merge "Add a Transform to SSA phase to the optimizing compiler."

Add a Transform to SSA phase to the optimizing compiler.

Change-Id: Ia9700756a0396d797a00b529896487d52c989329

Merge "Quick compiler: fix DCHECKS"

Merge "ART: Bitvector extensions for dumping and size handling"

Merge "Move mapping table and vmap table offsets to OatMethodHeader."

Merge "Fix typo of --oat-fd description in help"

Quick compiler: fix DCHECKS

The recent change to introduce k32, k64 and kReference operand
sizes missed updating a few DCHECKS.

Change-Id: I66eb617b07766e781b38962dc862fc5b023c2fbd

Fix typo of --oat-fd description in help

--oat-fd description in help was copied from --oat-file description.

Change-Id: Idbf36018af58db169f6d65382a7602ec950cce99
Signed-off-by: Wonil Kim <wonil.kim@gmail.com>

Merge "Rosalloc: fix signature for 64b"

Rosalloc: fix signature for 64b

Change-Id: I56df1df0118a0aa1584e51fb574902f7eb88cd80

Merge "Add "arch_test" gtest for assembly stub constants, add some ARM64 assembly code"

Add "arch_test" gtest for assembly stub constants, add some ARM64 assembly code

Add a test that (1) checks all callee-save method frame sizes for
all architectures, (2) checks thread offsets for the runtime
architecture and (3) checks callee-save method offsets for the
runtime architecture.

The "asm_support_XXX.h" files now only contain definitions that are
common between all architectures. Architecture-specific definitions
(i.e., special registers names) have been pushed into the corresponding
.S file. This change was required to be able to undefine definitions
in the test, so that multiple tests can be written in one file.

Test (1) above is in a sense two-stage. The arch_test gtest compares
constants (if it finds them) against the frame size as reported by
the ArtMethods created by the Runtime. This works for all architectures
as we can provide the instruction-set to CreateCalleeSaveMethod. The
second stage of the "test" are preprocessor tests with "#error" in the
case that the constants are not the expected value.

Optimally I'd like to change that to an actual runtime test exercising
the assembly code, which would also allow to check whether the right
registers are stored.

Also added missing assembly code for ARM64 for the callee-save macros.

Also fix X86_64 compilation for Clang 3.5.

Change-Id: I018e6433dffd3d31ba3bfcd75661653f4c7b6552