am b9d11f7e: Merge "x86_64: Fix wide argument increment"

* commit 'b9d11f7e2f60bdebfd2ece34e01ecc789921837d':
  x86_64: Fix wide argument increment

am c6f0f60b: Merge "x86_64: int-to-long should ensure that int in kCoreReg"

* commit 'c6f0f60bf3d4e3402c6e30bb4a4b88b345530954':
  x86_64: int-to-long should ensure that int in kCoreReg

am 10fdc6af: Merge "Fix various test dependencies."

* commit '10fdc6af15a8464fd10bd558a19f6395cf12345a':
  Fix various test dependencies.

Merge "Fix various test dependencies."

Fix various test dependencies.

Change-Id: I2e4761f4cc061f74319a7b64ebe67c553d5297df

Merge "x86_64: Fix wide argument increment"

Merge "x86_64: int-to-long should ensure that int in kCoreReg"

x86_64: int-to-long should ensure that int in kCoreReg

it is possible that int in xmm so implementation of int-to-long
should ensure that src in core reg before usage of move with
sign extension which does not support xmm case.

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

am d377971c: Merge "Multilib ART host."

* commit 'd377971c505e8cfacc878b8c632cc40fcf7d88c5':
  Multilib ART host.

am ba778fae: Merge "AArch64: Add memcmp.S for Arm64."

* commit 'ba778fae99ec3b38d4f98262e6b7072bab0e9de4':
  AArch64: Add memcmp.S for Arm64.

am 685d3526: Merge "Add more read barriers to the class linker."

* commit '685d3526f5be8f5d0ffce67a04ecf35fb2b07b1d':
  Add more read barriers to the class linker.

am fe7aee2d: Merge "More advanced timing loggers."

* commit 'fe7aee2d20b47dad6029a588e0c7394f0c75d080':
  More advanced timing loggers.

am 1f47b4d8: Merge "Add logging to monitor deflation."

* commit '1f47b4d8c20e443fac7e8b8b7a77c723fb0be3a1':
  Add logging to monitor deflation.

am 27113be9: Merge "Revert "Revert "ART: Split out more cases of Load/StoreRef, volatile as parameter"""

* commit '27113be940950155164ca4513bc9f7b1fe050dac':
  Revert "Revert "ART: Split out more cases of Load/StoreRef, volatile as parameter""

resolved conflicts for merge of 50e7d733 to klp-modular-dev-plus-aosp

Change-Id: I7f12783da2f656fd2b8d53de737399b3f7f6b524

x86_64: Fix wide argument increment

This patch fixes to always increment the index for a wide argument,
and fixes the index upper bound.
Otherwise, the mapping may be incorrect.

Change-Id: I0116d8fd0a0a5c1270a23129c73a9e3651132977
Signed-off-by: Chao-ying Fu <chao-ying.fu@intel.com>
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>

am 9658d24d: Merge "Fixed ImageWriter timings not being written out."

* commit '9658d24deaa9bd07781bfac860d2a6dd89066d55':
  Fixed ImageWriter timings not being written out.

am fbd18f19: Merge "Revert "ART: Split out more cases of Load/StoreRef, volatile as parameter""

* commit 'fbd18f1923334f3208cfe6ba5f1d4f9eb421b063':
  Revert "ART: Split out more cases of Load/StoreRef, volatile as parameter"

am 7e47f713: Merge "ART: Split out more cases of Load/StoreRef, volatile as parameter"

* commit '7e47f713067b55e24b5d24f2c892ceefd7971ebf':
  ART: Split out more cases of Load/StoreRef, volatile as parameter

am 1528b02c: Merge "Add some more instruction support to optimizing compiler."

* commit '1528b02c4d5241e785bb680f13de70c355e67429':
  Add some more instruction support to optimizing compiler.

Merge "Add more read barriers to the class linker."

Add more read barriers to the class linker.

This change makes it possible to concurrently scan the remaining roots
in the class linker (the non-class-table roots that are visited by
ClassLinker::VisitRoots()) by adding read barriers.

Bug: 12687968
Change-Id: I66fecf7a303eee7537429e018f38da8270b18c67

Merge "More advanced timing loggers."

More advanced timing loggers.

The new timing loggers have lower overhead since they only push into
a vector. The new format has two types, a start timing and a stop
timing. You can thing of these as brackets associated with a
timestamp. It uses these to construct various statistics when needed,
such as: Total time, exclusive time, and nesting depth.

Changed PrettyDuration to have a default of 3 digits after the decimal
point.

Exaple of a GC dump with exclusive / total times and indenting:
I/art     (23546): GC iteration timing logger [Exclusive time] [Total time]
I/art     (23546):   0ms InitializePhase
I/art     (23546):   0.305ms/167.746ms MarkingPhase
I/art     (23546):     0ms BindBitmaps
I/art     (23546):     0ms FindDefaultSpaceBitmap
I/art     (23546):     0ms/1.709ms ProcessCards
I/art     (23546):       0.183ms ImageModUnionClearCards
I/art     (23546):       0.916ms ZygoteModUnionClearCards
I/art     (23546):       0.610ms AllocSpaceClearCards
I/art     (23546):       1.373ms AllocSpaceClearCards
I/art     (23546):     0.305ms/6.318ms MarkRoots
I/art     (23546):       2.106ms MarkRootsCheckpoint
I/art     (23546):       0.153ms MarkNonThreadRoots
I/art     (23546):       4.287ms MarkConcurrentRoots
I/art     (23546):     43.461ms UpdateAndMarkImageModUnionTable
I/art     (23546):     0ms/112.712ms RecursiveMark
I/art     (23546):       112.712ms ProcessMarkStack
I/art     (23546):     0.610ms/2.777ms PreCleanCards
I/art     (23546):       0.305ms/0.855ms ProcessCards
I/art     (23546):         0.153ms ImageModUnionClearCards
I/art     (23546):         0.610ms ZygoteModUnionClearCards
I/art     (23546):         0.610ms AllocSpaceClearCards
I/art     (23546):         0.549ms AllocSpaceClearCards
I/art     (23546):       0.549ms MarkRootsCheckpoint
I/art     (23546):       0.610ms MarkNonThreadRoots
I/art     (23546):       0ms MarkConcurrentRoots
I/art     (23546):       0.610ms ScanGrayImageSpaceObjects
I/art     (23546):       0.305ms ScanGrayZygoteSpaceObjects
I/art     (23546):       0.305ms ScanGrayAllocSpaceObjects
I/art     (23546):       1.129ms ScanGrayAllocSpaceObjects
I/art     (23546):       0ms ProcessMarkStack
I/art     (23546):   0ms/0.977ms (Paused)PausePhase
I/art     (23546):     0.244ms ReMarkRoots
I/art     (23546):     0.672ms (Paused)ScanGrayObjects
I/art     (23546):     0ms (Paused)ProcessMarkStack
I/art     (23546):     0ms/0.610ms SwapStacks
I/art     (23546):       0.610ms RevokeAllThreadLocalAllocationStacks
I/art     (23546):     0ms PreSweepingGcVerification
I/art     (23546):   0ms/10.621ms ReclaimPhase
I/art     (23546):     0.610ms/0.702ms ProcessReferences
I/art     (23546):       0.214ms/0.641ms EnqueueFinalizerReferences
I/art     (23546):         0.427ms ProcessMarkStack
I/art     (23546):     0.488ms SweepSystemWeaks
I/art     (23546):     0.824ms/9.400ms Sweep
I/art     (23546):       0ms SweepMallocSpace
I/art     (23546):       0.214ms SweepZygoteSpace
I/art     (23546):       0.122ms SweepMallocSpace
I/art     (23546):       6.226ms SweepMallocSpace
I/art     (23546):       0ms SweepMallocSpace
I/art     (23546):       2.144ms SweepLargeObjects
I/art     (23546):     0.305ms SwapBitmaps
I/art     (23546):     0ms UnBindBitmaps
I/art     (23546):   0.275ms FinishPhase
I/art     (23546): GC iteration timing logger: end, 178.971ms

Change-Id: Ia55b65609468f212b3cd65cda66b843da42be645

Merge "Revert "ART: Split out more cases of Load/StoreRef, volatile as parameter""

Revert "ART: Split out more cases of Load/StoreRef, volatile as parameter"

This reverts commit 2689fbad6b5ec1ae8f8c8791a80c6fd3cf24144d.

Breaks the build.

Change-Id: I9faad4e9a83b32f5f38b2ef95d6f9a33345efa33

Merge "Multilib ART host."

Multilib ART host.

Build ART for the host as a multilib project with dalvikvm32 and dalvikvm64
running as 32 or 64-bit repsectfully. Note, currently multilib host builds
are not the default, you make the so by setting BUILD_HOST_64bit=1.
Extend tests to execute in both 32 and 64-bit modes. By default both 32 and
64-bit tests are run, add 32 or 64 to the end of a test name to run it in
purely that flavor.
Given the extra spam, modify oat tests to only generate console output when
the test fails.
Change the test harness so that common commands are run when a test should be
skipped, when it passes or when it fails. Use these commands to generate a
summary of passing, skipped and failing tests. Tests will be skipped if they
are known to be broken or if a test has already failed. Setting the variable
TEST_ART_KEEP_GOING=true will force working tests not to be skipped.
In this change all tests running on the optimizing compiler are marked broken
due to breakages running them in a multilib environment.
Break apart Android.common.mk into its constituent parts, along with other
pieces of reorganization.

Stylistic nit, we refer to make rule targets as targets thereby overloading
the term target. While consistent with make's terminology, its confusing with
the Android notion of target. I've switched to just calling targets rules to
avoid confusion in host tests.

Change-Id: I5190fc3de46800a949fbb06b3f4c258ca89ccde9

Merge "AArch64: Add memcmp.S for Arm64."

Merge "Add logging to monitor deflation."

Add logging to monitor deflation.

Change-Id: I0251ff19e0a3c3b9edc7c7e296f15eb3229f8f7c

am a76724f4: Merge "Use GCC on ARM64"

* commit 'a76724f4bc4a8d36c6c99243835b8e3ddf1eb2d7':
  Use GCC on ARM64

am 1bc67722: Merge "Log profile info only on debug builds."

* commit '1bc67722c1bcf26934900768a60ca0bfb0a6d1a0':
  Log profile info only on debug builds.

am 7db22606: Merge "Always compile leafs during profile based compilation."

* commit '7db22606228d21b3adba04851409a4a78e6b4263':
  Always compile leafs during profile based compilation.

am ad1e79fb: Merge "Move trampolines to thumb2 instead of ARM"

* commit 'ad1e79fb6871bee2b5e91560227ca7acddf99cb9':
  Move trampolines to thumb2 instead of ARM

am cbb0e809: Merge "Use reader lock of bulk free lock when not freeing."

* commit 'cbb0e809c0a4e8a4e8b7f5d3768a1864cfb381bb':
  Use reader lock of bulk free lock when not freeing.

Merge "Use GCC on ARM64"

Merge "Log profile info only on debug builds."

Merge "Always compile leafs during profile based compilation."

Merge "ART: Split out more cases of Load/StoreRef, volatile as parameter"

Merge "Use reader lock of bulk free lock when not freeing."

Use reader lock of bulk free lock when not freeing.

Should help reduce contention observed in systrace.

Change-Id: Iadb81728d4ba797c3a68acea795b15d7f212e89b

Update black list for klp-modular-dev

Change-Id: Ia6b85b249fe80578e49eacec5edd6315f8e7698c

am 5dcdc935: Merge "Fix implicit check option handling for non-cross-compiles"

* commit '5dcdc93539ddf48757b6d5ea9393812d6bf9861d':
  Fix implicit check option handling for non-cross-compiles

Merge "Fix implicit check option handling for non-cross-compiles"

Fix implicit check option handling for non-cross-compiles

This fixes an issue where the compiler was not being told to
generate implicit checks but the runtime was expecting them.

Bug: 15747876
Change-Id: I65e7475bac245c44d5094eb666d67bc1af327ab1

Merge "Revert "Revert "ART: Split out more cases of Load/StoreRef, volatile as parameter"""

Revert "Revert "ART: Split out more cases of Load/StoreRef, volatile as parameter""

This reverts commit de68676b24f61a55adc0b22fe828f036a5925c41.

Fixes an API comment, and differentiates between inserting and appending.

Change-Id: I0e9a21bb1d25766e3cbd802d8b48633ae251a6bf

Merge "Fixed ImageWriter timings not being written out."

Fixed ImageWriter timings not being written out.

During host runs of dex2oat the timings for ImageWriter would not be
written out.

Change-Id: I74ce8bae724d1e732a6a1efb1c82804473081693

Merge "Add some more instruction support to optimizing compiler."

Add some more instruction support to optimizing compiler.

This adds a few more DEX instructions to the optimizing compiler's
builder (constants, moves, if_xx, etc).

Also:
* Changes the codegen for IF_XX instructions to use a condition
  rather than comparing a value against 0.
* Fixes some instructions in the ARM disassembler.
* Fixes PushList and PopList in the thumb2 assembler.
* Switches the assembler for the optimizing compiler to thumb2
  rather than ARM.

Change-Id: Iaafcd02243ccc5b03a054ef7a15285b84c06740f

Merge "Move trampolines to thumb2 instead of ARM"

am 3c6c4645: Merge "Mac doesn\'t define MAP_ANONYMOUS"

* commit '3c6c4645ce50836387d70f6562fa117047d7eea1':
  Mac doesn't define MAP_ANONYMOUS

ART: Split out more cases of Load/StoreRef, volatile as parameter

Splits out more cases of ref registers being loaded or stored. For
code clarity, adds volatile as a flag parameter instead of a separate
method.

On ARM64, continue cleanup. Add flags to print/fatal on size mismatches.

Change-Id: I30ed88433a6b4ff5399aefffe44c14a5e6f4ca4e

am 9462a31c: Merge "x86_64: Correct fix for cmp-long"

* commit '9462a31caedefac3e04bd4aa5088e050ed188b30':
  x86_64: Correct fix for cmp-long

Move trampolines to thumb2 instead of ARM

Currently the entrypoint trampolines are compiled using the ARM32 assembler.
This changes that to use the thumb2 assembler for consistency with the
other compiled code.

Bug: 15455279
Change-Id: I6bacdf359e2ae4c0967fb5cd9dc694af11f802ec

Log profile info only on debug builds.

Change-Id: I80cce951bf4cce988c9ab8373be3e335860eadf3

Always compile leafs during profile based compilation.

Bug:12877748
Change-Id: Idbee9eaa87c419454728de195bdbdd9b0e6b7b50

Merge "x86_64: Correct fix for cmp-long"

x86_64: Correct fix for cmp-long

We cannot rely on the sign of the sub instruction because
LONG_MAX - LONG_MIN = -1 and the sign will indicate that
LONG_MAX < KONG_MIN and it is incorrect.

The fix also contains small improvement for load wide constant.

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

Merge "Mac doesn't define MAP_ANONYMOUS"

Mac doesn't define MAP_ANONYMOUS

Fix mac build.

Change-Id: I607e8fff227259fbf27e468193a705c8c3fc1b59

am be781e55: Merge "Revert "Revert "ART: Fix StoreValue to use RefDisp when necessary."""

* commit 'be781e55b86ddd2dacf12b80a26fa9a5612948c3':
  Revert "Revert "ART: Fix StoreValue to use RefDisp when necessary.""

am dd79f215: Merge "Limit the size of files created by run-test to 2MB."

* commit 'dd79f2157fcea35afc8dfbe75a0f8c4bbc6898b9':
  Limit the size of files created by run-test to 2MB.

am a5377fe9: Merge "Revert "ART: Fix StoreValue to use RefDisp when necessary.""

* commit 'a5377fe9231b441856b3c9bfa4aa003445436958':
  Revert "ART: Fix StoreValue to use RefDisp when necessary."

Merge "Limit the size of files created by run-test to 2MB."

am 56027931: Merge "More verbose logs for JDWP events"

* commit '560279311a756cdd7bc708beb52b141184064d15':
  More verbose logs for JDWP events

am 87523b6c: Merge "X86 Dis: Add missing mov byte; Add size suffixes"

* commit '87523b6c4df4dd33d4c1e383403e14d6fe06362f':
  X86 Dis: Add missing mov byte; Add size suffixes

Limit the size of files created by run-test to 2MB.

In MemMap::MapAnonymous on the host, avoid creating ashmem regions that are
backed by files, when the size of the file will be greater than the ulimit.

Change-Id: I7cbf2ac59bf5869ed85850bea8d71898f3b7e1c7

Merge "X86 Dis: Add missing mov byte; Add size suffixes"

X86 Dis: Add missing mov byte; Add size suffixes

Yet another instruction not disassembled properly.
Add 'b', 'w', 'q' to opcodes to diffferentiate between various versions
and make it more understandable.

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

am 5c1ad337: Merge "Use command line android root in dex2oat to find image."

* commit '5c1ad33701beac62afbfb144bb6f52755503c9f2':
  Use command line android root in dex2oat to find image.

am b493c298: Merge "x86_64: Enable some optimizations"

* commit 'b493c2983016a78de498c3a3aef302b1353dca99':
  x86_64: Enable some optimizations

am 4370d3d3: Merge "x86_64: Fix fp-to-core conversion"

* commit '4370d3d3a92b45ac3bdceae0c3059648b0f3fc1d':
  x86_64: Fix fp-to-core conversion

am 7499cd1d: Merge "x86_64: Fix comp-long"

* commit '7499cd1d4445fd00aa3c9503497a171fae9bf20e':
  x86_64: Fix comp-long

Merge "Revert "ART: Fix StoreValue to use RefDisp when necessary.""

Revert "ART: Fix StoreValue to use RefDisp when necessary."

This reverts commit 596bea8f6331fe8ad6eb456afa1f7dfe1b92e09c.

Change-Id: I1942bfc5127e8a631c5b24d107d304c7cd994e78

Merge "Use command line android root in dex2oat to find image."

Merge "x86_64: Fix comp-long"

am b06a0364: Merge "ART: Method verifier must check invoke-virtuals for non-zero args"

* commit 'b06a0364e9a5a27d2a33ab7a9c9b98c7332e6cea':
  ART: Method verifier must check invoke-virtuals for non-zero args

am 5c0fcdad: Merge "Shared single GC iteration accounting for all GCs."

* commit '5c0fcdad7578dd8ce0e53e9d8f02777b253762cc':
  Shared single GC iteration accounting for all GCs.

am faa69743: Merge "ART: ReadFully needs to fail when EOF"

* commit 'faa69743dc3e2c115d3807ada4a51951092d62a0':
  ART: ReadFully needs to fail when EOF

am 990f110b: Merge "ART: Fix StoreValue to use RefDisp when necessary."

* commit '990f110b45127055d99fe894afb1ae4cfd92046c':
  ART: Fix StoreValue to use RefDisp when necessary.

am 40bdf97b: Merge "Quick compiler: enable Arm64 compilation"

* commit '40bdf97beffc4d85c966965bdbe81b243141312b':
  Quick compiler: enable Arm64 compilation

am e617e9a1: Merge "ART: Fix register overlap checks in CopyArgumentRegs"

* commit 'e617e9a1e5c70bd3fd1e68056fa4090a3486a0b7':
  ART: Fix register overlap checks in CopyArgumentRegs

am 8d16f203: Merge "Add more read barriers for JNI roots."

* commit '8d16f203d40ab4018986baa018e52185ad05dae6':
  Add more read barriers for JNI roots.

Merge "Revert "Revert "ART: Fix StoreValue to use RefDisp when necessary."""

Revert "Revert "ART: Fix StoreValue to use RefDisp when necessary.""

This reverts commit e98297bafe84e2ac15eb73739f27826fcaae1203.

No matter if holding a reference or not, the store needs to be tagged
as updating a Dalvik register.

Change-Id: Icb80e1b6f82103cff33ee971faaaa02f9ad4a683

Merge "x86_64: Enable some optimizations"

Merge "x86_64: Fix fp-to-core conversion"

x86_64: Fix comp-long

If result and source are the same VR then implementation does
a wrong thing clearing dest in the beginning.

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

ART: Fix StoreValue to use RefDisp when necessary.

Split from https://android-review.googlesource.com/#/c/98605

Change-Id: Ib3a03428ddbec659591b838404bbfa1553fe18e9

Merge "More verbose logs for JDWP events"

More verbose logs for JDWP events

Logs more information about the JDWP event being reported with VLOG(jdwp). This
helps mainly when debugging JDWP with -verbose:jdwp option.

Change-Id: I49495bdab03dbcbdc0b6c00a8df718bfedbad91d

x86_64: Fix fp-to-core conversion

Long max value cannot be represented with double precision
and check fp against max_long does not guard invocation of
conversion instruction. As a result conversion ends up with
min long instead of max long.

The patch changes the guard check to not allow conversion
instruction for max long.

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

AArch64: Add memcmp.S for Arm64.

This patch adds a copy of bionic's memcmp.S to ART. A later patch
modifies this to memcmp16_arm64.S.

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

Use GCC on ARM64

This automatically enables the computed-goto interpreter as default.

Bug: 15763315
Bug: 15405621
Change-Id: I4f6ed5f7cad96333bc8dc1029e9b90ba1b6916df

am 2d2d6bb1: Merge "ART: Do not try to pretty-print method in dex file"

* commit '2d2d6bb1ff8692efdddf104dabfb4a2d39877ff6':
  ART: Do not try to pretty-print method in dex file

am 6d90efa6: Merge "ART: Method verifier needs to check 2-reg values"

* commit '6d90efa6a67a0de0dc6136874b3d9797c778e912':
  ART: Method verifier needs to check 2-reg values