am 31029988: Merge "ART: Make a soft failure in method verifier hard, add check"

* commit '31029988d48074b4193e44856eb863305a2658fb':
  ART: Make a soft failure in method verifier hard, add check

am 9cd20b21: Merge "ART: Use array-inl.h in reference_table_test"

* commit '9cd20b21c78a85c0ec01a6e282d23a2b2b5df0c5':
  ART: Use array-inl.h in reference_table_test

am b12d6055: Merge "Fix two read barrier bugs introduced in CL 97882."

* commit 'b12d6055432f6974317ca1b5c9260db0d410df3e':
  Fix two read barrier bugs introduced in CL 97882.

Merge "Fix two read barrier bugs introduced in CL 97882."

Fix two read barrier bugs introduced in CL 97882.

Bug: 12687968
Change-Id: Ib28da4c33977cc58b09913ef5b738dec75365714

Merge "ART: Make a soft failure in method verifier hard, add check"

ART: Make a soft failure in method verifier hard, add check

Make the soft error of using a primitive type in an instanceof a
hard verifier error.

Add a check when doing check_cast peep-hole optimization to check
whether the cast type is valid.

Bug: 16020300
Change-Id: Icaea560d17804f24a13dc09c115e9ec7039cb472

Merge "ART: Use array-inl.h in reference_table_test"

ART: Use array-inl.h in reference_table_test

The test allocates arrays, which is an inline definition.

Change-Id: I1423c5419949d7c352ed0e614d4f9c5920831deb

am 031ddea2: Merge "Fix identifier size in the heap dump."

* commit '031ddea20cb311dfdb3bd16a13750f9cb426b299':
  Fix identifier size in the heap dump.

am d07c9cdd: ART: Lock an arg reg in GenInlinedGet

* commit 'd07c9cddccd0476456a5340b716bce3f107013b0':
  ART: Lock an arg reg in GenInlinedGet

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

Change-Id: I5e379c083a08051909a8ddf5ff1c158ce9e7ee9a

Merge "Fix identifier size in the heap dump."

Fix identifier size in the heap dump.

We're using addresses as IDs and previously we would have written
sizeof(void*) as the identifier size in the header. Heap references
are actually uint32_t so writting 4 bytes should be enough.

Bug: 16016862
Change-Id: I785abf5030681b6009370b7b2d7552e8b4b54e67

am 6dfa4188: Merge "Avoid direct use of QuasiAtomic by using Atomic."

* commit '6dfa4188d5cbe7cb1c89011cda80e1c261ae30f0':
  Avoid direct use of QuasiAtomic by using Atomic.

am fb8a07bd: Merge "ART: Refactor GenSelect, refactor gen_common accordingly"

* commit 'fb8a07bdf92ab097c1d309a8a6b70dacc81f4478':
  ART: Refactor GenSelect, refactor gen_common accordingly

Merge "Avoid direct use of QuasiAtomic by using Atomic."

Avoid direct use of QuasiAtomic by using Atomic.

Change-Id: I92d0f822cddb2085de7cbb4c44c3a429081892a2

ART: Lock an arg reg in GenInlinedGet

Must lock an explicit target register so that it doesn't get used
for a temp and clobbered.

Bug: 16352802

(cherry picked from commit 30ab8a8a6597962adf63f3a438ec34b754cc117e)

Change-Id: Ie478ee42732d439c2a41aac48d15e3c2e71e3fee

Revert "Revert "Revert "Revert "Add intrinsic for Reference.get()""""

Fixed TargetReg issue causing build failure for x86.
This reverts commit 9e82bd3f0ce9e5f5777bea2f752ff3e251d32f9f.

(cherry picked from commit 4ee7a665e7f9cd2c5ace2d6304e33f64067b209f)

Change-Id: I555f4e06955711262e6b37ffbeabee9698ec695c

am 62327cda: Fix class initialization checks in interpreter

* commit '62327cda9a75e9d15daab0749b37257a146631f4':
  Fix class initialization checks in interpreter

am be0562fb: Disable adding main and non moving spaces to immune region in GSS

* commit 'be0562fb14e6754ee932b8d9c97e2a6df3a91119':
  Disable adding main and non moving spaces to immune region in GSS

am de0aafe2: Merge "Remove some to-done TODOs."

* commit 'de0aafe25a25455cbe930b46a43904125bfa6299':
  Remove some to-done TODOs.

am 18179818: Merge "ART: Lock an arg reg in GenInlinedGet"

* commit '1817981832a531580c83f7d71f745da4e06889c3':
  ART: Lock an arg reg in GenInlinedGet

am b2a59010: Merge "Fix lint errors."

* commit 'b2a59010b787bd9d5d9bf36d32682faa5ad8da24':
  Fix lint errors.

am b0777f04: Merge "Use the thumb2 assembler for the optimizing compiler."

* commit 'b0777f04c9d79a43c2e490716fe4300f1fb68df7':
  Use the thumb2 assembler for the optimizing compiler.

am 3a07c04d: Merge "Fix test 304-method-tracing failure for Mips."

* commit '3a07c04d2e6feec188994f73a3e6f48c6c4c49d6':
  Fix test 304-method-tracing failure for Mips.

Fix class initialization checks in interpreter

Check field's class initialization after handling null pointer exception.

Bug: 16324235

(cherry picked from commit 1edbd8e5405486a96d69481f5f30a8ca16d510b5)

Change-Id: I31ef07a08ff70c5695dda1079afea5b2579bb8e3

Merge "Fix lint errors."

Fix lint errors.

Change-Id: I7bbaad590d84db97acfa7f8c4d093bc40e64684a

Merge "Use the thumb2 assembler for the optimizing compiler."

am 81457a3c: Merge "Add back a deleted check related to verification."

* commit '81457a3cd8fca14396b5785a4e4c8070c259b07a':
  Add back a deleted check related to verification.

am 233b0e37: Merge "Remove errors/warnings from tautological comparisons."

* commit '233b0e37b463ba3e880262f6fea255b95f0f89c1':
  Remove errors/warnings from tautological comparisons.

Merge "ART: Refactor GenSelect, refactor gen_common accordingly"

ART: Refactor GenSelect, refactor gen_common accordingly

This adds a GenSelect method meant for selection of constants. The
general-purpose GenInstanceof code is refactored to take advantage of
this. This cleans up code and squashes a branch-over on ARM64 to a
cset.

Also add a slow-path for type initialization in GenInstanceof.

Change-Id: Ie4494858bb8c26d386cf2e628172b81bba911ae5

Merge "ART: Lock an arg reg in GenInlinedGet"

ART: Lock an arg reg in GenInlinedGet

Must lock an explicit target register so that it doesn't get used
for a temp and clobbered.

Bug: 16352802
Change-Id: I9e4ac76f92460cdbe59392989d12f8d0321cc624

Merge "Remove some to-done TODOs."

Remove some to-done TODOs.

Change-Id: I976d35b1aa79f4dded86502a18bc679ba232d664

Merge "Add back a deleted check related to verification."

am 9791bb42: Merge "Fix art test failures for Mips."

* commit '9791bb427fd812c1268edab6fb3ac7b82ad9fb93':
  Fix art test failures for Mips.

Merge "Fix test 304-method-tracing failure for Mips."

Fix test 304-method-tracing failure for Mips.

Change-Id: Id5ce9fd3c880dd568b11a1deea7e42ea587934ba
Signed-off-by: Douglas Leung <douglas@mips.com>

Merge "Remove errors/warnings from tautological comparisons."

am ebe8f799: Merge "Revert "Revert "Revert "Revert "Add implicit null and stack checks for x86"""""

* commit 'ebe8f799c5053f85adffee23cb46851efebbfd92':
  Revert "Revert "Revert "Revert "Add implicit null and stack checks for x86""""

am 403ef95b: Merge "Add another test to the trace test black list."

* commit '403ef95be8c2a91bf73b133cede8ecd2a146cec6':
  Add another test to the trace test black list.

am 479f131d: Merge "Disable adding main and non moving spaces to immune region in GSS"

* commit '479f131d4bd3829dd512312020808b05f5a591f1':
  Disable adding main and non moving spaces to immune region in GSS

Disable adding main and non moving spaces to immune region in GSS

Disabled adding the main and non moving space to the immune region.
This will enable us to recycle bump pointer spaces for malloc space
-> malloc space compaction as well as collector transitions.

Also added logic for falling back to the non moving space, we may
copy objects there.

Refactored mod union table logic into MarkReachableObjects.

No measurable performance benefit or regression.

Bug: 14059466
Bug: 16291259

(cherry picked from commit 4c13a3ff475f206c4d0a86ee2595c45392fd942f)

Change-Id: I858b4fbddca888e164052ad247565a0bdbea68b5

am e66f6318: Merge "Fix x86 build."

* commit 'e66f6318635c49313f796e62170fa8f59340e926':
  Fix x86 build.

am 792f5967: Merge "Fix class initialization checks in interpreter"

* commit '792f596728d6bfb78546020a326927285aeeaad4':
  Fix class initialization checks in interpreter

am aab012d6: Merge "Add --trace variants of the run-test testing rules."

* commit 'aab012d6196bd29b3167963ec8acb0b9780672b2':
  Add --trace variants of the run-test testing rules.

Add back a deleted check related to verification.

Change-Id: Ib939d1a7395a4b6cc9a7256cfb0a3498ba296744

Use the thumb2 assembler for the optimizing compiler.

Change-Id: I2b058f4433504dc3299c06f5cb0b5ab12f34aa82

Merge "Fix art test failures for Mips."

Merge "Revert "Revert "Revert "Revert "Add implicit null and stack checks for x86"""""

Revert "Revert "Revert "Revert "Add implicit null and stack checks for x86""""

This reverts commit 0025a86411145eb7cd4971f9234fc21c7b4aced1.

Bug: 16256184
Change-Id: Ie0760a0c293aa3b62e2885398a8c512b7a946a73

Merge "Disable adding main and non moving spaces to immune region in GSS"

Disable adding main and non moving spaces to immune region in GSS

Disabled adding the main and non moving space to the immune region.
This will enable us to recycle bump pointer spaces for malloc space
-> malloc space compaction as well as collector transitions.

Also added logic for falling back to the non moving space, we may
copy objects there.

Refactored mod union table logic into MarkReachableObjects.

No measurable performance benefit or regression.

Bug: 14059466
Bug: 16291259

Change-Id: If663d9fdbde943b988173b7f6ac844e5f78a0327

am ea1a9bdc: Merge "Remove object_utils.h."

* commit 'ea1a9bdce403524d19f80095e7d51d5f714f20ff':
  Remove object_utils.h.

am a4fe789a: Merge "Make ART fail gracefully when it can\'t update the desired code."

* commit 'a4fe789a74d2eb0f1d14ab971103feacef02620c':
  Make ART fail gracefully when it can't update the desired code.

Remove errors/warnings from tautological comparisons.

Change-Id: I2728b08d55abccd14c111c6c9da93068d4e14275

am b960cd73: Use sched_yield in Monitor::MonitorEnter.

* commit 'b960cd7342cd68a614bb0c54d0d676cc2b8caf29':
  Use sched_yield in Monitor::MonitorEnter.

am fbde4dd1: Merge "Use sched_yield in Monitor::MonitorEnter."

* commit 'fbde4dd1cb6db729e3f3ee5bdae0cdd824d73054':
  Use sched_yield in Monitor::MonitorEnter.

Merge "Add another test to the trace test black list."

Add another test to the trace test black list.

Change-Id: Ib2266b4fa2684e477791fe317a598c22d79e3400

Use sched_yield in Monitor::MonitorEnter.

Previously we used NanoSleep(1000), but this was unreliable. It could
result in waiting for >= 40ms instead of 1us. Since this was in a loop
it was especially bad if the GC was trying to suspend all the
threads when we were sleeping. This resulted in thread suspension
occasionally taking longer than a second.
Results on the provided picasso-sample app on Nexus 5:
Longest GC pause before: ~1.5s.
Longest GC pause after: <5ms.

Also added a warning if thread suspension takes longer than a
threshold (currently 5ms).

Bug: 16307460
External bug: https://code.google.com/p/android-developer-preview/issues/detail?id=367

(cherry picked from commit 251755cd511463260e60be98bf138b6aa1c14bf3)

Change-Id: I491c0799cc23ee6e68abc1bf77d1f0f4263e8ef5

Fix art test failures for Mips.

This patch fixes the following art test failures for Mips:
003-omnibus-opcodes
030-bad-finalizer
041-narrowing
059-finalizer-throw

Change-Id: I4e0e9ff75f949c92059dd6b8d579450dc15f4467
Signed-off-by: Douglas Leung <douglas@mips.com>

Merge "Make ART fail gracefully when it can't update the desired code."

Make ART fail gracefully when it can't update the desired code.

ART was exiting with a fatal error when it couldn't clean an obsolete
file. Relaxing this and failing gracefully preserves the behaviour that
Dalvik had.

Bug: 15313272
Change-Id: I8d0d6d374c90d2a434909dd4ae56f0799f30134d

Merge "Fix class initialization checks in interpreter"

Fix class initialization checks in interpreter

Check field's class initialization after handling null pointer exception.

Bug: 16324235
Change-Id: I33d537168e068b7fb51ddf97bc5aadee9dd65f67

am ebaca192: Merge "Revert "Revert "Revert "Revert "Add intrinsic for Reference.get()"""""

* commit 'ebaca192314e21d26b97646fa962e468ff07b893':
  Revert "Revert "Revert "Revert "Add intrinsic for Reference.get()""""

am 0e359ef6: Merge "Break apart header files."

* commit '0e359ef64546e472ed7e2292c8a48354f9907a37':
  Break apart header files.

Merge "Add --trace variants of the run-test testing rules."

Add --trace variants of the run-test testing rules.

Only for host and the default compiler at the moment.
Also, fix test-art-host32 and test-art-host64.

Change-Id: Ic13190e766aca522de95d2b5b12926c906ba8fb8

Merge "Fix x86 build."

Fix x86 build.

Also fix attributes/annotalysis on entrypoint_utils functions now we have
clang that is smarter wrt warnings than GCC.

Change-Id: I69257b4ad9a27d07acbc973d21a1cfa4260a8ed6

Merge "Use sched_yield in Monitor::MonitorEnter."

Use sched_yield in Monitor::MonitorEnter.

Previously we used NanoSleep(1000), but this was unreliable. It could
result in waiting for >= 40ms instead of 1us. Since this was in a loop
it was especially bad if the GC was trying to suspend all the
threads when we were sleeping. This resulted in thread suspension
occasionally taking longer than a second.
Results on the provided picasso-sample app on Nexus 5:
Longest GC pause before: ~1.5s.
Longest GC pause after: <5ms.

Also added a warning if thread suspension takes longer than a
threshold (currently 5ms).

Bug: 16307460
External bug: https://code.google.com/p/android-developer-preview/issues/detail?id=367

Change-Id: I3c2a9636357e255f38634615101eff8ca84e632f

Merge "Remove object_utils.h."

Remove object_utils.h.

Break into object_lock, field_helper and method_helper.
Clean up header files following this.
Also tidy some of the Handle code in response to compiler errors when resolving
the changes in this CL.

Change-Id: I73e63015a0f02a754d0866bfaf58208aebcaa295

am 76e6773d: Merge "AArch64: improve usage of TargetReg() and friends."

* commit '76e6773dfed9e9bf382bbb8d6c8654525fa44b0c':
  AArch64: improve usage of TargetReg() and friends.

am 524e5e38: Merge "ART: Squash a cmp w/ zero and b.ls to cbz (ARM/ARM64)"

* commit '524e5e3815d2b16ee77beda9976b7ec3aa54aba6':
  ART: Squash a cmp w/ zero and b.ls to cbz (ARM/ARM64)

am 07b84413: Merge "Fix a braino in the stack layout."

* commit '07b8441303ea82fca3cb85d71ecf8752d73cedd7':
  Fix a braino in the stack layout.

Merge "ART: Squash a cmp w/ zero and b.ls to cbz (ARM/ARM64)"

ART: Squash a cmp w/ zero and b.ls to cbz (ARM/ARM64)

In case of array bounds checks at constant index 0 we generate a
compare and a branch. Squash into a cbz.

Change-Id: I1c6a6e37a7a2356b2c4580a3387cedb55436e251

Merge "Revert "Revert "Revert "Revert "Add intrinsic for Reference.get()"""""

Revert "Revert "Revert "Revert "Add intrinsic for Reference.get()""""

Fixed TargetReg issue causing build failure for x86.
This reverts commit 9e82bd3f0ce9e5f5777bea2f752ff3e251d32f9f.

Change-Id: I7e6a526954467aaf68deeed999880dfe9aa5f06e

Merge "Break apart header files."

Break apart header files.

Create libart-gtest for common runtime and compiler gtest routines.
Rename CompilerCallbacksImpl that is quick compiler specific.
Rename trace clock source constants to not use the overloaded profiler term.

Change-Id: I4aac4bdc7e7850c68335f81e59a390133b54e933

Merge "AArch64: improve usage of TargetReg() and friends."

AArch64: improve usage of TargetReg() and friends.

TargetReg(arg1) does now always return a 32-bit register. We also avoid
using this function directly and rather use the two-arguments overload
or TargetPtrReg().

Change-Id: I746b3c29a2a2553b399b5c3e7ee3887c7e7c52c3

am 59fe711f: Fix infinite loop when calling SetStatus after OOM.

* commit '59fe711f88191cd8ca1a386c4fa0d2f9e484af50':
  Fix infinite loop when calling SetStatus after OOM.

am 635561b8: Merge "Add read barriers for the roots in Runtime."

* commit '635561b86ac03f5562bdb779baa6db12f31b3cae':
  Add read barriers for the roots in Runtime.

Merge "Fix a braino in the stack layout."

Fix a braino in the stack layout.

Also do some refactoring to have this code be just in CodeGenerator.

Change-Id: I88de109889138af8d60027973c12a64bee813cb7

am 63c8d355: am 0b316726: am cf90ba7e: Merge "ART: Add another special case to GenSelect for ARM64"

* commit '63c8d355a13f62a1e4becb04c8f2cc0eccf849d0':

am c9d58c54: am ccdaa5ea: am 6ef2aa67: Merge "ART: Rework TargetReg(symbolic_reg, wide)"

* commit 'c9d58c54017312f702f2592d987a3854552ff5ed':

am 999d0ca1: am 87354a55: am 3b7da819: Merge "ART: Correct disassembling of regs from opcodes"

* commit '999d0ca1981fc7d0202e6c8ccc6a58d2fdfe1ece':

am a4f12f78: am 949e5980: am b7e78cb6: Merge "AArch64: Fix and enable reverseBytes intrinsic."

* commit 'a4f12f78d8bb6b1e92ff7154884e9c07e702d9d5':