Merge "Fix moving GC bugs in quick_field_entrypoints.cc"

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

* commit 'a18ea28591abc4f998064fbcd3efa4b8d35ee233':
  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)"
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
am: 81c7315130

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

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
am: 8385ed035b

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

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."""

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

* commit '49772e2c6b36107f0eadbd6c80092c789dced81a':
  Extend De Morgan factorisation to `HBooleanNot`.

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

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

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

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

Merge "Support CMOV for x86_64 Select" am: c8d2b0c174
am: 8745ded071

* commit '8745ded071110934b739ea9ab566c4057e691e23':
  Support CMOV for x86_64 Select

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

* commit '3e14e3b32877cb38628c5223b79897ebd583cde8':
  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
am: 9fd30bdf1b

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

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

* commit '594053fff5e21d17409c6fc3bfca84637dd53503':
  Revert "Disable test for now."

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

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

Merge changes I514f1db3,I7dc1c7c0 am: 519281001d
am: 344c8ac9b9

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

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

* commit '181a29d4b710651388ae990c68e9cc3232f3768d':
  ART: Avoid uninitialized padding in LinkerPatch.

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

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

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

* commit '3220aad231a5cc5372e2cc5baa7cd4d3496e779e':
  Disable test for now.

Fix moving GC bugs in quick_field_entrypoints.cc

There was some missing handles. Fixed style.

Bug: 26990650

Change-Id: Iee0a3308c11f7275609ee43662508bcb8769bef1

Merge "Store relative paths in the profile file"

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

This reverts commit bd89a5c556324062b7d841843b039392e84cfaf4.

Change-Id: I08d190431520baa7fcec8fbdb444519f25ac8d44

Store relative paths in the profile file

Bug: 26409579
Bug: 26080105
Change-Id: I45e6e382946a45129ed43d92768c7e870cf41aff

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 "Revert "Disable app images for now""

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

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

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"

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

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

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
am: 918a579840

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

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
am: 42824affb1

* commit '42824affb1cf137aa0ef8f3f256c78d17b70c808':
  madvise away app image dex cache arrays

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
am: ff41bd2599

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

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
am: 7d61442025

* commit '7d61442025cdb228485ff38f7044771fa7fe3765':
  Don't use native bridge for non zygote-spawn processes

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
am: b0a8d95760

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

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

* commit '92ec0c5e8dd11bfe895f1bf7409d47097081aa50':
  Ignore DeflaterOutputStreamTest#testSyncFlushEnabled failures with CC.

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
am: 120b36a903

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

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>