Merge \"Fix interpreter tests.\"
am: 3e6bc4c76a

Change-Id: I639ae01f1f1c64ac5dba4e9d5fd141ff573ff592

Merge "Fix interpreter tests."

Fix interpreter tests.

Monitor operations can throw NPEs.

bug:29321958
Change-Id: If7a2e35a2073a7984a5126a826e722c918ba8d3a

Merge \"Fix mac build.\"
am: c23d6450c9

Change-Id: I38d3f92361b0176074d08c88e1f1dfae1f6a361a

Merge "Fix mac build."

Fix mac build.

bug:29321958
Change-Id: Ibd51d408dd97876eb3237df5606dbe9290ca7254

Merge \"Integer.bitCount and Long.bitCount intrinsics for ARM\"
am: b98a57d276

Change-Id: I263c798aebdea8e5f567036c4064f7b2340bc950

Merge "Integer.bitCount and Long.bitCount intrinsics for ARM"

Integer.bitCount and Long.bitCount intrinsics for ARM

Change-Id: Ia535865721313e72925fdec851a759470173cf89

Merge \"Remove the old obsolete profiler\"
am: 9c23cec953

Change-Id: I10553eb463ed036f0d1d481e6a8f777a5a0565f0

Merge \"Do checks on the fault address when we think it\'s an NPE.\"
am: fdbc39551e

Change-Id: I7125db6461341e6e7b890af9633484a906ffb7b4

Merge "Remove the old obsolete profiler"

Merge "Do checks on the fault address when we think it's an NPE."

Merge \"ARM64: Ensure stricter alignment when loading and storing register pairs\"
am: 34bccc1d85

Change-Id: I6c343eb20f3c35c732170b06a79362adef0fe2da

Merge "ARM64: Ensure stricter alignment when loading and storing register pairs"

Do checks on the fault address when we think it's an NPE.

bug:29321958
Change-Id: I28f4da56eb3e0b48721d3ac41114858bc80daadb

Merge \"Revert \"Optimize IMT\"\"
am: bb8d501c9b

Change-Id: Id3b16cec52bf776741bb972c48bea9f6cf14a525

Merge \"Revert \"Refactor GetIMTIndex\"\"
am: e36389f20c

Change-Id: I476e05b611bace3cb8ca378e625de1f1fc83633a

Merge "Revert "Optimize IMT""

Revert "Optimize IMT"

Bug: 29188168 (for initial CL)
Bug: 29778499 (reason for revert)

This reverts commit badee9820fcf5dca5f8c46c3215ae1779ee7736e.

Change-Id: I32b8463122c3521e233c34ca95c96a5078e88848

Merge "Revert "Refactor GetIMTIndex""

Revert "Refactor GetIMTIndex"

I need to revert this to get https://android-review.googlesource.com/#/c/244190/ to cleanly revert. Matthew, do you mind rewriting it?

This reverts commit 50706437d8216e41f0fea1e413cda7891324d397.

Change-Id: I5c1435f5dffb46dbb5b613b22adb88c7770304f2

Merge \"Revert \"Propagate flags to ShouldHaveEmbeddedVTable\"\"
am: abf64415cf

Change-Id: I0e59d5731f8a790309fc27ad5718403ae9b799b0

Merge "Revert "Propagate flags to ShouldHaveEmbeddedVTable""

Revert "Propagate flags to ShouldHaveEmbeddedVTable"

Bug: 29188168 (for initial CL)
Bug: 29778499 (reason for revert)

This reverts commit efeb9f01308c5c4b723ff905cf5bea1636f7a549.

Change-Id: I21bc32fa07014bbce2eedba3686f09d3d45aec65

Merge \"Add ability to generate a random test profile\"
am: 4dd9d82d6c

Change-Id: I4b2ceb684ad2fcea89b8b8212ae2064cf8070d30

Merge "Add ability to generate a random test profile"

Merge \\"ARM assembler support for VCNT and VPADDL.\\" am: 5cb6513d51
am: fc03b672c1

Change-Id: I3fc473fff12413ed9870b7214dbc0a6ad6c054b0

Merge \"ARM assembler support for VCNT and VPADDL.\"
am: 5cb6513d51

Change-Id: I39725f5849d1721ffc6af18bd5f0aecb7634e611

Merge "ARM assembler support for VCNT and VPADDL."

Merge \\"Add debug option to ignore checksum checks in profiles.\\" am: ece0c2e4de
am: b9440ff8a5

Change-Id: Ia38e17ce0ced75fc3fd5d90fca51dfcadb22e2f0

Merge \"Add debug option to ignore checksum checks in profiles.\"
am: ece0c2e4de

Change-Id: I9f0963f97ddcec78a8fa3658fe3197342a6236f2

ARM assembler support for VCNT and VPADDL.

Test: Gtest assembler_thumb2_test.
Change-Id: I8a0e47da746e1c67650cb68196a9f661deed7383

Merge "Add debug option to ignore checksum checks in profiles."

Remove the old obsolete profiler

Bug: 24698874
Test: all the existing run-tests pass.
Change-Id: I56d25c75541275512a64ae5708569461b875df9c

Add ability to generate a random test profile

This is usefull to drive automated profile guided compilations.

Test: added TestProfileGeneration in profile_assistant_test.

bug: 29066961
Change-Id: Ib7ce7de898abefae1c2cf1b8ff16c4f0fc9b0d0e

Add debug option to ignore checksum checks in profiles.

Test: existing tests passing, profile_compilation_info_test.

Bug: 29066961
Change-Id: I6084f50d6a6ada7407e1c7ad92fea9af3351715e

Merge \\"Optimize away useless masking operations on shift amounts.\\" am: f249230400
am: e3250eaf2e

Change-Id: If5eb9a5491ef3bbe0ec7308883b778f8fd9b4597

Merge \"Optimize away useless masking operations on shift amounts.\"
am: f249230400

Change-Id: I7012b320625831f0cd63c5b49e545d4dc9922b18

Merge \\"MIPS64: Remove the workaround in VisitInvokeStaticOrDirect\\" am: d5d0c7b6ab
am: 09c82e7732

Change-Id: I600a15c245face8a86c58146017182056f3ba630

Merge \"MIPS64: Remove the workaround in VisitInvokeStaticOrDirect\"
am: d5d0c7b6ab

Change-Id: I6c720caaeee80704339c7022836707c65a4b9180

Merge "Optimize away useless masking operations on shift amounts."

Merge "MIPS64: Remove the workaround in VisitInvokeStaticOrDirect"

Merge \\"Fix 004-ThreadStress.\\" am: 99376148af
am: 1171e22d7c

Change-Id: Id648563e739da8cf70acb1d2aead055eaac46582

Merge \"Fix 004-ThreadStress.\"
am: 99376148af

Change-Id: Ifc329644f6039a2563a8d49292f0d51faa12e2af

Merge \\"Optimizing: Do not depend on sharpening in test 450\\" am: 3ce4f88d3c
am: 48abef53d2

Change-Id: I80eb681d9ac41452f06d9631078e6fbcc6784890

Merge \"Optimizing: Do not depend on sharpening in test 450\"
am: 3ce4f88d3c

Change-Id: I511d3219d66d45afd80e3ae5da15a1b631964cda

Merge "Fix 004-ThreadStress."

ARM64: Ensure stricter alignment when loading and storing register pairs

The impetus for this change is the fact that loads that cross a 64 byte
boundary and stores that cross a 16 byte boundary are a performance issue
on Cortex-A57 and A72.

Change-Id: I81263dc72272192ad2d190b741a955f175880461

Merge "Optimizing: Do not depend on sharpening in test 450"

Merge \\"MIPS32: Drop redundant moves in integer conversions\\" am: 47fe36d8de
am: c8305b4578

Change-Id: I404f23c318dcd7c2aafb878a65285b3cf4a3e7ea

Merge \"MIPS32: Drop redundant moves in integer conversions\"
am: 47fe36d8de

Change-Id: I35818e12007d2e651c23154e9dd8229e9013a1e2

Merge "MIPS32: Drop redundant moves in integer conversions"

Merge \\"Clean up StringReferenceValueComparator and DexFile.\\" am: 05f0167cdd
am: 38a7126105

Change-Id: Iccfa0b4c195216fa2c5de32aeb2bd66623336b89

Merge \"Clean up StringReferenceValueComparator and DexFile.\"
am: 05f0167cdd

Change-Id: I58076c6c5df6c5765d565505909f17499281950a

Merge "Clean up StringReferenceValueComparator and DexFile."

Merge \\"Revert \\"Gray only immune objects mutators access.\\"\\" am: 105683791e
am: d3b2b36186

Change-Id: I317a09cbf2bfc5e431fca4e4d4a1a575a9c7516c

Optimize away useless masking operations on shift amounts.

Change-Id: I90d69757cc45312eda3d57cc2d32bfb4e47963f5

Merge \"Revert \"Gray only immune objects mutators access.\"\"
am: 105683791e

Change-Id: I4e8861233e18706fc6a5cf4dfeb7c04288d73d16

Merge "Revert "Gray only immune objects mutators access.""

Revert "Gray only immune objects mutators access."

Fails with:
    art F 11338 11338 art/runtime/gc/collector/concurrent_copying-inl.h:83] Check failed: !kGrayImmuneObject || updated_all_immune_objects_.LoadRelaxed() || gc_grays_immune_objects_

Bug: 29516465
Bug: 12687968

This reverts commit 16292fcc98f03690576d0739b2e5fb04b375933c.

Change-Id: I1d2d988b7707e03cc94f019cf8bef5b9a9099060

Merge \\"Gray only immune objects mutators access.\\" am: 450ee8fe76
am: 021a3dc9ee

Change-Id: Ib7b18a58f16bf14260921aac4c0cddbf0a2acb5c

Merge \"Gray only immune objects mutators access.\"
am: 450ee8fe76

Change-Id: I366757c58818264ce10f30cb6d6b7aacdd9e6be4

Merge "Gray only immune objects mutators access."

Gray only immune objects mutators access.

To reduce image/zygote dirty pages.

GC doesn't gray immune space objects except when visiting the thread
GC roots of suspended threads during the thread flip. GC updates the
fields of immune space objects without pushing/popping them through
the mark stack. GC sets a bool flag after updating the fields of
immune space objects. After this point, mutators don't need gray to
immune space objects. Removed the mark bitmaps for immune spaces.

Bug: 29516465
Bug: 12687968
Test: art tests, libartd device boot, ritzperf.
Change-Id: Idfcffbdb94dfc8bfc89c30d6aff8888f04990a56

ART: Disable no-prebuild test for run-test 608
am: 0748ed9fcf

Change-Id: Iadcc7929cc22f4f5b1425b76cd5908ffed010e9c

Merge \\"ART: Disable no-prebuild test for run-test 608\\" am: 6023704356
am: 77b5f1872e

Change-Id: Ib68392eebfeef1d0454ac66a8076508c94c6c097

Merge \"ART: Disable no-prebuild test for run-test 608\"
am: 6023704356

Change-Id: I4bea5c202a2dbe53f2f8409876da6eff03e0c066

Do not remove loads/store with unresolved accesses.
am: c2f1735e04

Change-Id: Iec193d6177f3b7a6115e6828aa83c73d3593dfc7

ART: Disable no-prebuild test for run-test 608

This test relies on unresolved types, which is not the case in a
no-prebuild setting (as all parts will be on the classpath).

Bug: 29433999
Change-Id: I11b1c6b446905aacd7a520796d0b2217319cefc7
Test: m ART_TEST_RUN_TEST_NO_PREBUILD=true test-art-host-run-test
(cherry picked from commit 06b7c4de44979f1cb51707d2236142bc733ebf75)

Merge "ART: Disable no-prebuild test for run-test 608"

MIPS32: Drop redundant moves in integer conversions

Change-Id: I0e3f393d87538eb9e35b3012ea36e81c8b7a225e

ART: Disable no-prebuild test for run-test 608

This test relies on unresolved types, which is not the case in a
no-prebuild setting (as all parts will be on the classpath).

Bug: 29433999
Change-Id: I11b1c6b446905aacd7a520796d0b2217319cefc7
Test: m ART_TEST_RUN_TEST_NO_PREBUILD=true test-art-host-run-test

Do not remove loads/store with unresolved accesses.

Due to AOT and compiling classes that are not ready yet
(eg missing a super class), we need to be conservative when
accessing fields and can end up in a situation where the same
field can be resolved and unresolved within the same method
(because of inlining). Therefore, disable removing loads and
stores in a method when there are unresolved accesses.

bug:29433999

(cherry picked from commit b93a16517dce12b83e9d5d3b599446e25fcc157a)

Change-Id: I8fcfb52c584222474a8220eb16c6581350b702e0

Merge \\"Do not remove loads/store with unresolved accesses.\\" am: 779743a90d
am: d8331886e6

Change-Id: I7160d5123c0da9a66db2cd1fba892ad9b5aebd4c

Merge \"Do not remove loads/store with unresolved accesses.\"
am: 779743a90d

Change-Id: I4e0048ef5043e86cb5cced5c2af5db4a1c78eb58

Merge "Do not remove loads/store with unresolved accesses."

Do not remove loads/store with unresolved accesses.

Due to AOT and compiling classes that are not ready yet
(eg missing a super class), we need to be conservative when
accessing fields and can end up in a situation where the same
field can be resolved and unresolved within the same method
(because of inlining). Therefore, disable removing loads and
stores in a method when there are unresolved accesses.

bug:29433999

Change-Id: I8fcfb52c584222474a8220eb16c6581350b702e0

Clean up StringReferenceValueComparator and DexFile.

Add a helper function to StringReference, so that the
StringReferenceValueComparator doesn't need to repeatedly
reference the same StringReference variable which led to
the bug 29602109.

Also clean up the DexFile, moving definitions of functions
that depend on the dex_file-inl.h from dex_file.h also to
the dex_file-inl.h.

Test: Run standard ART test suite on host and Nexus 5.
Bug: 29602109
Change-Id: I83e2aa67173c62b3432c26478e528bbb53b6d09f

Optimizing: Do not depend on sharpening in test 450

The sharpening may or may not remove the ArtMethod*
parameter, so the test must not depend on its absence.
This fixes the test 450 checker test on mips/mips64.

Change-Id: Ie62c5a7da5994c3b0c6f0452c9abb37575216806

Merge \\"Fixes and cleaning for test/458-checker-instruction-simplification.\\" am: 337fcb78e2
am: 887537bc3a

Change-Id: If55a6b9246f964f43e75263937feec0475f4c9e1

Merge \"Fixes and cleaning for test/458-checker-instruction-simplification.\"
am: 337fcb78e2

Change-Id: I8aa22fc03dd4586c63487c9839a36399b87d3ce7

Merge "Fixes and cleaning for test/458-checker-instruction-simplification."

Fix 004-ThreadStress.

Synchronize the start of performing the tasks after all
threads have been created and runners reported their start.
This prevents a daemon thread from allocating all memory
before the main thread tries to allocate the next daemon
thread, which is the likely culprit of failing to print any
output whatsoever (the main thread exits due to an uncaught
OOME and printing the message also fails due to OOME).

Instead of repeating attempts to print "Finishing worker"
until it succeeds, print the message through JNI which does
not require any Java heap memory. Similarly, print caught
Throwables through JNI. Also make sure to preallocate
Strings that are used once we start running tasks to avoid
const-string throwing OOME. This also prevents a partial
success followed by a retry; we have seen a failure to
print the EOL, yielding "Finishing workerFinishing worker".

Change the "Thread exited..." message to be printed only in
DEBUG mode and compare the full output, ignoring different
numbers as the "Starting worker for <id>" may be reordered.

Test: "make -j 32 test-art-host-run-test-004-ThreadStress"
    (4 configs: with or without gcstress, 32-bit and 64-bit)
    100 times while also keeping the machine under load with
    "make -j 32" to repeatedly build another full AOSP tree.
Bug: 27371304
Change-Id: I28286d70a08be64c36d0e49dd5558686813ff264

Fixes and cleaning for test/458-checker-instruction-simplification.

- Use the `doThrow` trick to defeat inlining.
- Fix the order of `expected` and `found` arguments for a few tests.

Change-Id: Id8e5041993d72f61e75b4586b49180006cda4b5c

Merge \\"ART: OneBit intrinsics should use 1ULL for 64-bit shift\\" am: 1e28d1e145
am: 90685485df

Change-Id: Ie3269e263cb146a655217e348ae1cd5e146c7a91

Merge \\"Adjust ART tests expectations after libcore changes.\\" am: 7caad2a084
am: f44e3d662a

Change-Id: I7b28ce4b8111d19db941d9fdee349b09efde15ca

Merge \"ART: OneBit intrinsics should use 1ULL for 64-bit shift\"
am: 1e28d1e145

Change-Id: Ifca8f91f444537704477824a52ae50309db93858

Merge \"Adjust ART tests expectations after libcore changes.\"
am: 7caad2a084

Change-Id: I0c2666a36e76c6faccbf0fb265d198c8dd77f431

Merge "ART: OneBit intrinsics should use 1ULL for 64-bit shift"

Merge "Adjust ART tests expectations after libcore changes."

Adjust ART tests expectations after libcore changes.

Test: test/100-reflect2, test/201-built-in-exception-detail-messages
Change-Id: I0c13e37973d0b954cbafa15b049cb8934b88ebe5

Merge \\"Fix StringReferenceValueComparator.\\" am: 8543d324d1
am: 88ca66a386

Change-Id: I82d33dece4ea64aed7ce428200b9cf7658bfcf8d

Merge \"Fix StringReferenceValueComparator.\"
am: 8543d324d1

Change-Id: Ie45f9019bf8189029efc74d1a09dce5e62c923d4

Merge "Fix StringReferenceValueComparator."

Fix StringReferenceValueComparator.

Test: Added a regression test in string_reference_test.cc,
run the standard ART test suite on host and Nexus 5.

Bug: 29602109
Change-Id: Idcc059a07df048a0e3ece257b16b6556f242243e

Merge changes I4d8da7ce,I4da5be01,Idfbead82 am: 31167a5785
am: f7511ac497

Change-Id: I0bc49fd2385ef240f5fc477c01495148b4de319b

Merge changes I4d8da7ce,I4da5be01,Idfbead82
am: 31167a5785

Change-Id: Id0131ff628a47b611e39c026243cf5f11ce9bc49

Merge changes I4d8da7ce,I4da5be01,Idfbead82

* changes:
  Re-enable most intrinsics with read barriers.
  Fix ARM & ARM64 UnsafeGetObject intrinsics with read barriers.
  Fix x86 & x86-64 UnsafeGetObject intrinsics with read barriers.

Merge \\"Track libcore change 694e617f54a7bfbdad24913ce96f5d56f1a1960a\\" am: bad21cb538
am: a5364b014b

Change-Id: I9a5755da3aba9b370b35bac17e184de58f7a27be