Merge "ART: Curb lock-verification-failure spam"
am: d64842cc44

* commit 'd64842cc4425a1ae31405e58826ce445054b8d20':
  ART: Curb lock-verification-failure spam

Merge "ART: Curb lock-verification-failure spam"

ART: Curb lock-verification-failure spam

Just print one message per class. Also print a helpful message once.

Bug: 26951356
Change-Id: I83702b67dc535d86e03835df7a72afda081c83be

Merge "Point fixes after OSR change."
am: 74472a8eea

* commit '74472a8eea93bbe67aebed24a1fd5163a37a20dc':
  Point fixes after OSR change.

Merge "Point fixes after OSR change."

Point fixes after OSR change.

- ldr -> ldrb in osr stub for arm32
- disable 570-checker-osr for tracing
- don't osr proxy methods.

Change-Id: I9c713c9b7eab86ca9beb75f228fb3b76185621ef

Merge "ART: Better InitWithoutImage error messages"
am: 37c36dcf2e

* commit '37c36dcf2e1a78dd5798828c304d84717f79c62d':
  ART: Better InitWithoutImage error messages

Merge "ART: Better InitWithoutImage error messages"

Merge "Use 8-byte increment bracket sizes for rosalloc thread local runs."
am: e6141d7aba

* commit 'e6141d7aba1078c0465294408133aaf00ae47f1b':
  Use 8-byte increment bracket sizes for rosalloc thread local runs.

ART: Better InitWithoutImage error messages

InitWithoutImage can easily fail when assumptions of the runtime
with respect to core classes are wrong. Give some more general
helpful messages.

Change-Id: Icedac9a0d6ad84b2d1dde9e30bf4fab2f48a3efd

Merge "Use 8-byte increment bracket sizes for rosalloc thread local runs."

Merge "Address missed amend"
am: f3c3595435

* commit 'f3c35954350ab3386cff538a123bc1b8e78e89f0':
  Address missed amend

Merge "Address missed amend"

Address missed amend

Forgot to amend after addressing code review comments.

Bug: 26990650

(cherry picked from commit 3cfc2e4fb6c4735a145aa184db14d0bf33653e33)

Change-Id: Ic8483a50e0670cac1291385a82b7c5e4a313cd18

Use 8-byte increment bracket sizes for rosalloc thread local runs.

Very small space savings (< 1%) after device boot and up to 10%
allocation speedup.

Some minor cleanup.

Bug: 9986565

Change-Id: I51d791c4674d6944fe9a7ee78537ac3490c1a02c

Merge "Disable kDebugStackWalk"
am: 5102279062

* commit '51022790628270153c2d46bfebbc4b6aaafa2757':
  Disable kDebugStackWalk

Merge "Fix moving GC bugs in quick_field_entrypoints.cc"
am: 0adb678252

* commit '0adb678252bacf861dbaca0d9b5edf1a32ac7cd9':
  Fix moving GC bugs in quick_field_entrypoints.cc

Merge "Disable kDebugStackWalk"

Disable kDebugStackWalk

Someone left it enabled.

Bug: 27039049

Change-Id: I2bec610a3f20e1214dc34b4e06227609da2a2ab8

Merge "Fix moving GC bugs in quick_field_entrypoints.cc"

Fix moving GC bugs in quick_field_entrypoints.cc

There was some missing handles. Fixed style.

Bug: 26990650

(cherry picked from commit 1fd00287bedce99cd602aec00d421bddb7a9a107)

Change-Id: If7cb0de61e6b19004838e98345fdbc60ccdd5767

Merge "Implemented compare/signum intrinsics as HCompare (with all code generation for all)"
am: 45b93ca314

* commit '45b93ca31448888d13b3859a8c3d0c90d4b00e54':
  Implemented compare/signum intrinsics as HCompare (with all code generation for all)

Merge "Implemented compare/signum intrinsics as HCompare (with all code generation for all)"

Merge "Make test 971-iface-super smaller to prevent gcstress timeouts."
am: e2caf540e8

* commit 'e2caf540e8d90eb93c00d2ac1c515bef215c1362':
  Make test 971-iface-super smaller to prevent gcstress timeouts.

Merge "Make test 971-iface-super smaller to prevent gcstress timeouts."

Merge "Revert "Revert "Implement on-stack replacement for arm/arm64/x86/x86_64."""
am: 9fba3f67a0

* commit '9fba3f67a0792ad5eeb495e489d11a87211c318f':
  Revert "Revert "Implement on-stack replacement for arm/arm64/x86/x86_64.""

Implemented compare/signum intrinsics as HCompare
(with all code generation for all)

Rationale:
At HIR level, many more optimizations are possible, while ultimately
generated code can take advantage of full semantics.

Change-Id: I6e2ee0311784e5e336847346f7f3c4faef4fd17e

Merge "Revert "Revert "Implement on-stack replacement for arm/arm64/x86/x86_64."""

Revert "Revert "Implement on-stack replacement for arm/arm64/x86/x86_64.""

This reverts commit bd89a5c556324062b7d841843b039392e84cfaf4.

Change-Id: I08d190431520baa7fcec8fbdb444519f25ac8d44

Merge "Extend De Morgan factorisation to `HBooleanNot`."
am: 0f638c6468

* commit '0f638c64686c03256c949051331b0838c9888a2a':
  Extend De Morgan factorisation to `HBooleanNot`.

Merge "Extend De Morgan factorisation to `HBooleanNot`."

Merge "Revert "Implement on-stack replacement for arm/arm64/x86/x86_64.""
am: 11aa3f5c5e

* commit '11aa3f5c5efd5ce8e761dfd76378a91d9f8dc42b':
  Revert "Implement on-stack replacement for arm/arm64/x86/x86_64."

Merge "Revert "Implement on-stack replacement for arm/arm64/x86/x86_64.""

Revert "Implement on-stack replacement for arm/arm64/x86/x86_64."

DCHECK whether loop headers are covered fails.

This reverts commit 891bc286963892ed96134ca1adb7822737af9710.

Change-Id: I0f9a90630b014b16d20ba1dfba31ce63e6648021

Merge "Implement on-stack replacement for arm/arm64/x86/x86_64."
am: 7ac9aeb501

* commit '7ac9aeb501520089334c7b9e0deca1e089945bc8':
  Implement on-stack replacement for arm/arm64/x86/x86_64.

Merge "Implement on-stack replacement for arm/arm64/x86/x86_64."

Merge "Support CMOV for x86_64 Select"
am: c8d2b0c174

* commit 'c8d2b0c174dc45c931eecf45a5fdd45fdf4be311':
  Support CMOV for x86_64 Select

Merge "Support CMOV for x86_64 Select"

Extend De Morgan factorisation to `HBooleanNot`.

Change-Id: I81aa92277fa136d675e7ef01be8e4acdbd3d3b7c

Merge "Do not emit DWARF debug info if there are no methods."
am: 3c258f4d7a

* commit '3c258f4d7a6492af733a9351ba430d876a3e5ccf':
  Do not emit DWARF debug info if there are no methods.

Merge "Do not emit DWARF debug info if there are no methods."

Merge "Add some more tests to those breaking since move to OpenJDK"
am: c4252b1179

* commit 'c4252b1179ee9204384e9ca1ac8d5524691316a5':
  Add some more tests to those breaking since move to OpenJDK

Merge "Add some more tests to those breaking since move to OpenJDK"

Merge "Revert "Disable test for now.""
am: 5dfb2020dc

* commit '5dfb2020dcb415eee8330d61f45a09c5843581f7':
  Revert "Disable test for now."

Merge "Revert "Disable test for now.""

Revert "Disable test for now."

This reverts commit 0a57b45f83d6aa8876e6e91d0e5b3f62a1690d3f.

Change-Id: I8348fdaa62cc86e3b03466844715397f6601b7c8

Merge "Revert "Revert "X86: Use the constant area for more operations."""
am: e6e0cfd798

* commit 'e6e0cfd7984b2e0794dda751d17d2f528786b05c':
  Revert "Revert "X86: Use the constant area for more operations.""

Merge "Revert "Revert "X86: Use the constant area for more operations."""

Do not emit DWARF debug info if there are no methods.

Change-Id: I43406e54b454dbac45980d5c5edd90cd5593b9fd

Add some more tests to those breaking since move to OpenJDK

Adds the names of some new failing tests that were created by
splitting out parts of the SimpleDateFormatTest method
test_parseLjava_lang_StringLjava_text_ParsePosition into their
own methods. That was done to highlight those failures that were
previously hidden by an earlier failure in the method.

Bug: 25998255
Change-Id: If3bde4fa4c8ae4b39fd89e18e8fd6a9a2ab739ba

Implement on-stack replacement for arm/arm64/x86/x86_64.

High-level overview:
- osr_method_threshold is used to know when to compile a method
  in osr mode (-> treat all loops as irreducible).
- branch instructions in the compiler query whether they can
  jump to an osr method.
- An osr entry point is found through the stack maps: if a stack
  map is duplicated in the CodeInfo, it is an osr entry point.

Change-Id: Ifb39338cd281e2c7eccce67f4e18d46428be71e4

Merge changes I514f1db3,I7dc1c7c0
am: 519281001d

* commit '519281001dabe9060e2bfefc2382570bab26c96a':
  Generate mini-debug-info on separate thread.
  Change the method which generates DWARF mini-debug-info.

Merge changes I514f1db3,I7dc1c7c0

* changes:
  Generate mini-debug-info on separate thread.
  Change the method which generates DWARF mini-debug-info.

Generate mini-debug-info on separate thread.

The generation and compression of mini-debug-info is a lot of work.
However, we can do it on background thread when the main thread is
busy with .rodata and .text related I/O.

Change-Id: I514f1db3cb50aa250639f3ef697faa9bc9976d12

Change the method which generates DWARF mini-debug-info.

This splits some code from CL198651. It moves the WriteSection
call out one level and does not otherwise change behaviour.

Change-Id: I7dc1c7c08b577b50bf6fa366a9b0ca757048b81e

Merge "ART: Avoid uninitialized padding in LinkerPatch."
am: f1ea14af52

* commit 'f1ea14af52b64adb45ba57ce8c525511c208d0e2':
  ART: Avoid uninitialized padding in LinkerPatch.

Merge "ART: Avoid uninitialized padding in LinkerPatch."

Merge "Revert "ART: Enable Jit Profiling in Mterp for arm/arm64""
am: 586996afc9

* commit '586996afc905518ed926e4680aab67bedabec9b7':
  Revert "ART: Enable Jit Profiling in Mterp for arm/arm64"

Merge "Revert "ART: Enable Jit Profiling in Mterp for arm/arm64""

Revert "ART: Enable Jit Profiling in Mterp for arm/arm64"

Not super happy to revert this, but unfortunately, too many problems when testing:

arm: tests timeout when running run-tests with the interpreter.

arm64 failures:
test-art-target-run-test-ndebug-prebuild-jit-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-003-omnibus-opcodes64
test-art-target-run-test-ndebug-prebuild-jit-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-005-annotations64
test-art-target-run-test-ndebug-prebuild-jit-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-064-field-access64
test-art-target-run-test-ndebug-prebuild-jit-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-406-fields64

  org.apache.harmony.jpda.tests.jdwp.Events.EventWithExceptionTest#testBreakpoint_BeforeException
org.apache.harmony.jpda.tests.jdwp.Events.EventWithExceptionTest#testFieldAccess
org.apache.harmony.jpda.tests.jdwp.Events.EventWithExceptionTest#testFieldModification
org.apache.harmony.jpda.tests.jdwp.Events.EventWithExceptionTest#testMethodExit
org.apache.harmony.jpda.tests.jdwp.Events.EventWithExceptionTest#testMethodExitWithReturnValue
org.apache.harmony.jpda.tests.jdwp.Events.FieldAccessTest#testFieldAccessEvent
org.apache.harmony.jpda.tests.jdwp.Events.FieldModification002Test#testFieldModifyEvent
org.apache.harmony.jpda.tests.jdwp.Events.FieldModificationTest#testFieldModifyEvent
org.apache.harmony.jpda.tests.jdwp.Events.MethodExitWithReturnValueTest#testMethodExitWithReturnValueException

This reverts commit a0a16105423459287497a98129dcba2828ccd7f0.

Change-Id: I8ff0512265ed0a422be67e7410998ad02639509c

Merge "Disable test for now."
am: 3aec725d03

* commit '3aec725d03e9ba28562e7c2578445307ecb7dd2d':
  Disable test for now.

Merge "Disable test for now."

Disable test for now.

Change-Id: I03af3ee1627d6eade46d8bd74ac9eb60245d66fc

Merge "Use GC critical section for adding and removing app images"
am: b2b6d8a778

* commit 'b2b6d8a778b5e9879319ffed07ed7b5abe09efb5':
  Use GC critical section for adding and removing app images

Merge "Use GC critical section for adding and removing app images"

Use GC critical section for adding and removing app images

A SuspendAll was not sufficient since there are points during the GC
when the mutator lock can be temporarily released. This was causing
at least some assert failures in CC such as attempting to clear
black read barrier pointers on app image objects with white read
barrier pointers.

Bug: 26979834
Bug: 26970811
Bug: 26786304
Bug: 22858531

Change-Id: I375fc05e787e629c40de6f345802447c02601ff0

Merge "runtime: Cleanup comments for kAccSkipAccessChecks"
am: bd707ab60f

* commit 'bd707ab60fa33ec1ba921ee0c7d3b24d70f00a54':
  runtime: Cleanup comments for kAccSkipAccessChecks

Merge "runtime: Cleanup comments for kAccSkipAccessChecks"

Merge "ART: Enable Jit Profiling in Mterp for arm/arm64"
am: a4e981265d

* commit 'a4e981265dd57adbe755e12a814c0f30ae073c2f':
  ART: Enable Jit Profiling in Mterp for arm/arm64

Merge "ART: Enable Jit Profiling in Mterp for arm/arm64"

ART: Avoid uninitialized padding in LinkerPatch.

Bug: 26956807
Change-Id: I0cb54a4443109de9bc95728d96bd1f80810e68a3

Merge "madvise away app image dex cache arrays"
am: e99d23aba1

* commit 'e99d23aba15f540b52d69d58bd7aee710dfc0610':
  madvise away app image dex cache arrays

Merge "madvise away app image dex cache arrays"

Merge "Use correct requested alloc space begin for CC"
am: 6f0b0426f3

* commit '6f0b0426f3780a5d0f7bb63551d5130c83925754':
  Use correct requested alloc space begin for CC

Merge "Don\'t use native bridge for non zygote-spawn processes"
am: 27c6b48902

* commit '27c6b4890264444e2724972d2aa6c2c9a5bb65bb':
  Don't use native bridge for non zygote-spawn processes

Merge "Use correct requested alloc space begin for CC"

madvise away app image dex cache arrays

This saves ram and PSS by reducing the number of mapped pages. Also
PROT_NONE the dex cache arrays section to make accesses fault.

virtual                     shared   shared  private  private
size      RSS      PSS    clean    dirty    clean    dirty     swap
-------- -------- -------- -------- -------- -------- -------- --------
Before:
/data/app/com.facebook.katana-2/oat/arm/base.art
52484    52144    43522    17244        0        0    34900        0
/data/app/com.google.android.apps.maps-2/oat/arm/base.art
 6876     6816     6816        0        0     2008     4808        0

After:
/data/app/com.facebook.katana-2/oat/arm/base.art
52484    49204    42032    14344        0        0    34860        0
/data/app/com.google.android.apps.maps-2/oat/arm/base.art
 6876     6388     6388        0        0     1584     4804        0

Bug: 22858531

Change-Id: I87888a7c1518a92736dbd29eed338e7472ae3587

Merge "Don't use native bridge for non zygote-spawn processes"

Don't use native bridge for non zygote-spawn processes

... unless we force it for testing purposes.

Bug: 26954924
Change-Id: Ib4660a35175abb479d0399e18b6194eb04dbc248

runtime: Cleanup comments for kAccSkipAccessChecks

Bug: 22414682
Change-Id: I5a19cba0fb1d6b55639c173582d547f85bc26779

Merge "Ignore ZipFileTest#testZipFileWithLotsOfEntries failures with CC on N5."
am: 6234003307

* commit '62340033078bcd965e940b8ee390bf07b7867cab':
  Ignore ZipFileTest#testZipFileWithLotsOfEntries failures with CC on N5.

Merge "Ignore DeflaterOutputStreamTest#testSyncFlushEnabled failures with CC."
am: 496450a9f4

* commit '496450a9f4a2aacc5598051acbb384c5145ddca0':
  Ignore DeflaterOutputStreamTest#testSyncFlushEnabled failures with CC.

Revert "Revert "X86: Use the constant area for more operations.""

This reverts commit cf8d1bb97e193e02b430d707d3b669565fababb4.

Handle the case of an intrinsic where CurrentMethod is still an input.
This will be the case when there are unresolved classes in the
hierarchy.

Add a test case to confirm that we don't crash when handling Math.abs,
which wants to add a pointer to the constant area for the bitmask to be
used to remove the sign bit.

Enhance 565-checker-condition-liveness to check for the case of deeply
nested EmitAtUseSite chains.

Change-Id: I022e8b96a32f5bf464331d0c318c56b9d0ac3c9a

Merge "Ignore ZipFileTest#testZipFileWithLotsOfEntries failures with CC on N5."

Make test 971-iface-super smaller to prevent gcstress timeouts.

Change-Id: I0c0c5eb58936335c48e15cb7e899ab3a7ddbc00f

Merge "Ignore DeflaterOutputStreamTest#testSyncFlushEnabled failures with CC."

Ignore ZipFileTest#testZipFileWithLotsOfEntries failures with CC on N5.

This test sometimes fails with a
java.util.concurrent.TimeoutException on ART Builbot's
hammerhead concurrent collector configuration:
* libcore.java.util.zip.ZipFileTest#testZipFileWithLotsOfEntries

Disable it while we investigate.

Bug: 26711853
Change-Id: I22d2c1f7ae6fc447045dfa0f7075b10e5b2094ae

Use correct requested alloc space begin for CC

Now we restore the original value instead of setting to null.

Bug: 26970047
Change-Id: I94d546c3a69ddf11805990a1003707bf55013328

Ignore DeflaterOutputStreamTest#testSyncFlushEnabled failures with CC.

This test sometimes fails with a
java.util.concurrent.TimeoutException on ART Builbot's
x86 concurrent collector configuration:
* libcore.java.util.zip.DeflaterOutputStreamTest#testSyncFlushEnabled

Disable it while we investigate.

Bug: 26507762
Change-Id: I4f026c20a555f985a17bf785606cae53f3f804d3

Merge "Fixed bug on premature DCHECK. With fail-before/pass-after test"
am: 85b3b52a93

* commit '85b3b52a931b54380be3753216e97431ae2215a9':
  Fixed bug on premature DCHECK. With fail-before/pass-after test

Merge "Fixed bug on premature DCHECK. With fail-before/pass-after test"

Support CMOV for x86_64 Select

If possible, generate CMOV to implement HSelect.  Tricky cases are
an FP condition (no single CC generated), FP inputs (no FP CMOV)
and when the condition is a boolean or not emitted at the use site.
In these cases, keep using the existing HSelect code.

Added Load32BitValue for int and FP and used that to remove code
duplication. Added minimal checker test for int/long CMOV generation.

Change-Id: Id71e515f0afa5a30f53c5de3a5244de1ea429aae
Signed-off-by: Mark Mendell <mark.p.mendell@intel.com>

Merge "Re-enable some MIPS/MIPS64 run-tests"
am: f6171e1b62

* commit 'f6171e1b62c1442956e83e9a95eb0c0e385622e9':
  Re-enable some MIPS/MIPS64 run-tests

Merge "Re-enable some MIPS/MIPS64 run-tests"

ART: Enable Jit Profiling in Mterp for arm/arm64

Adds the hooks for branch profiling to arm and arm64.  The
other Jit profiling modes are handled in common code.

Stubbed out support for on-stack replacement.

Change-Id: Ic298a81139108c3d7f1325b59d97e14a9de08de6

Merge "Test 565-checker-doublenegbitwise applies to all architectures."
am: 9004b817d7

* commit '9004b817d7dcfa955a02bb7827de9401cdf4d291':
  Test 565-checker-doublenegbitwise applies to all architectures.

Merge "Test 565-checker-doublenegbitwise applies to all architectures."

Re-enable some MIPS/MIPS64 run-tests

These tests are passing now on MIPS/MIPS64:
- 441-checker-inliner
- 536-checker-intrinsic-optimization
- 557-checker-instruction-simplifier-ror

Change-Id: I57b82f8ec1e774e8e0a16ba289e50aa4a764b96b

Merge "Reduce the size of native debug info generated by JIT."
am: ac6a195ec8

* commit 'ac6a195ec8e1b5a7a6bd3d0c53d8997ff7a5e2cc':
  Reduce the size of native debug info generated by JIT.

Merge "Reduce the size of native debug info generated by JIT."