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.

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 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 35d9a38d: Merge "Set REG0_USED on X86 Set8R instruction"

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

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

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

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

Change-Id: I6886aa3f4066eead9242eb77042d89c4ee28da1b

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 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 b5c5646b: Merge "Use the uncached form of BacktraceMap::Create."

* commit 'b5c5646b5ffd3d451af5e8d52b50743bf61d4c40':
  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 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 "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

Merge "Add x86_64 optimized __memcmp16 implementation; fix tabs in 32-bit implementation."

Add x86_64 optimized __memcmp16 implementation;
fix tabs in 32-bit implementation.

Change-Id: I7bbfb344074aed66511c1a845998dc38798116ea
Signed-off-by: Varvara Rainchik <varvara.rainchik@intel.com>

Merge "x86_64: Load double constant using core register"

x86_64: Load double constant using core register

This changes the double const loading scheme for x86_64 in order to
reduce number of memory accesses.

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

Merge "Set REG0_USED on X86 Set8R instruction"

Set REG0_USED on X86 Set8R instruction

Since this instruction only affects the low byte of the register, it is
preceded by an XOR to zero the upper 3 bytes.  Set8R isn't marked as
using operand 0 as an input.  In practice, this works for now, as the
Xor sets the CC and Set8R uses the CC (although not that of the Xor, but
of a Cmp generally).

This just marks REG0 as using the previous contents of the register, as
it is modifying only 1 byte of 4.

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

Merge "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
Change-Id: I8f2538861d5c3fa7b9a04d2c3f516319cc060291

am 9e9b15dd: Merge "Add trampoline invoke and ART callback tests for native-bridge"

* commit '9e9b15dd29ff4d91f31ed1063561963b2f99ad1c':
  Add trampoline invoke and ART callback tests for native-bridge

am 1a09d286: Merge "AArch64: Implement InexpensiveConstant methods."

* commit '1a09d286bdb4abc9aae06dcf4a8b052648816e2b':
  AArch64: Implement InexpensiveConstant methods.

am c7975a33: Merge "AArch64: Add inlining support for ceil(), floor(), rint(), round()"

* commit 'c7975a33b9c8fb36e85c41162462aeec8e76232a':
  AArch64: Add inlining support for ceil(), floor(), rint(), round()

Merge "AArch64: Implement InexpensiveConstant methods."

AArch64: Implement InexpensiveConstant methods.

Implement IsInexpensiveConstant and friends for A64.
Also extending the methods to take the opcode with respect to which
the constant is inexpensive. Additionally, logical operations (i.e.
and, or, xor) can now handle the immediates 0 and ~0 (which are not
logical immediates).

Change-Id: I46ce1287703765c5ab54983d13c1b3a1f5838622

am 316fb6e4: Merge "Make verifier identify unresolved methods as constructors by <init>."

* commit '316fb6e4992f41fa0742f1360c6497787c9433e3':
  Make verifier identify unresolved methods as constructors by <init>.

am 04ce1134: Make verifier identify unresolved methods as constructors by <init>.

* commit '04ce113458b6b01f96d58dbb4c41682ee22d8b5d':
  Make verifier identify unresolved methods as constructors by <init>.

Make verifier identify unresolved methods as constructors by <init>.

Check was previously missing brackets in the method name.

Bug: 16560516
Change-Id: I6b28f8413f3d4c14269a6de38731213d7c964af6

am 7aaa8dbf: Merge "ART: Implement the easy long division/remainder by a constant"

* commit '7aaa8dbfbcde7f58564b42044892891559669838':
  ART: Implement the easy long division/remainder by a constant

am 4e97f656: Merge "Move image classes options from art to AndroidRuntime [art]"

* commit '4e97f6564fa51609d5651374dcdd2a8f4e79f272':
  Move image classes options from art to AndroidRuntime [art]

am dc3ca147: Merge "Move from dalvik.vm.image-dex2oat-flags to dalvik.vm.dex2oat-filter [art]"

* commit 'dc3ca147ee880088eb2f52b7b0355062ba5749be':
  Move from dalvik.vm.image-dex2oat-flags to dalvik.vm.dex2oat-filter [art]

am 77a5b501: Merge "AArch64: Remove unnecessary work around for sp."

* commit '77a5b50148633c2651edb7f35f5254e40cc878dc':
  AArch64: Remove unnecessary work around for sp.

am f7f6c978: Merge "Inform about System.exit() call in logcat"

* commit 'f7f6c978d44a785d4c12879c6ca20ca1832d3217':
  Inform about System.exit() call in logcat

Use canonical paths when searching for dex files

Apps which use the DexPathClassLoader directly may
pass symlinks when trying to load dex files. This
will not work as we use string comparision to find
the dex in an oat file. The CL fixes this issue by
using using dex conical paths for comparisons.

Bug: 15313272

(cherry picked from commit 4e1d579d6401fef2dd57b16f8d406e33221a69d9)

Change-Id: I441f1ef18388c4a17c747a7e55b57f917724db85

Use canonical paths when searching for dex files

Apps which use the DexPathClassLoader directly may
pass symlinks when trying to load dex files. This
will not work as we use string comparision to find
the dex in an oat file. The CL fixes this issue by
using using dex conical paths for comparisons.

Bug: 15313272

Change-Id: Ic314374b17612c3afbcadec93a88b2515a0aca5e

am 7906b25f: Merge "ART: Rework ARM64 entry sequence"

* commit '7906b25f79f24af3a8b8eee0454b4db1278b9b77':
  ART: Rework ARM64 entry sequence

am d4c3c62c: Fix broken JNI IsAssignableFrom.

* commit 'd4c3c62cc68296fbaf35e7b37108c2fde2278a3b':
  Fix broken JNI IsAssignableFrom.

am 43c63b5a: Merge "Fix broken JNI IsAssignableFrom."

* commit '43c63b5aaa7cd1c6d0ca13b7ea28cc321848d127':
  Fix broken JNI IsAssignableFrom.

Fix broken JNI IsAssignableFrom.

The expected argument order was the opposite of what the spec
dictated.

Reported-By: Mikhail Naganov <mnaganov@google.com>
bug: 16531674

(cherry picked from commit 1268b742c8cff7318dc0b5b283cbaeabfe0725ba)

Change-Id: I2b6636998c4e15c1eb10dc96a57940aca56c4d0d

Merge "Inform about System.exit() call in logcat"

Inform about System.exit() call in logcat

This diagnostic message helps to filter out crashes during System.exit()
when thread's behaviour is undefined.

Change-Id: I772932a35b503b1efca695e7da7bf2e562c03a7c
Signed-off-by: Dmitry Petrochenko <dmitry.petrochenko@intel.com>

Merge "Fix broken JNI IsAssignableFrom."

ART: Fix lock max count definition

The lock max count should utilize 14 bits since 2 highest bits are reserved for lock state.

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

Fix broken JNI IsAssignableFrom.

The expected argument order was the opposite of what the spec
dictated.

Reported-By: Mikhail Naganov <mnaganov@google.com>
bug: 16531674

Change-Id: I1ebc6d5814ebb4feeb96f3ecdf1a4602da4ba7ce

Merge "Make verifier identify unresolved methods as constructors by <init>."

Make verifier identify unresolved methods as constructors by <init>.

Check was previously missing brackets in the method name.

Bug: 16560516

(cherry picked from commit 04ce113458b6b01f96d58dbb4c41682ee22d8b5d)

Change-Id: Ie98c5b84e73ffc43ea08e1b08ae9c6e41bcd5891

Merge "ART: Implement the easy long division/remainder by a constant"

ART: Implement the easy long division/remainder by a constant

Also optimizes long/int divisions by power-of-two values.

Also do some clean-up.

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

Merge "Move image classes options from art to AndroidRuntime [art]"

Merge "Add trampoline invoke and ART callback tests for native-bridge"

Add trampoline invoke and ART callback tests for native-bridge

Create trampolines for JNI functions in native-bridge. Test trampoline
invoking from ART side.

Add tests for ART callbacks which called from native-bridge. In this test
they are invoked from the trampoline of JNI_OnLoad.

Change-Id: Ibaa5103fdf49a6b29a1c12f45410fe5f82d46b1e

Merge "Move from dalvik.vm.image-dex2oat-flags to dalvik.vm.dex2oat-filter [art]"

Merge "ART: Rework ARM64 entry sequence"