am b68c6e57: Reduce interpret-only compile time.

* commit 'b68c6e578a28a9717d78dfd522d9d9b8befaedf2':
  Reduce interpret-only compile time.

Reduce interpret-only compile time.

Before:
39.04user 5.18system 0:29.24elapsed 151%CPU (0avgtext+0avgdata 164176maxresident)k
38.87user 5.16system 0:29.14elapsed 151%CPU (0avgtext+0avgdata 164144maxresident)k

After:
36.26user 3.25system 0:27.00elapsed 146%CPU (0avgtext+0avgdata 162592maxresident)k
36.25user 3.28system 0:26.28elapsed 150%CPU (0avgtext+0avgdata 162688maxresident)k

Disabled implicit stack protection for the compiler, this reduces page faults.

Added support for not timing every method compilation and verification. NanoTime is
slow and adds ~2 seconds of real time. This is currently enabled since people want
to know which methods are slow to compile.

Bug: 16853450

(cherry picked from commit 8e219ae27624116b6d23e858fb21e93342f81d66)

Change-Id: I349ffb3f36db8c437137387aa6914dc17d743f09

am f59b605f: Merge "Fix race condition in TransitionCollector." into lmp-dev

* commit 'f59b605f82c0e2b7bfc6019ee7bd650628f50d3a':
  Fix race condition in TransitionCollector.

am ccb1a5d4: ART: Reject field accesses to non-reference vregs.

* commit 'ccb1a5d46bc1b4ee888e74775f5c1291d600ff54':
  ART: Reject field accesses to non-reference vregs.

Merge "Fix race condition in TransitionCollector." into lmp-dev

Fix race condition in TransitionCollector.

There was a minor race condition that could happen if multiple
threads called TransitionCollector at the same time. Starting out
in CMS:

Thread 1: TransitionCollector(SS).
Thread 2: TransitionCollector(CMS) sees that it is already CMS so
has copying_transition == false. But then thread 2 changes to SS.
Thread 1 resumes but incorrectly doesn't check the
disable_moving_gc_count_ possibly resulting in errors if
disable_moving_gc_count_ != 0 due to JNI since we are going from
SS -> CMS and are goign to move objects.

Bug: 17189964

Change-Id: I4f0d002717516b81355d66f26e0e8ebe3958348d

ART: Reject field accesses to non-reference vregs.

The verifier will now reject any iget/iput insts
that do not have a reference object in vB.

Bug: 17207857
Signed-off-by: Stuart Monteith <stuart.monteith@arm.com>
(cherry picked from commit 695c5982eeddee18c643cc2f94c9d78fa75599ab)

Change-Id: I3790b8687673643bb142549d74f36a3e2978bf13

am 506bcfc6: Merge "Map heap maps at a lower address." into lmp-dev

* commit '506bcfc66454555f792fbc7893ff3b2877c9b288':
  Map heap maps at a lower address.

Merge "Map heap maps at a lower address." into lmp-dev

am b2634e97: Merge "ART: kSuper also has IncompatibleClassChangeError" into lmp-dev

* commit 'b2634e9790a7099cf84d3509afe7af18f4349f11':
  ART: kSuper also has IncompatibleClassChangeError

am 7f28597b: Merge "ART: Fix IncompatibleClassChange check for ArtMethod" into lmp-dev

* commit '7f28597bfa4b5e11911e4d4caa13fe2ab234a53e':
  ART: Fix IncompatibleClassChange check for ArtMethod

Merge "ART: kSuper also has IncompatibleClassChangeError" into lmp-dev

Merge "ART: Fix IncompatibleClassChange check for ArtMethod" into lmp-dev

am 83182ed4: Merge "Reduce lock contention when debugging" into lmp-dev

* commit '83182ed48c55729cd34780fc598f269cc98af739':
  Reduce lock contention when debugging

Merge "Reduce lock contention when debugging" into lmp-dev

am cb50ed40: Merge "Quick compiler: fix DCHECK" into lmp-dev

* commit 'cb50ed40fc39a1c1786e57f5c839a3cd230e1744':
  Quick compiler: fix DCHECK

Merge "Quick compiler: fix DCHECK" into lmp-dev

am 118785c6: CheckAndClearResolveException should allow LinkageError

* commit '118785c6f82ac1ec7074b5efe367b3dafd904940':
  CheckAndClearResolveException should allow LinkageError

CheckAndClearResolveException should allow LinkageError

Bug: 16565964
Change-Id: Ibf236c627519a5cdef9479b7bc21e931d5474abf

ART: kSuper also has IncompatibleClassChangeError

When resolving methods, kSuper can also have
IncompatibleClassChangeError.

Bug: 17266767
Change-Id: I5fafe03ad578f605825a3d0c89f1254a4a385b76

ART: Fix IncompatibleClassChange check for ArtMethod

In case of kSuper we should complain sometimes.

Bug: 17265201
Change-Id: Ifad903a750495963c7128296713fbd7c4add71cd

am eac98d7c: Merge "Reduce GC log spam." into lmp-dev

* commit 'eac98d7cb0f84c664d22deebf1ae5062730d7a31':
  Reduce GC log spam.

Merge "Reduce GC log spam." into lmp-dev

Reduce GC log spam.

Homogeneous compaction and collector transitions now print to
VLOG(gc) instead of LOG(INFO).

Bug: 17166556
Change-Id: I5d54acf36a9b2763922e6329596a2f191d45535c

Map heap maps at a lower address.

We now map the heap mmaps at a lower address in case the app needs
larger continuous address space.

Bug: 16502380

Change-Id: I2cc11d0c207c0eae8db0c5025a48b11119c5a802

Quick compiler: fix DCHECK

A DCHECK intended to verify the soundness of the mapping between
a wide Dalvik value and a pair of temp registers incorrectly fired
when the temp register pair was associated with a wide temporary
(i.e. - when (SReg() == INVALID_SREG)).

In this particular situation, the wideness flag is meaningful only when
there is an associated valid vreg.  Code rearranged to only perform
the DCHECK and reset the partner's wideness bit when we're dealing with
a live wide value.

b/16484538

Change-Id: I3b8a7bebc5d918c65ea56ae8db0a8a5cd8003386

am 0699fbf7: Reduce log spam by changing a LOG(INFO) to VLOG(startup)

* commit '0699fbf769befd3a850005790b26941c38fa2fb4':
  Reduce log spam by changing a LOG(INFO) to VLOG(startup)

am 507e6180: Support running without a boot image.

* commit '507e6180ad271eb719c67ce7394852c731d975a5':
  Support running without a boot image.

Reduce log spam by changing a LOG(INFO) to VLOG(startup)

Bug: 17166556

(cherry picked from commit b6cabc1345ec307559a6e85141fb69b7caa9413c)

Change-Id: I9975ba07ad8f119a6e367e2770a60040a462149b

Support running without a boot image.

Bug: 17000769

(cherry picked from commit 64ad14dbe2225441fb7734bf6d89358d96692eea)

Change-Id: I6404d5050c8a2f4ee6e70d58532eb25ee9de248e

am 73aa0b03: Merge "Fix GC memory overhead accounting." into lmp-dev

* commit '73aa0b03a56a19b73c51136c32624bc5069782e0':
  Fix GC memory overhead accounting.

Merge "Fix GC memory overhead accounting." into lmp-dev

am 83a2192d: Merge "Add fallback for non moving space being full." into lmp-dev

* commit '83a2192d360cdf678c9d4422b5f1bc73d5df602b':
  Add fallback for non moving space being full.

Merge "Add fallback for non moving space being full." into lmp-dev

Add fallback for non moving space being full.

When the non moving space becomes full, we disable moving GC and
make the main space the new non moving space.

Also added a runtime option for changing the non moving space size:
-XX:NonMovingSpaceCapacity.

Bug: 17189964
Change-Id: I04d00d0f457f6c1f5724bf32932b6b6ce4d341b1

am 1e4fbd8b: Merge "Using the correct time for wait_start_ms when lock contention logging is enabled" into lmp-dev

* commit '1e4fbd8b7d65064d087c5afe812943b203ed205a':
  Using the correct time for wait_start_ms when lock contention logging is enabled

Merge "Using the correct time for wait_start_ms when lock contention logging is enabled" into lmp-dev

Using the correct time for wait_start_ms when lock contention logging is enabled

Bug: 17208457
Change-Id: I35555e0acf3b37bb3776a62c64d59ae93403b237

Fix GC memory overhead accounting.

There was some missing null checks.

Bug: 16238192

Change-Id: Iaf8d752db5f21e76f668c0066a063239ff374eee

am 1f8ea040: Merge "ART: Allow uninitialized runtime in oat code" into lmp-dev

* commit '1f8ea0402915b394461bb6a3ca58e05c62f63082':
  ART: Allow uninitialized runtime in oat code

Merge "ART: Allow uninitialized runtime in oat code" into lmp-dev

am fd684f6d: Merge "Always setup the native bridge library" into lmp-dev

* commit 'fd684f6d51cf4bee6e7bf62c9d632fbebd7dd8b7':
  Always setup the native bridge library

am 84a9ba10: Merge "Fix testing wrong variable for non zygote." into lmp-dev

* commit '84a9ba10a826b8480be53ea164fc6168544b2a48':
  Fix testing wrong variable for non zygote.

Merge "Always setup the native bridge library" into lmp-dev

Merge "Fix testing wrong variable for non zygote." into lmp-dev

Fix testing wrong variable for non zygote.

For non zygote we want to only have a single non moving space to
let dex2oat use RosAlloc for all allocations.

Fixed stale low memory logic in parsed options.

Bug: 17189964
Change-Id: I85a565002fd18ae30105df084796521abd60ea12

Reduce lock contention when debugging

Uses a ReaderWriterMutex for the breakpoint lock to reduce contention during
debugging session.

Also adds missing thread safety annotations on fields and methods related to
instrumentation and debugging.

Bug: 16814665
Bug: 11667502
Change-Id: I056cdafa91109e0c83806c8d8df75c37ade0a354

Always setup the native bridge library

... even if the string is empty. This will initialize the native bridge
library but mark it as unavailable.

- also, rename native_bridge_library_path to
native_bridge_library_filename to be closer to the actual meaning (it's
just the filename without any path).

Bug: 16404669.
Change-Id: I94628639691459d48d1fbf0841f36b68d51818e7

am e246f336: Fixed wrong field usage in Instrumentation::RemoveListener

* commit 'e246f33605a2d6b96f2c7221fa0165dc067c3079':
  Fixed wrong field usage in Instrumentation::RemoveListener

am 96add978: Optimized instrumentation listener handling

* commit '96add97815ba8418fb4e8c0fc08dbf9c7198f244':
  Optimized instrumentation listener handling

ART: Allow uninitialized runtime in oat code

Oatdump does not initialize a runtime. Avoid a segfault.

Bug: 17194559
Change-Id: I1a1bd3fc7762ff4714251617a975e832522fa879

Fixed wrong field usage in Instrumentation::RemoveListener

Signed-off-by: Daniel Mihalyi <daniel.mihalyi@mattakis.com>
Bug: 16814665

(cherry picked from commit 6644521cfaff718a55e3602efb3b54bf19a0cd1a)

Change-Id: Ie68c70ba6ee2e0bd918717c8659c057f98e66ff7

Optimized instrumentation listener handling

Some instrumentation listener lists may be modified while iterating
over the list to deliver an instrumentation event. Therefore the
previous implementation copied the list of listeners before starting
the iteration.

This new implementation only copies the list of instrumentation
listeners when the list is changed. Instances of the list are
reference counted using std::shared_ptr<>.

Bug: 16814665

(cherry picked from commit ca1d06cfa2f2b8d2be4390644e126cb68cdbb5ba)

Change-Id: Ib2e6b980de85b75f1c8f4a8825bdc7767154663a

am 39c2a379: Merge "Fix fugu build" into lmp-dev

* commit '39c2a379795460374b41e627257130e3c4ab961f':
  Fix fugu build

Merge "Fix fugu build" into lmp-dev

Fix fugu build

Bug: 17188805
Change-Id: Ibbfc759b42ec0f08a1beb35bf52d3ca92b8fe1ea

am 574758ef: Merge "Make a couple of map checks debug only." into lmp-dev

* commit '574758ef03b0930ebd1cb8d9afe741cb374419a1':
  Make a couple of map checks debug only.

Merge "Make a couple of map checks debug only." into lmp-dev

Make a couple of map checks debug only.

This cost us close to 80ms in app startup times.

The checks that a reused region was within an already
existent map has been demoted to a debug check.

A couple of other negative checks have been removed
outright because one of them was superflous and the other
wasn't guaranteed to be correct.

bug: 16828525
Change-Id: I00f76de06df0ea4ced40fdcb7825248d4b662045

am b9c38883: Remove profiler log line

* commit 'b9c3888380666a7b44718f04f787693787cd57c6':
  Remove profiler log line

Remove profiler log line

This removes a LOG(INFO) saying that the profile is disabled.

Bug: 17166556
Change-Id: I4ddb4b0f792eb60ed5c927e17d06cff4ae0c073f

am 9beaac31: Merge "Check pending exception result in AllocObjectWithAllocator." into lmp-dev

* commit '9beaac31f89ad9f47c6e99e81cbfbefa25940c1d':
  Check pending exception result in AllocObjectWithAllocator.

am 03324d0e: Merge "Fix fault handler to unregister on shutdown" into lmp-dev

* commit '03324d0e4b983863a75800d9973fcb7a1eb809b5':
  Fix fault handler to unregister on shutdown

am 45c3ea10: Merge "Add supported architectures to signal test." into lmp-dev

* commit '45c3ea10b918ed8bb6834a1f4c2356b5874a8c75':
  Add supported architectures to signal test.

Merge "Check pending exception result in AllocObjectWithAllocator." into lmp-dev

Merge "Fix fault handler to unregister on shutdown" into lmp-dev

Merge "Add supported architectures to signal test." into lmp-dev

Check pending exception result in AllocObjectWithAllocator.

Possible previous bug:
Allocation fails due to OOM and the collector transitions.
This caused us to incorrectly retry the allocation with a pending
exception. We now return null if there is a pending exception.

Bug: 17164348
Change-Id: I22eab472afb2fdea6e800963ccb35ec0755ba0e6

Fix fault handler to unregister on shutdown

This fixes a problem with the fault handler where it wasn't
unregistering itself during shutdown of the runtime.

Bug: 17133266
Change-Id: If4758b14ded0fba194897191b1a8d8d7b0b3e6e5

am c7fa9e30: Merge "Use homogeneous space compact by default if not low memory." into lmp-dev

* commit 'c7fa9e3018434eed890034b14bd85645077ec2e9':
  Use homogeneous space compact by default if not low memory.

am 16dd4dfd: Merge "ART: Relax GetInstructionSetFromString" into lmp-dev

* commit '16dd4dfd01c0ac432a32d081a15205222ffb1f1c':
  ART: Relax GetInstructionSetFromString

Add supported architectures to signal test.

This adds ARM, AARCH64, i386 and x86_64 to the supported architectures
in 004-SignalTest

Bug: 16948053
Change-Id: I82b3f053e9340b9e2e8fc04775d03fb0592fa06e

am 167350d9: Add null check to CheckVirtualMethod

* commit '167350d9c781c5e3893714bb26ab5cb1c4abf6b4':
  Add null check to CheckVirtualMethod

Add null check to CheckVirtualMethod

There was a runtime SIGSEGV that should have been a check jni
failure.

Also added regression test.

Bug: 16320699
Change-Id: If6c8e73959cefb24e4703f1562cdddb343d86630

am a846c1ac: Merge "X86: Fix alias info in GenInlinedIndexOf()." into lmp-dev

* commit 'a846c1ac65b197844380b8d28377a850dfa1fc94':
  X86: Fix alias info in GenInlinedIndexOf().

am c6a62592: Merge "Quick compiler: fix Arm64 inline locking code" into lmp-dev

* commit 'c6a62592836612b09ff1a8021e49b1328d85493b':
  Quick compiler: fix Arm64 inline locking code

Merge "Quick compiler: fix Arm64 inline locking code" into lmp-dev

Quick compiler: fix Arm64 inline locking code

The Arm64 monitor enter code was incorrectly comparing the
thread ID to zero rather than the lock value to determine if the
inline fast path was doable.  As a result, we ended up always
using the out-of-line case.

This fix delivers a 6% improvement to DeltaBlue.

b/16707253

Change-Id: I499530da84beecf1f3f5c5ebb666e3eb12599ac0

am d6ee5033: Merge "Make apps able to run with a failing patchoat" into lmp-dev

* commit 'd6ee5033433619a893a8f20319efcf8c8b1d1016':
  Make apps able to run with a failing patchoat

am 20e4afeb: Merge "Workaround problem reading main stack on intel devices." into lmp-dev

* commit '20e4afeb1ad9d142fe9e46efe812f0d562c28f1f':
  Workaround problem reading main stack on intel devices.

am 38bd1e1b: ART: Fix ARM invoke stub, run Generic JNI in jni_compiler_test

* commit '38bd1e1b8aa39bdc6e67c5dc205cc8d294d985c5':
  ART: Fix ARM invoke stub, run Generic JNI in jni_compiler_test

Merge "Workaround problem reading main stack on intel devices." into lmp-dev

ART: Fix ARM invoke stub, run Generic JNI in jni_compiler_test

On devices which support it, run all jni compiler tests also with
the Generic JNI infrastructure.

Fix art_quick_invoke_stub for ARM32 to enforce 16B stack alignment.
As the native convention does not require as much, the previous
code would keep it unaligned.

Bug: 17107929

(cherry picked from commit 6e498695b60f1532d2264ec6badb1cd6e10ecaa9)

Change-Id: I1481500ff7c627ab4e5bfbfd7245099a639aa049

Workaround problem reading main stack on intel devices.

This works around a problem where ART can't read the lowest page
of the main stack.  The workaround is to add 4K to the stack
start address on the main stack on Intel.

Please see https://b2.corp.google.com/issues/17111575 for
underlying cause.

Bug: 17031544
Change-Id: Ifc3216e10160bb1eec9d989fd57b10345bc89537

am e22c03ce: Merge "ART: Set default visibility to protected" into lmp-dev

* commit 'e22c03ceaa847e3b6c8fcb7ee9d838f61945cda7':
  ART: Set default visibility to protected

am 5afe466c: Check for constructor first when installing tracing stubs.

* commit '5afe466c027c6ac821bf7c9a88cd7de5260a639b':
  Check for constructor first when installing tracing stubs.

Check for constructor first when installing tracing stubs.

Addresses comments in
https://android-review.googlesource.com/#/c/104383/ made after the commit.

Bug: 16386215
Change-Id: I0e59b8507f601fd6508fbda8d24dd3c98ead1537

Merge "ART: Set default visibility to protected" into lmp-dev

am 214f4f15: Merge "Fix JDWP event races" into lmp-dev

* commit '214f4f15fe0137c285a05b3f70e52a211aecfdb5':
  Fix JDWP event races

Merge "Use homogeneous space compact by default if not low memory." into lmp-dev

Use homogeneous space compact by default if not low memory.

Otherwise we use semispace as the background collector.

Bug: 16401001
Change-Id: I7b20e0888695521604758c98d09f0709fa2158ef

Merge "ART: Relax GetInstructionSetFromString" into lmp-dev

ART: Relax GetInstructionSetFromString

Do not abort on an unknown instruction set string. Instead return
kNone and let the caller handle this.

Also simplify the patchoat tool to use this.

Bug: 17136416
Change-Id: I24131914bcf91c04ae93179bf809a2907f1f2b7a

Merge "Make apps able to run with a failing patchoat" into lmp-dev

Make apps able to run with a failing patchoat

Bug: 17000769

(cherry picked from commit 9dcc4572949f6a8231a1b4ed859676ba6f411726)

Change-Id: I0a1a4dc7f5d4bb268530840302ecfb1555231e05

am 5a001b3e: Merge "AttachCurrentThread should only warn on missing thread name if CheckJNI is enabled." into lmp-dev

* commit '5a001b3e4a4b4d066fd987071eb38b2b97493807':
  AttachCurrentThread should only warn on missing thread name if CheckJNI is enabled.

am 6724d864: Fix heap corruption and verification.

* commit '6724d8649ab73e4fb86c8014bda51b13bddf2f3f':
  Fix heap corruption and verification.

Fix heap corruption and verification.

There was a faulty assumption that space End() was always aligned to
card boundaries. This was true for all spaces other than the image
and resulted in heap corruption when the last object of the image
space contained a reference to an object in another space.

Also fixed an error where we called the pre GC verification post GC.

Bug: 17080623

Change-Id: I041ee564518f53b79c6e8dc2ad782a3152577a4e

am dfa7a6c8: Disable load/store elimination. DO NOT MERGE

* commit 'dfa7a6c8711623bf20aad58f57ae980b497ef4d7':
  Disable load/store elimination. DO NOT MERGE

Disable load/store elimination. DO NOT MERGE

The optimization has several issues and it has been
effectively disabled by
    https://android-review.googlesource.com/103511
Disable it completely to avoid wasting compilation time.

Bug: 16862804
Change-Id: I42f69ae64170dc44f5fd6455b6a51c9bd3f93672