am 2a462670: am bbcd0a75: Merge "Fix heap trimmer daemon sleeping." into lmp-dev

* commit '2a4626708d2c8fa2f456021428ff008f8c907a08':
  Fix heap trimmer daemon sleeping.

am bbcd0a75: Merge "Fix heap trimmer daemon sleeping." into lmp-dev

* commit 'bbcd0a75324dd9554109d91f2e9f7ae5d4c4a28a':
  Fix heap trimmer daemon sleeping.

am 217dc450: Merge "VisitClassesWithoutClassesLock isn\'t safe if classes move."

* commit '217dc450cf0fef8539beaded77c704cfcb6669e9':
  VisitClassesWithoutClassesLock isn't safe if classes move.

Merge "Fix heap trimmer daemon sleeping." into lmp-dev

Fix heap trimmer daemon sleeping.

Problem:
Heap trimmer daemon sleeping caused app launch occasionally to take a
while due stopping the heap trimming thread taking a long time.

The heap trimmer thread now never sleeps, we prevent issues caused
by frequent back and forth process state changes by only
transitioning to background if it has been kCollectorTransitionWait
time since the last transition to background. Similar logic for heap
trimming.

Bug: 17310019

Change-Id: I23980421cc388f36b66a4c03ed15dd11d43f59b5

resolved conflicts for merge of 7b04ae29 to lmp-dev-plus-aosp

Change-Id: I14be394092fb83ef2707614e3a9822acb4306d72

Merge "VisitClassesWithoutClassesLock isn't safe if classes move."

Merge "Fix stack overflow and duplicate methods while tracing."

am 3c2559a1: am 57f2bbda: ART: Fix x86_64 GenSelect case when destination is Ref

* commit '3c2559a194fce54300e6e1d86f6c2ce57ff25e4d':
  ART: Fix x86_64 GenSelect case when destination is Ref

Fix stack overflow and duplicate methods while tracing.

Bug: 16386215
Change-Id: I0d0ae0113a3a00013ce84a1f5a110e2c52f19b86
(cherry picked from commit 6b28a456b075fe53dfb7e924a44dbf35d0d41eb3)

am 57f2bbda: ART: Fix x86_64 GenSelect case when destination is Ref

* commit '57f2bbda741d2d3d769a50268b4e0b9b54959684':
  ART: Fix x86_64 GenSelect case when destination is Ref

am 53558354: Merge "ART: Fix x86_64 GenSelect case when destination is Ref"

* commit '5355835411cafcc2772e061cb7aef3b8ca1e6ea1':
  ART: Fix x86_64 GenSelect case when destination is Ref

ART: Fix x86_64 GenSelect case when destination is Ref

Reference in x86_64 is a 64-bit solo register. As a result, the invocation
  of OpRegImm results in an error when Select opcode of the kind:
    ref = boolean ? null : null;
because opRegImm does not support 64-bit destination for OpMov.

The case above is only possible for ref because no one other constant except
  null is possible.

Bug: 17327895
Change-Id: I7541e744ec1c8619711712fd17be72764efcf3a8
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>
(cherry picked from commit 3f51e7d942c22edaab3a7e703a1e6a2dd6a26f77)

Merge "ART: Fix x86_64 GenSelect case when destination is Ref"

ART: Fix x86_64 GenSelect case when destination is Ref

Reference in x86_64 is a 64-bit solo register. As a result, the invocation
  of OpRegImm results in an error when Select opcode of the kind:
    ref = boolean ? null : null;
because opRegImm does not support 64-bit destination for OpMov.

The case above is only possible for ref because no one other constant except
  null is possible.

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

am 90fd7f57: am 607dcd8a: Merge "ART: Fix computation of frame size for direct proxy methods" into lmp-dev

* commit '90fd7f5762bfbc9009fb1acfeb7815fb2ca6cfb9':
  ART: Fix computation of frame size for direct proxy methods

am f96ad932: Merge "ART: Reduce LockCallTemps usage"

* commit 'f96ad932cf2c4f814f92ed3a8679d50aa1b02dcd':
  ART: Reduce LockCallTemps usage

am 607dcd8a: Merge "ART: Fix computation of frame size for direct proxy methods" into lmp-dev

* commit '607dcd8a2aca4e44a564b7839c285315230545fd':
  ART: Fix computation of frame size for direct proxy methods

am 2fb54e87: Merge "ART: Allow oatdump to print vr stack locations"

* commit '2fb54e875697126de075ae736fd6cb136b7b4f94':
  ART: Allow oatdump to print vr stack locations

Merge "ART: Fix computation of frame size for direct proxy methods" into lmp-dev

Merge "ART: Reduce LockCallTemps usage"

ART: Reduce LockCallTemps usage

Using FlushAllRegs/LockCallTemps in integer arithmetics causes
excess register flushing and clobbering. This patch adds API that
allows to flush, clobber and lock only those registers we really
need for calculations.

Change-Id: Idabaa4fff4d18a33e5040a80f66f2df6432f8be0
Signed-off-by: Max Kazantsev <maxim.kazantsev@intel.com>

am 328c430e: Merge "ART: Vectorization opcode implementation fixes"

* commit '328c430e923ce8d18532d0fdd26cd233e73763dd':
  ART: Vectorization opcode implementation fixes

Merge "ART: Allow oatdump to print vr stack locations"

ART: Allow oatdump to print vr stack locations

For both debugging and performance analysis, it is necessary to understand
stack layout. This patch adds capability to oatdump to print out the offsets
of the locals, ins, method*, and out VRs.

Change-Id: I73512f59e4fd2d2b12725a6c76d602182c46ff78
Signed-off-by: Razvan A Lupusoru <razvan.a.lupusoru@intel.com>

am d73f52b1: Merge "Add numerator check for integer divide and modulo"

* commit 'd73f52b1b9bea825d98830a906511a9e97ab277d':
  Add numerator check for integer divide and modulo

am bfa2b553: Merge "ART: Only call CalculateBasicBlockInformation once"

* commit 'bfa2b5537a138c15895a67c55de38bb83fcd03c3':
  ART: Only call CalculateBasicBlockInformation once

Merge "ART: Vectorization opcode implementation fixes"

Merge "Add numerator check for integer divide and modulo"

Merge "ART: Only call CalculateBasicBlockInformation once"

ART: Only call CalculateBasicBlockInformation once

CalculateBasicBlockInformation gets called twice now when dirty is true.

Change-Id: I8aa0569e4c00ea884a45ecea957a4ae06a328969
Signed-off-by: Jean Christophe Beyler <jean.christophe.beyler@intel.com>

am 7b3dd82b: Merge "ART: Fix computation of frame size for direct proxy methods"

* commit '7b3dd82bfaa2b3558587f1614144ad8ccfdbd7a9':
  ART: Fix computation of frame size for direct proxy methods

ART: Fix computation of frame size for direct proxy methods

Proxy method has only one direct method and it is a constructor which
is cloned from java.lang.reflect.Proxy class together with code.
As a result its body is a compiled quick code and frame size should be
computed accordingly in contrast with other virtual methods which are
invoked through stub.

Change-Id: I0ac99b2f567b281c9342152bad1149dd6cd39480
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>
(cherry picked from commit 805bab1738549b2477b3ad4d9d57fd7c681451b9)

Merge "ART: Fix computation of frame size for direct proxy methods"

ART: Fix computation of frame size for direct proxy methods

Proxy method has only one direct method and it is a constructor which
is cloned from java.lang.reflect.Proxy class together with code.
As a result its body is a compiled quick code and frame size should be
computed accordingly in contrast with other virtual methods which are
invoked through stub.

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

am 23628703: Merge "Fix native allocation watermark clamping."

* commit '236287039744ee1938f4a4e3fd1fe48090c359d0':
  Fix native allocation watermark clamping.

am d83f82c8: am e1a86627: Merge "Fix native allocation watermark clamping." into lmp-dev

* commit 'd83f82c8e1d934d569c868b06f03b33698616f7d':
  Fix native allocation watermark clamping.

Merge "Fix native allocation watermark clamping."

Fix native allocation watermark clamping.

The main issue causing the test to fail is that
native_footprint_gc_watermark_ becoming > growth_limit_ due to no
clamping.

Temporary runFinalization fix is calling runFinalization 2x.

Bug: 17371542

(cherry picked from commit 4c7fc5950853b0c368e2148db77ced7c4d3c303c)

Change-Id: I05b85e95560c32c33d53bc96abf87d5262007395

am e1a86627: Merge "Fix native allocation watermark clamping." into lmp-dev

* commit 'e1a8662792260953efe05fd991d945e508298aa8':
  Fix native allocation watermark clamping.

Merge "Fix native allocation watermark clamping." into lmp-dev

am badae981: Merge "Add exception check to AllocateInternalWithGc"

* commit 'badae981ada1fa279e54360df177ab6f40c299b9':
  Add exception check to AllocateInternalWithGc

Fix native allocation watermark clamping.

The main issue causing the test to fail is that
native_footprint_gc_watermark_ becoming > growth_limit_ due to no
clamping.

Temporary runFinalization fix is calling runFinalization 2x.

Bug: 17371542
Change-Id: I188cb530a44dd109e066a22091f12f8d2d4350c3

Merge "Add exception check to AllocateInternalWithGc"

am dbddcf7c: am c4978258: Merge "Add exception check to AllocateInternalWithGc" into lmp-dev

* commit 'dbddcf7ccc664c5818f2f04e87775ed0cd6fa7c5':
  Add exception check to AllocateInternalWithGc

am c4978258: Merge "Add exception check to AllocateInternalWithGc" into lmp-dev

* commit 'c4978258172beb35c3c8f0c31a5f7eb06b9a1de2':
  Add exception check to AllocateInternalWithGc

Add exception check to AllocateInternalWithGc

Trying to figure out if the pending exception comes before or during
AllocateInternalWithGc.

Bug: 17164348

(cherry picked from commit 98d43cb0da3c34d1888d1e31d953c2940fd0f897)

Change-Id: I2aac4b67f0c3f1c0fe3c9dc321316f76494c4074

Merge "Add exception check to AllocateInternalWithGc" into lmp-dev

am a07557cc: Merge "Remove abuse of mirror::Object* to reference special values."

* commit 'a07557ccece64fa7084bb01b9d26957bd0977c10':
  Remove abuse of mirror::Object* to reference special values.

VisitClassesWithoutClassesLock isn't safe if classes move.

Which they do, so avoid by doing an array allocation.
Also, tidy member variables to the end of ClassLinker.
Remove unnecessary mutable. Tidy and fix a locks required/excluded.

Change-Id: I2404a9e7a1ea997d68ab1206f97d2a20dffbda06

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

Change-Id: I30a0acb0ba2e8dd6d983e738af744397ee5e53ae

Merge "Remove abuse of mirror::Object* to reference special values."

Remove abuse of mirror::Object* to reference special values.

Remove kInvalidIndirectRefObject, kClearedJniWeakGlobal and
ObjectRegistry::kInvalidObject. Handle error conditions by passing in or
returning an error value.
GetObjectRefType is simplified to be faster and not return invalid references
that are not expected according to the spec. Adjust check JNI and
jni_internal_test appropriately.
Fix cases in the debugger/JDWP of out arguments being passed by reference.
Bug: 17376993

Change-Id: I3ce8a28c01827e163f4dc288449959464da788b1

am 6b28a456: Fix stack overflow and duplicate methods while tracing.

* commit '6b28a456b075fe53dfb7e924a44dbf35d0d41eb3':
  Fix stack overflow and duplicate methods while tracing.

Fix stack overflow and duplicate methods while tracing.

Bug: 16386215
Change-Id: I0d0ae0113a3a00013ce84a1f5a110e2c52f19b86

Add exception check to AllocateInternalWithGc

Trying to figure out if the pending exception comes before or during
AllocateInternalWithGc.

Bug: 17164348

Change-Id: Id46d82bfc0f84d16e5ffdafe41658aee4cc2c702

ART: Vectorization opcode implementation fixes

This patch fixes the implementation of the x86 vectorization opcodes.

Change-Id: I0028d54a9fa6edce791b7e3a053002d076798748
Signed-off-by: Razvan A Lupusoru <razvan.a.lupusoru@intel.com>
Signed-off-by: Udayan Banerji <udayan.banerji@intel.com>
Signed-off-by: Philbert Lin <philbert.lin@intel.com>

am b119fc31: am ff4cfe07: Fix SetupResourceMasks to pass correct parameters

* commit 'b119fc31eca8eaf9318a1a49cb10cdab03f7d1b1':
  Fix SetupResourceMasks to pass correct parameters

am ff4cfe07: Fix SetupResourceMasks to pass correct parameters

* commit 'ff4cfe07009ac0b0f07475ef2aa91d9aaf9ab291':
  Fix SetupResourceMasks to pass correct parameters

am b148ad48: am f72fd02c: Merge "Quick compiler: disable GVN DO NOT MERGE" into lmp-dev

* commit 'b148ad486920d464c0713f34e77eb53778f5d36d':
  Quick compiler: disable GVN DO NOT MERGE

am 1c9a7470: am c60bdf66: Fix native allocation test.

* commit '1c9a7470e0c2cc2b50ae48584f6a08296030d4c8':
  Fix native allocation test.

am 1e64b55d: am 4d05d931: Merge "ART: Fix read-out-of-bounds in the compiler" into lmp-dev

* commit '1e64b55d5bec5d8555bb9f3cbdaafe24b57defd7':
  ART: Fix read-out-of-bounds in the compiler

am f4bb9ef6: am 635531a2: Merge "Change native allocations to use growth limit." into lmp-dev

* commit 'f4bb9ef65cdc2c8cb8d50d023ab57320310cf96a':
  Change native allocations to use growth limit.

am 3fa54ea1: am 37bb8184: Merge "Pre-allocate the NoClassDefFoundError to be thrown for boot classes." into lmp-dev

* commit '3fa54ea15bcf6d0f79c1b04c1bd1f804a4526412':
  Pre-allocate the NoClassDefFoundError to be thrown for boot classes.

Merge '0122cf8b' "Reduce and speed-up class def searches." into lmp-dev-plus-aosp.

Conflicts:
compiler/driver/compiler_driver.cc
runtime/class_linker.cc

Change-Id: I9c98e6a34858f6f9863fc6a0df0506f3a28bd7ee

Fix SetupResourceMasks to pass correct parameters

Calling SetupTargetResourceMasks function with correct parameter.
SetupTargetResourceMasks(lir, flags, use_mask, def_mask) is correct order.

b/17369806

(cherry-picked from commit sha 2a09504334a3a3b4c47100197df0827cc6740433)

Change-Id: I49118c3f2d3d06fac83fa3d5014b7fdaff6a2b00

am f72fd02c: Merge "Quick compiler: disable GVN DO NOT MERGE" into lmp-dev

* commit 'f72fd02c4ea0f1aa610b8a4f9f51ef86c285b1cf':
  Quick compiler: disable GVN DO NOT MERGE

Merge "Quick compiler: disable GVN DO NOT MERGE" into lmp-dev

am 2a095043: Merge "Fix SetupResourceMasks to pass correct parameters"

* commit '2a09504334a3a3b4c47100197df0827cc6740433':
  Fix SetupResourceMasks to pass correct parameters

Merge "Fix SetupResourceMasks to pass correct parameters"

am 859efc4e: Merge "ART fix oat debug source map operations"

* commit '859efc4e6f5f0bba503ff55dc08a59d8ce7a81d1':
  ART fix oat debug source map operations

Merge "ART fix oat debug source map operations"

am a70343f9: Merge "Reduce lock contention when debugging"

* commit 'a70343f996aea274a2568e8f2bcda829cdbc61d7':
  Reduce lock contention when debugging

Merge "Reduce lock contention when debugging"

am dc720bf1: am e610faf9: Merge "ART: Tighten verifier list reading and offsets" into lmp-dev

* commit 'dc720bf11457c7a3d568f6e23e82586b6ce18ffd':
  ART: Tighten verifier list reading and offsets

resolved conflicts for merge of 64dc9daf to lmp-dev-plus-aosp

Change-Id: I4f69fc547e4cde43f4ab6d8080076d5b62b0fe75

Quick compiler: disable GVN DO NOT MERGE

Disable global value numbering optimization for lmp (but
keep on for master).

b/16398693

Change-Id: I125c588c987a81db54a15da1eec9dee2b009956f

am 4a946795: Merge "Quick compiler, aarch64: Insane sanity checker"

* commit '4a9467959d4693f805c39536eed3c3bc5e8bbfd8':
  Quick compiler, aarch64: Insane sanity checker

Merge "Quick compiler, aarch64: Insane sanity checker"

am 878bb62d: Merge "Fix native allocation test."

* commit '878bb62d86636b42ea0a0203945f160606adfc61':
  Fix native allocation test.

am 2cc4e2a1: Merge "Change native allocations to use growth limit."

* commit '2cc4e2a1ae9a8827503b1508dca8346a465da20a':
  Change native allocations to use growth limit.

am 47e877ab: Merge "ART: Fix StubTest Interface Trampoline test"

* commit '47e877ab2d6744f4360d37f179eb5affd6a46470':
  ART: Fix StubTest Interface Trampoline test

Merge "Fix native allocation test."

Merge "Change native allocations to use growth limit."

am c60bdf66: Fix native allocation test.

* commit 'c60bdf667b5c6fa767548942f0115547523593b4':
  Fix native allocation test.

Merge "ART: Fix StubTest Interface Trampoline test"

ART: Fix StubTest Interface Trampoline test

The imt conflict test does not actually need a proper conflict,
as we do not invoke through a lookup.

Added simple check for art_quick_invoke_interface_trampoline_with_access_check.
Only non-failure testing, as failure requires a properly setup frame etc.

Change-Id: I173dac04db950bd23f62eaf13341a7b19802e5e8

am 2013de57: Merge "Add missing read barriers to intern table."

* commit '2013de579050cff717a6fb910ec386a6e61330bc':
  Add missing read barriers to intern table.

am 579123b2: Merge "ART: Fix read-out-of-bounds in the compiler"

* commit '579123b22546d36ed47e896a567a7ca6b5470d1a':
  ART: Fix read-out-of-bounds in the compiler

am fa6fe2df: Merge "ART: Tighten verifier list reading and offsets"

* commit 'fa6fe2dfb401e30890f1feb48b664eb19636b8d4':
  ART: Tighten verifier list reading and offsets

am 62bcfec9: Merge "Change intern table to unordered set."

* commit '62bcfec936692ff4f75cdc3267c081080b0e6acc':
  Change intern table to unordered set.

am 4d05d931: Merge "ART: Fix read-out-of-bounds in the compiler" into lmp-dev

* commit '4d05d931d3d608f5047e0d9dfb422df018ace716':
  ART: Fix read-out-of-bounds in the compiler

am 635531a2: Merge "Change native allocations to use growth limit." into lmp-dev

* commit '635531a2d99f413339187ce39b08affdf1b31d9f':
  Change native allocations to use growth limit.

am 37bb8184: Merge "Pre-allocate the NoClassDefFoundError to be thrown for boot classes." into lmp-dev

* commit '37bb8184e091e8d9199cfc2fc888dfb40a5c0f20':
  Pre-allocate the NoClassDefFoundError to be thrown for boot classes.

am 9db18e16: Merge "Reduce and speed-up class def searches." into lmp-dev

* commit '9db18e1679800bc57c1e30f7c5ff786949ffe5cf':
  Reduce and speed-up class def searches.

am e610faf9: Merge "ART: Tighten verifier list reading and offsets" into lmp-dev

* commit 'e610faf917fd280fa0d9e02d5c2d0dbacbb037b7':
  ART: Tighten verifier list reading and offsets

Fix native allocation test.

Forgot to update with last CL.

(cherry picked from commit c60bdf667b5c6fa767548942f0115547523593b4)

Change-Id: I41ad168a5906ca07938bdd212a55f0e995ee5df7

Change native allocations to use growth limit.

Previously native allocation tracking used a GC footprint limit
which would cause GC in the allocating thread. This prevented
excessive growth of the heap but could cause jank due to GC in
the allocating thread. The new behavior is using the growth_limit
instead of the native footprint limit.

(cherry picked from commit d9819ecc0bc21a2bb356a4de9b013e36fe618627)

Change-Id: Ia40ed830e8c674cc49d4c0a6fd773d6cb8ff97fa

Merge "Add missing read barriers to intern table."

Fix native allocation test.

Forgot to update with last CL.

Change-Id: Id2f57870b8d4f848cd93012d9da69beee5184be4