Merge "Fix a SIGSEGV at background-to-foreground collector transition." into lmp-dev

ART: Make assembler_test less chatty

Do not save temporaries in case we use a different encoding than
the host assembler.

Bug: 16505797

(cherry picked from commit 54e15de4a3ea869488d50694fa01138901e70c4e)

Change-Id: I9508d9b09d930db70587a69c04bb3c8950339d79

Fix a SIGSEGV at background-to-foreground collector transition.

The mem map that's used to create the backup main rosalloc space is
set to PROT_NONE by the compaction and the rosalloc space creation
attempts to write the debug magic number in the first page.

This is a debug build only issue.

Cherry pick commit c1276c838d1d0896048b8c14458432b94274e869.

Bug: 16854263
Change-Id: Ide2545d92bcafd3a095ac7f0c89c5b43d6dd78de

Add arraycopy intrinsic for arm and arm64.

Implement intrinsic for
java.lang.System.arraycopy(char[], int, char[], int, int).

Bug: 16241558

(cherry picked from commit 947717a2b085f36ea007ac64f728e19ff1c8db0b)

Change-Id: I8199f5c9ce9827f869f0f93aaff7ec359a84d922

Check for a NULL pointer, do not call Build.

When calling BacktraceMap::Create(), a NULL pointer is returned if Build
fails. Building twice can cause problems and might leak memory.

(cherry picked from commit 836572a07142627ff291d686a4e9e03a0988344b)

Change-Id: I59d29bb6e5324de6eb099916045c2ab1d9e56630

Merge "Use the right instruction set for checking dex-cache staleness." into lmp-dev

Use the right instruction set for checking dex-cache staleness.

We should use the instruction set we're asked to use, and not
the instruction set of the current runtime.

bug: 16876489
Change-Id: Ic2a30b7e7d23192b0189c1f868b73c5f4c7eb8f9

Fix performance regression in OatFile::GetOatDexFile().

Try to avoid calculating the canonical location of the
dex file if possible and when we have to calculate it,
cache the lookup result for subsequent lookups.

Bug: 16828525
Bug: 16859671

(cherry picked from commit 3f5838d7d0b9fc63db0ccc35c2ea05ed29264986)

Change-Id: Ifd9a45dada2cc724382fd03c10f6437a6b71e666

AArch64: Implicit StackOverflow/NPE/Suspend checks.

This implements implicit stack overflow checks and null pointer exceptions
for AArch64.  Suspend checks are implemented but not switched on yet.

Bug: 16256184
Change-Id: I2eb076f2c0c9d94793d5a898fea49cf409b4eb66
Signed-off-by: Stuart Monteith <stuart.monteith@arm.com>

Fix checks for kLiteral in local optimizations.

The check for kLiteral (literal load) just checked the kLiteral
bit in the def mask.  The kEncodeAll mask has the kLiteral bit
set so this check was triggering.  The fix is to check for
only the kLiteral bit being set and no other special bits.

The semantics of the special bits in the use/def mask is that
only one of them can be set at the same time.

Bug: 16824330

Change-Id: I0f1c1157e017870414ffef11767e5433d1fd4401

Reset goog/lmp-dev to goog/master

Change-Id: If30071b5f86bfe723abf799c9248397e8840a8a0

am 56a49adb: am 860feb0a: Merge "Add card table test."

* commit '56a49adb43fa1c8c8485f7ce07286d3b3829ee7d':
  Add card table test.

am 860feb0a: Merge "Add card table test."

* commit '860feb0a60d0fe9311f28bd590058f6660d130b1':
  Add card table test.

am 82ec2f07: am 484e2c2d: Merge "Revert "Rules depend on DEX2OATD, not DEX2OATD_DEPENDENCY.""

* commit '82ec2f0790eb59a7ed71490d503a9b7cdef25ce4':
  Revert "Rules depend on DEX2OATD, not DEX2OATD_DEPENDENCY."

am c75dc324: am 38449637: Merge "When performing a long jump, update SP last."

* commit 'c75dc3242ab3d1a1d8684b049b6299a4b2ca57cf':
  When performing a long jump, update SP last.

am dc6414fe: am cd0d9b5a: Merge "When performing a long jump, update SP last." into lmp-dev

* commit 'dc6414fe6b56f2854dacbaac82c2b0b4b57d1e6f':
  When performing a long jump, update SP last.

am da113966: am 1d5c4953: Merge "Fix SIB for base + index addressing in x86_64 assembler."

* commit 'da113966c3a06facd1073e71436c12a4d470f12b':
  Fix SIB for base + index addressing in x86_64 assembler.

am a84690e3: am 0a1143de: Merge "Add x86_64 optimized __memcmp16 implementation; fix tabs in 32-bit implementation."

* commit 'a84690e3b0607f286e4680350b98d54e208fa614':
  Add x86_64 optimized __memcmp16 implementation; fix tabs in 32-bit implementation.

am 0b8c1530: am a1227a18: Fix byte_cas to use uintptr_t Atomic.

* commit '0b8c153082caf6a970e238c9ac34ce0601947dd2':
  Fix byte_cas to use uintptr_t Atomic.

am 5ce0921f: am 4853f940: Merge "Fix byte_cas to use uintptr_t Atomic."

* commit '5ce0921fbaf6a8eead373a253fc8813cde54770a':
  Fix byte_cas to use uintptr_t Atomic.

am 1d874ca3: am ae5092fc: Merge "Fix movw in x86_64 assembler."

* commit '1d874ca361749a062f9a8861425873f46242b98c':
  Fix movw in x86_64 assembler.

am c598f8f8: am 4b6d27c2: Merge "x86: Fix assembler for Pextr"

* commit 'c598f8f88ca70283472434fcfc0ab5af8ddb30f1':
  x86: Fix assembler for Pextr

am 1622baf3: am cbb5747b: Merge "ART: Fix lock max count definition"

* commit '1622baf30c9ef3897fd80d8d4fb0860511608573':
  ART: Fix lock max count definition

am 484e2c2d: Merge "Revert "Rules depend on DEX2OATD, not DEX2OATD_DEPENDENCY.""

* commit '484e2c2d3531e5bb36f0e1e12f26c708939c6579':
  Revert "Rules depend on DEX2OATD, not DEX2OATD_DEPENDENCY."

am 38449637: Merge "When performing a long jump, update SP last."

* commit '38449637b1ee8209e2eb81b3f27f40bb9ab3934f':
  When performing a long jump, update SP last.

am cd0d9b5a: Merge "When performing a long jump, update SP last." into lmp-dev

* commit 'cd0d9b5aaec7125dd62775d9f970151a1e0eab90':
  When performing a long jump, update SP last.

am 1d5c4953: Merge "Fix SIB for base + index addressing in x86_64 assembler."

* commit '1d5c4953fa8cfa5c7276ab1a2d2c12c1baacbf08':
  Fix SIB for base + index addressing in x86_64 assembler.

am 0a1143de: Merge "Add x86_64 optimized __memcmp16 implementation; fix tabs in 32-bit implementation."

* commit '0a1143de1414abc73634f2ff8941cdd6acd5e9a8':
  Add x86_64 optimized __memcmp16 implementation; fix tabs in 32-bit implementation.

am c276c14f: am f768db4c: Merge "X86: Assembler: Correct r8_form for some cases"

* commit 'c276c14f16c6dde063311631bcb0e9096d822aeb':
  X86: Assembler: Correct r8_form for some cases

am b74629b7: am f6e8d030: Merge "x86_64: Load double constant using core register"

* commit 'b74629b7136a7ca8eec19215847ba808b01d2648':
  x86_64: Load double constant using core register

am 277c6bb7: am 6175442a: Merge changes Ibe92d8b5,I9bada598

* commit '277c6bb72a0899dd7c4462bcd8a67fcdeab90650':
  Make system use patchoat to relocate during runtime.
  Temporary fix for bad LIR generation in GenInlinedGet.

Make system use patchoat to relocate during runtime.

Change dalvik_system_DexFile.cc so that isDexOptNeededInternal will be
able to indicate that a patchoat is required. Change default of relocate
option to be on.

Bug: 15358152

(cherry picked from commit 6e183f2e973a20f2eaca135c240908e1bf98c5d0)

Change-Id: Ib21f4f41b6cbf18094e3ca1a30d65a3b197b71b0

Merge "When performing a long jump, update SP last." into lmp-dev

When performing a long jump, update SP last.

Bug: 16571371
Change-Id: I719d9880a64e277330a2da698d5f6f7d320014d0

am e94cddb6: am a7975e0d: Merge "ART: Enable x86_64 bit support for intrinsic for System.arraycopy(char[], ..)"

* commit 'e94cddb6c748511f068dc9ea9afeab9b54b132db':
  ART: Enable x86_64 bit support for intrinsic for System.arraycopy(char[], ..)

am a1227a18: Fix byte_cas to use uintptr_t Atomic.

* commit 'a1227a185a5d6e2af83c532379b2f9b4600f6e16':
  Fix byte_cas to use uintptr_t Atomic.

am 4853f940: Merge "Fix byte_cas to use uintptr_t Atomic."

* commit '4853f940efe0aaccf83a4af55a8dd640d8ef7274':
  Fix byte_cas to use uintptr_t Atomic.

Fix byte_cas to use uintptr_t Atomic.

Previously we used int32_t Atomic with a uintptr_t bit shift.
This was a mismatch on 64 bit and resulted in occasionally
having the cas return without having succeeded.

Bug: 16819816

(cherry picked from commit aa3c3e5ee83c061e7f387c75b1b29c9f248ac39c)

Change-Id: Ib97bf5880153ddfacf52409be3e9e821657bac59

am ae5092fc: Merge "Fix movw in x86_64 assembler."

* commit 'ae5092fcaf2175df9f8c7cf7f5542b31e829ac72':
  Fix movw in x86_64 assembler.

am 4b6d27c2: Merge "x86: Fix assembler for Pextr"

* commit '4b6d27c2079e246cdeb56dca6648539a16493dca':
  x86: Fix assembler for Pextr

am cbb5747b: Merge "ART: Fix lock max count definition"

* commit 'cbb5747ba911d12cfe8fd97d96441c7e40ab193e':
  ART: Fix lock max count definition

am f768db4c: Merge "X86: Assembler: Correct r8_form for some cases"

* commit 'f768db4c1cb7ec9374e203305cf3582f32147302':
  X86: Assembler: Correct r8_form for some cases

am f6e8d030: Merge "x86_64: Load double constant using core register"

* commit 'f6e8d0304d0b0024f1af8e63c2a3a3ad78761d59':
  x86_64: Load double constant using core register

am 6175442a: Merge changes Ibe92d8b5,I9bada598

* commit '6175442a246cded6a9caae4658a0a1dc7cf49071':
  Make system use patchoat to relocate during runtime.
  Temporary fix for bad LIR generation in GenInlinedGet.

am a7975e0d: Merge "ART: Enable x86_64 bit support for intrinsic for System.arraycopy(char[], ..)"

* commit 'a7975e0db7ce521b60ac5d4861a69bc76b7fbddc':
  ART: Enable x86_64 bit support for intrinsic for System.arraycopy(char[], ..)

Merge "Fix byte_cas to use uintptr_t Atomic."

Fix byte_cas to use uintptr_t Atomic.

Previously we used int32_t Atomic with a uintptr_t bit shift.
This was a mismatch on 64 bit and resulted in occasionally
having the cas return without having succeeded.

Bug: 16819816

Change-Id: I10fa9c2f09ae8fe0997a4c9293fa123c6bde5920

Merge "Fix movw in x86_64 assembler."

Fix movw in x86_64 assembler.

Change-Id: Ibceb03fd57adea09643aa77a9399be196fa14709

am 29c1a6d5: am 35d9a38d: Merge "Set REG0_USED on X86 Set8R instruction"

* commit '29c1a6d596ad5a8ed3ac9b21e1a0a5a62d6f6626':
  Set REG0_USED on X86 Set8R instruction

am 35d9a38d: Merge "Set REG0_USED on X86 Set8R instruction"

* commit '35d9a38d14fe8528310dced6d6a41f112ad95fcd':
  Set REG0_USED on X86 Set8R instruction

am 420d6990: am 09f60a9f: Merge "Use canonical paths when searching for dex files" into lmp-dev

* commit '420d6990cf9a31113b5ae5397e10a657fba9e42c':
  Use canonical paths when searching for dex files

am 09f60a9f: Merge "Use canonical paths when searching for dex files" into lmp-dev

* commit '09f60a9f53e6a87d5be84388f7fbf9a091fe0345':
  Use canonical paths when searching for dex files

am f6735280: resolved conflicts for merge of ac304133 to lmp-dev-plus-aosp

* commit 'f6735280cdb627985ec10a10391211aba7a67a57':
  ART: Fix memory unmapped twice issue in ElfFile::Load(bool)

resolved conflicts for merge of ac304133 to lmp-dev-plus-aosp

Change-Id: I6886aa3f4066eead9242eb77042d89c4ee28da1b

am d77c40a8: am 6bdbec76: Merge "ART: Correct disassembling of 64bit immediates on x86_64"

* commit 'd77c40a890bd1b30b0da64f4e240d9ea2dbc7040':
  ART: Correct disassembling of 64bit immediates on x86_64

am 2fb8b8b9: am 4d3401e8: Merge "x86_64: Fix Test32RM"

* commit '2fb8b8b9abc001ab840c0327dd30e39291901fbc':
  x86_64: Fix Test32RM

am 6bdbec76: Merge "ART: Correct disassembling of 64bit immediates on x86_64"

* commit '6bdbec76e16494e68632bab45f655e13aac477b8':
  ART: Correct disassembling of 64bit immediates on x86_64

am 4d3401e8: Merge "x86_64: Fix Test32RM"

* commit '4d3401e8e3c45203707431494739cb0a7181972b':
  x86_64: Fix Test32RM

am 3c1a3200: am 32155e0a: Merge "Use canonical paths when searching for dex files"

* commit '3c1a32000553b919803c8c125fcd5533d2cf99ae':
  Use canonical paths when searching for dex files

am 582d9367: am 3513cd30: Merge "Create allocator adapter for using Arena in std containers."

* commit '582d936772cc167d9337c98cf193f59f62d9079a':
  Create allocator adapter for using Arena in std containers.

am e339cbfe: am 8f1dc7a2: Merge "ART: Fix memory unmapped twice issue in ElfFile::Load(bool)"

* commit 'e339cbfe36ea45dbecd9428ade7697dbda9acdf3':
  ART: Fix memory unmapped twice issue in ElfFile::Load(bool)

am 32155e0a: Merge "Use canonical paths when searching for dex files"

* commit '32155e0a3373b7ce6e490fceb40ca24074dbcbb5':
  Use canonical paths when searching for dex files

am 3513cd30: Merge "Create allocator adapter for using Arena in std containers."

* commit '3513cd30ee8e710d58a289a52104e0eb6be9e277':
  Create allocator adapter for using Arena in std containers.

am 8f1dc7a2: Merge "ART: Fix memory unmapped twice issue in ElfFile::Load(bool)"

* commit '8f1dc7a20049ba6e2f3c7f800908bff811cdbea1':
  ART: Fix memory unmapped twice issue in ElfFile::Load(bool)

ART: Fix memory unmapped twice issue in ElfFile::Load(bool)

Root Cause:
  The overlapped memory region will be unmapped by
  (1) ~MemMap() of reservation MemMap (reserve) and
  (2) ~MemMap() of "reuse" MemMap (segment).
  Someone takes the memory region after (1) and it will be unmapped in (2).
  So, SIGSEGV occurs when using the unmapped memory region.

Solution:
  Fixes this issue by skip unmap "reuse" MemMap in destructor.
  And always create reservation MemMap before "reuse" MemMap. (It also solved
  the fixupELF case which does not reserve the whole needed memory region).

Bug: 16486685

(cherry picked from commit a62a588a9202f69e53fbeb3045ea8ea5ec2587f8)

Change-Id: Icb83c8e87fa168027d9d8adb34925000399d3d2a

am 940cb994: am b5c5646b: Merge "Use the uncached form of BacktraceMap::Create."

* commit '940cb994fa06b7c6d1226c2d3744817d615185e9':
  Use the uncached form of BacktraceMap::Create.

am b5c5646b: Merge "Use the uncached form of BacktraceMap::Create."

* commit 'b5c5646b5ffd3d451af5e8d52b50743bf61d4c40':
  Use the uncached form of BacktraceMap::Create.

am f0bddada: am 27a10f61: Use the uncached form of BacktraceMap::Create.

* commit 'f0bddada64f2cc2bb6fbd8f3cc3049b3a00575bd':
  Use the uncached form of BacktraceMap::Create.

am 27a10f61: Use the uncached form of BacktraceMap::Create.

* commit '27a10f618357cf85cc0677a04f0a5a3a8a437aed':
  Use the uncached form of BacktraceMap::Create.

am 154c3c93: am c52361cf: Merge "ART: Refactor long ops in quick compiler"

* commit '154c3c9308ee8342355862d377b7d1c6864dda66':
  ART: Refactor long ops in quick compiler

am c52361cf: Merge "ART: Refactor long ops in quick compiler"

* commit 'c52361cfaae8be2fbaa25b175d4f6c37d4110764':
  ART: Refactor long ops in quick compiler

Merge "ART: Correct disassembling of 64bit immediates on x86_64"

Merge "x86_64: Fix Test32RM"

x86_64: Fix Test32RM

This patch fixes Test32RM use flags and the format.

Change-Id: I486cb7f27e65caeefccbd3bbcc38257ddca033c8
Signed-off-by: Chao-ying Fu <chao-ying.fu@intel.com>

Merge "ART: Refactor long ops in quick compiler"

ART: Refactor long ops in quick compiler

Make GenArithOpLong virtual. Let the implementation in gen_common be
very basic, without instruction-set checks, and meant as a fall-back.
Backends should implement and dispatch to code for better implementations.
This allows to remove the GenXXXLong virtual methods from Mir2Lir, and
clean up the backends (especially removing some LOG(FATAL) implementations).

Change-Id: I6366443c0c325c1999582d281608b4fa229343cf

Merge "Add card table test."

Add card table test.

Tests some of the functionality supported by the card table.

Removed some logcat spam from monitor_pool.

Change-Id: I1423816a72572f78aca44552effa2b4c6aac46c8

Merge "x86: Fix assembler for Pextr"

Merge "Use canonical paths when searching for dex files"

Merge "Create allocator adapter for using Arena in std containers."

Create allocator adapter for using Arena in std containers.

Create ArenaAllocatorAdapter, similar to the existing
ScopedArenaAllocatorAdapter, for allocating memory for
standard containers via the ArenaAllocator. Add the ability
to specify allocation kind rather than just kArenaAllocSTL
to both adapters. Move the scoped arena allocator to the
scoped_arena_containers.h header file.

Define template aliases for containers using the new adapter
and change a few MIRGraph and Mir2Lir members to use them.

Change-Id: I9bbc50248e0fed81729497b848cb29bf68444268

Merge "Revert "Rules depend on DEX2OATD, not DEX2OATD_DEPENDENCY.""

Revert "Rules depend on DEX2OATD, not DEX2OATD_DEPENDENCY."

This reverts commit b1efe9746724f28773d185c7137c16ca8207ed47.

Change-Id: I5af2fae2c7b72e6ded72f00d3f7d4f306f6703ee

Merge "Use the uncached form of BacktraceMap::Create."

Use the uncached form of BacktraceMap::Create.

The code assumes that calling BacktraceMap::Create creates a map based
on the current state of the maps. This is not true by default, so
call the Create function indicating we want an uncached BacktraceMap object.

(cherry picked from 27a10f618357cf85cc0677a04f0a5a3a8a437aed)

Change-Id: Ic7a2dedaea9950174b24ede4dcda30ffe036fe93

Merge "Fix SIB for base + index addressing in x86_64 assembler."

Fix SIB for base + index addressing in x86_64 assembler.

Change-Id: Ib630bc28e6d694ffbe4a4a71cc988e36d00f6633

Merge changes Ibe92d8b5,I9bada598

* changes:
  Make system use patchoat to relocate during runtime.
  Temporary fix for bad LIR generation in GenInlinedGet.

Make system use patchoat to relocate during runtime.

Change dalvik_system_DexFile.cc so that isDexOptNeededInternal will be
able to indicate that a patchoat is required. Change default of relocate
option to be on.

Bug: 15358152

Change-Id: Ibe92d8b55a24bbf718b0416a21b76e5df7a2de26

Temporary fix for bad LIR generation in GenInlinedGet.

In GenInlinedGet we will sometimes try to load a class directly. If
we are doing that and find one, but cannot get the direct pointer
(due to image compilation or including patch information) we might
end up generating incorrect patching code due to the changing of the
cu_->dex_file attribute.

Bug: 16656190
Change-Id: I9bada598bf4fd972d6164163eedd4a496da306d6

Merge "ART: Fix lock max count definition"

Merge "X86: Assembler: Correct r8_form for some cases"

Merge "ART: Enable x86_64 bit support for intrinsic for System.arraycopy(char[], ..)"

ART: Enable x86_64 bit support for intrinsic for System.arraycopy(char[], ..)

Implements x86_64 support for intrinsic for java.lang.System.arraycopy(char[], int, char[], int, int).

With this fix the intrinsic works on x86 and x86_64 architectures.

Change-Id: Icc2889ccd0cf7d821522abb7437893e3149e7c99
Signed-off-by: Daniil Sokolov <daniil.y.sokolov@intel.com>

Merge "Use canonical paths when searching for dex files" into lmp-dev

Use the uncached form of BacktraceMap::Create.

The code assumes that calling BacktraceMap::Create creates a map based
on the current state of the maps. This is not true by default, so
call the Create function indicating we want an uncached BacktraceMap object.

Change-Id: I2c35149ba43db29ef0a9411fa5d9267d7c4dcbb3

Merge "When performing a long jump, update SP last."

When performing a long jump, update SP last.

Bug: 16571371

(cherry picked from commit 76a0723959c2185ace249e705774af9a679a0e0d)

Change-Id: Ib3a7bcda8aecdf7e92ea40a76de32aed83a42ee0