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 "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 "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 "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 "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."

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"

ART: Rework ARM64 entry sequence

Try to fold one sub of SP in the ARM64 entry sequence. When the
framesize is small, generate a sub over the full frame-size, and
adjust the spill offsets accordingly. If the framesize is too
large, use a pre-indexed store and fill upwards from there.

Change-Id: I1c15ac6276fb62b8164372de02fd92437f605938

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

AArch64: Add inlining support for ceil(), floor(), rint(), round()

This patch adds inlining support for the following Math, StrictMath
methods in the ARM64 backend:
  * double ceil(double)
  * double floor(double)
  * double rint(double)
  * long round(double)
  * int round(float)

Also some cleanup.

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

Merge "AArch64: Remove unnecessary work around for sp."

AArch64: Remove unnecessary work around for sp.

Use RegRegRegExtend to encode instruction for "sub/add sp, sp, Xm".

Change-Id: I13d3d2d386a7bd827e1396f291a7dcb9bffd5a29

am 89c210bf: Merge "Revert "Revert "Enable Load Store Elimination for ARM and ARM64"""

* commit '89c210bf418a152ccabfbbf853ddcce33aea450d':
  Revert "Revert "Enable Load Store Elimination for ARM and ARM64""

am 59a5525e: Merge "Add read barriers for the roots in the verifier."

* commit '59a5525e72d723c8598b6271c7b09845999beeb0':
  Add read barriers for the roots in the verifier.

Merge "Revert "Revert "Enable Load Store Elimination for ARM and ARM64"""

am 2eb3769c: Merge "Make ImageSpace skip validation if we are using a relocated image"

* commit '2eb3769cf75724ce84dcb1b11bddcc1247444799':
  Make ImageSpace skip validation if we are using a relocated image

am 1175aec5: Merge "Fix image not regenerating on changes to bootclasspath"

* commit '1175aec5f686b9e3d429282d7d4022c39436f71f':
  Fix image not regenerating on changes to bootclasspath

am 07206af3: Reduce time and memory usage of GVN.

* commit '07206af370746e6d7cf528e655b4854e7a865cfa':
  Reduce time and memory usage of GVN.

am 00d1ef41: Merge "ART: LoadConstWide should clobber temp reg"

* commit '00d1ef41d0d0d438d604d102183a6bfac58643e7':
  ART: LoadConstWide should clobber temp reg

am 70f7d48b: Merge "Reduce time and memory usage of GVN."

* commit '70f7d48b9f2ad768c1e1a5bd96979870379f05e2':
  Reduce time and memory usage of GVN.

am 36b111c7: Merge "ART: Build fix"

* commit '36b111c7d3d635e262114dabde4c26952c7dcbe6':
  ART: Build fix

am cc75c1b3: Merge "1. Fix CFI for quick compiled code in x86 & x86_64; 2. Emit CFI in .eh_frame instead of .debug_frame."

* commit 'cc75c1b38e811cc1cb93ce8f8fcc9a83e20844fb':
  1. Fix CFI for quick compiled code in x86 & x86_64; 2. Emit CFI in .eh_frame instead of .debug_frame.

am 9900a610: Merge "Add --always-clean flag to run-test."

* commit '9900a610fbb40c49b0503183b79c4bbb376c4128':
  Add --always-clean flag to run-test.

Reduce time and memory usage of GVN.

Filter out dead sregs in GVN. Reclaim memory after each LVN
in the GVN modification phase.

Bug: 16398693

(cherry picked from commit b19955d3c8fbd9588f7e17299e559d02938154b6)

Change-Id: I33c7912258a768b4c99d787056979fbc3b023b3b

am b99b8d6c: ART: Fix verifier mishandling erroneous array component types

* commit 'b99b8d6cffe08d8c9d30175c936e5c88d3101802':
  ART: Fix verifier mishandling erroneous array component types

am e34e467f: Merge "ART: Fix verifier mishandling erroneous array component types"

* commit 'e34e467f9adf59ba7febdc22b6329ef4b853e123':
  ART: Fix verifier mishandling erroneous array component types

ART: Fix verifier mishandling erroneous array component types

The verifier must not assume that component types are not erroneous.

Bug: 16661259

(cherry picked from commit aa910d5ef43256102809e397de305c23f1c315e6)

Change-Id: I6d607310593ac337616581bfdff5eb29a8dd1b9d

am 6bff7130: Add support for bump pointer spaces to DdmSendHeapSegments.

* commit '6bff7130878128c9836a21ae58d0cfdb0075b1b5':
  Add support for bump pointer spaces to DdmSendHeapSegments.

am da4a1264: Merge "Add support for bump pointer spaces to DdmSendHeapSegments."

* commit 'da4a1264cd7d5db9ed01cfd1eeaed71cb11c6c71':
  Add support for bump pointer spaces to DdmSendHeapSegments.

Add support for bump pointer spaces to DdmSendHeapSegments.

Added support for bump pointer spaces as well as
differentiating between DlMallocSpaces and RosAllocSpaces.
Added logic to reset the start of next chunk to prevent
inbetween space regions counted as free.

Fixed a debug build bug where we were doing an mprotect after
creating a rosalloc space. In debug builds, this writes a magic
value in the page. This was faulting since it was set to PROT_NONE.
The fix moves the mprotect before the RosAlloc space creation.

Bug: 16408851

(cherry picked from commit 36dab3607e77505ce139eacef1c62a1c4bc4affd)

Change-Id: I5095feb01fa7e248178a2833da7f821f4fd52c89

Move image classes options from art to AndroidRuntime [art]

Bug: 15165413
Change-Id: I3dfa702ac29279cf76d4cc451138c03a5158ac3a

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

This will allow us to conditionally change the compiler-filter based on other properties.

Bug: 15165413
Change-Id: I6877a6eab221ab142909b595fd28191b5d3a095b

Merge "ART: Build fix"

ART: Build fix

Make lint happy, as comments should be separated from code.

Change-Id: I4bfd88357302be9a6a104f1152e3b1fda386371e

Merge "ART: Fix verifier mishandling erroneous array component types"

ART: Fix verifier mishandling erroneous array component types

The verifier must not assume that component types are not erroneous.

Bug: 16661259
Change-Id: I23b2f517259ca9c0b8a1aa38f6348fcd61e0b22e

Merge "Add support for bump pointer spaces to DdmSendHeapSegments."

Add support for bump pointer spaces to DdmSendHeapSegments.

Added support for bump pointer spaces as well as
differentiating between DlMallocSpaces and RosAllocSpaces.
Added logic to reset the start of next chunk to prevent
inbetween space regions counted as free.

Fixed a debug build bug where we were doing an mprotect after
creating a rosalloc space. In debug builds, this writes a magic
value in the page. This was faulting since it was set to PROT_NONE.
The fix moves the mprotect before the RosAlloc space creation.

Bug: 16408851
Change-Id: Ibe18198d05ff353fbb57d10b2f7719fdcbf1c5f0

am 01a0ebb6: Merge "Make 32bit run-tests work correctly on 64bit default devices."

* commit '01a0ebb6b96c32879dcad99d515fc0cd8de2a266':
  Make 32bit run-tests work correctly on 64bit default devices.

am c7c6d18a: Stop thread from reattaching during runtime shutdown while tracing.

* commit 'c7c6d18a90079b67a7592d3f2cb3d4c6f0081512':
  Stop thread from reattaching during runtime shutdown while tracing.

am 78b367a6: Merge "Submitted wrong changelist!" into lmp-dev

* commit '78b367a666deb0325b5f509021b14a53625d3786':
  Submitted wrong changelist!

am fb74d873: Merge "Stop thread from reattaching during runtime shutdown while tracing." into lmp-dev

* commit 'fb74d873bde1ea396f5e8603f2c193e7b539345f':
  Stop thread from reattaching during runtime shutdown while tracing.

am 5cac343e: ART: Check for duplicate class definitions in dex file verifier

* commit '5cac343e28fe28d4bc9ccc6db5cab38db1f2b59d':
  ART: Check for duplicate class definitions in dex file verifier

am ef57ed72: Merge "ART: Remove bogus dcheck"

* commit 'ef57ed72a3ac856f6115ae89c609d4fd892e8c41':
  ART: Remove bogus dcheck

am eaa6da1b: ART: Bump oat version for embedded vtable

* commit 'eaa6da1b8381bc7cb9076e90b7ec06812b9d0d2d':
  ART: Bump oat version for embedded vtable

am 9854fe24: Fix build, missing spaces around =/<.

* commit '9854fe24d3e22a2e232597b4334ac93205ff17ed':
  Fix build, missing spaces around =/<.

ART: Correct disassembling of 64bit immediates on x86_64

The patch fixes an issue with disassembling 'movsxd' and 'movabsq'
instructions altered with 64bit immediates: not only a REX.W prefix
may be prepended to these instructions.

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

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

Change-Id: I678a541f0bcef726db44ccb50cb6feb71ec498e5

Stop thread from reattaching during runtime shutdown while tracing.

Bug: 16024763

(cherry picked from commit c5d824a20c225763761a6dff43294b229ff35469)

Change-Id: Ideab9adedabf3c58f5162e43e79c72af0642bdfa

am f90283f6: Merge "ART: Check for duplicate class definitions in dex file verifier"

* commit 'f90283f61d6ca37abf3a9fb8447d05e79caf0160':
  ART: Check for duplicate class definitions in dex file verifier