Fix incorrect dex2oat check
am: 821dad8

* commit '821dad8f384f8c520f062a317b3b66c78aa12b47':
  Fix incorrect dex2oat check

Fix incorrect dex2oat check

Should have been VerifyOnlyProfile.

Bug: 27688727
Change-Id: Ieec38904196ce756c2b1d8057dbf6d1e8a58c81d

Merge "Add verify-profile compiler filter" into nyc-dev
am: cc4c168

* commit 'cc4c1686a1e7125ec8b64341e573cc0959936dd3':
  Add verify-profile compiler filter

Merge "Add verify-profile compiler filter" into nyc-dev

Merge "ART: Speed up stack guard page install"
am: 895375b

* commit '895375bb30716a488bf7b8234e900d1e15de1d21':
  ART: Speed up stack guard page install

Merge "ART: Speed up stack guard page install" into nyc-dev
am: 6716fc2

* commit '6716fc2c4d68647f7840511d24ed94a95976c43c':
  ART: Speed up stack guard page install

Merge "ART: Speed up stack guard page install" into nyc-dev

Add verify-profile compiler filter

Only verifies and dex2dex compiles classes in the profile. Goal
is to reduce application launch time.

~2x faster than interpret-only for Facebook.

Bug: 27688727
Change-Id: Ic9979c4f7ba4930298d0983c4e3c70c63500213f

Merge "ART: Speed up stack guard page install"

ART: Speed up stack guard page install

Only the main thread doesn't have its stack mapped in under normal
conditions. Reading each page is a lot of overhead and we should
try to avoid it.

Rewrite to first try a (non-fatal) protect. If the outcome is a
success, finish. Otherwise do the stack mapping, and try again.

Bug: 27718174

(cherry picked from commit 2c2d2a05eaf81d07df27418f8dfd68de6fa28ac1)

Change-Id: I16b214567585ed2f09970f618ccdec7eed219fd3

ART: Speed up stack guard page install

Only the main thread doesn't have its stack mapped in under normal
conditions. Reading each page is a lot of overhead and we should
try to avoid it.

Rewrite to first try a (non-fatal) protect. If the outcome is a
success, finish. Otherwise do the stack mapping, and try again.

Bug: 27718174
Change-Id: I16b214567585ed2f09970f618ccdec7eed219fd3

Merge "Improve resolved classes saving strategy" into nyc-dev
am: 4ea8bec

* commit '4ea8becfbfd73ac03ece671956d7c2cb3bc755af':
  Improve resolved classes saving strategy

Merge "Improve resolved classes saving strategy" into nyc-dev

Merge "Ensure art::HRor support boolean, byte, short and char inputs."
am: 38ceb62

* commit '38ceb62339514a8012695673b9e1110d13546f02':
  Ensure art::HRor support boolean, byte, short and char inputs.

Merge "Ensure art::HRor support boolean, byte, short and char inputs."

Improve resolved classes saving strategy

If we already have a non empty profile file it means that we already
saved once the resolved classes. So there's no need to hurry up and
start the profile saver eagerly after 2s.

Bug: 27600652
Change-Id: I92ac5869bf68c8d6cbac8fc1503fe3c4930241b4

Merge "Disable jsr166.Collection8Test tests from running."
am: b1aff6c

* commit 'b1aff6c7f248632028b6a62a17f02675007c9ce3':
  Disable jsr166.Collection8Test tests from running.

Revert "Revert "Fix issue with proxy invocation on default methods""
am: 5457b86

* commit '5457b86f9dc02045436cc53327c4b6383805cb6a':
  Revert "Revert "Fix issue with proxy invocation on default methods""

Merge "Disable jsr166.Collection8Test tests from running."

Disable jsr166.Collection8Test tests from running.

This class shouldn't be run on its own, it's made to
be extended.

Bug: 27673907
Change-Id: If224104736344697947f9549b5e8a0bb513416dc

Revert "Revert "Fix issue with proxy invocation on default methods""

This reverts commit daf58c80d42f024aae0cb94ebd2c0bd61ebbf240.

Bug: 27621360

(cherry picked from commit e9dd04f633fcffc485f8d74861f39f66fc8bc522)

Change-Id: I56fccf19ae80451be4c316ba14911c667cbe9e47

Ensure art::HRor support boolean, byte, short and char inputs.

Also extend tests covering the IntegerRotateLeft,
LongRotateLeft, IntegerRotateRight and LongRotateRight
intrinsics and their translation into an art::HRor
instruction.

Bug: 27682579
Change-Id: I89f6ea6a7315659a172482bf09875cfb7e7422a1

Merge "Generate native debug stackmaps before calls as well."
am: e943c3b

* commit 'e943c3b831dc0da4a6b09e940ae25c3285850e96':
  Generate native debug stackmaps before calls as well.

Merge "Generate native debug stackmaps before calls as well."

Fix zygote handling of when secondary image is out of date.
am: 02d97c3

* commit '02d97c3727a4c7fb611b73fac1a0ff19568de2c9':
  Fix zygote handling of when secondary image is out of date.

Fix zygote handling of when secondary image is out of date.

Simple fix for case when primary image is current, but a secondary image
is out of date. Will want to do a cleaner fix in the future.

Bug: 27340451

(cherry-picked from commit b8510d2311964489342d8875ef7f5b681ba874b4)

Change-Id: I034f001834d84b2a4aa367da722543091a2a7b3a

Merge "Remove TEMP_FAILURE_RETRY for unlink"
am: 7c06aef

* commit '7c06aef061fa176331b77a88c1ff2c6ae401a5f0':
  Remove TEMP_FAILURE_RETRY for unlink

Merge "Remove TEMP_FAILURE_RETRY for unlink"

Merge changes Ib979b2a4,I35b3fed2,I356acd03
am: 0f1b6b7

* commit '0f1b6b76e67144349fc0a66c25f0ba13f5efb45a':
  Revert "Ignore 145-alloc-tracking-stress run-test failures with CC."
  Revert "Ignore 145-alloc-tracking-stress failures in interpreter with CC."
  Revert "Ignore 145-alloc-tracking-stress failures with JIT and CC."

Merge changes Ib979b2a4,I35b3fed2,I356acd03

* changes:
  Revert "Ignore 145-alloc-tracking-stress run-test failures with CC."
  Revert "Ignore 145-alloc-tracking-stress failures in interpreter with CC."
  Revert "Ignore 145-alloc-tracking-stress failures with JIT and CC."

Merge "Complete the 1.8 unsafe memory fences tests."
am: 0f47487

* commit '0f474872afa353ac30012e679d77fa622e574d78':
  Complete the 1.8 unsafe memory fences tests.

Remove TEMP_FAILURE_RETRY for unlink

unlink(2) does not generate EINTR error

Change-Id: I546b00a61b8ffcbd43b01f5ae6ff31f335be00ab

Merge "Complete the 1.8 unsafe memory fences tests."

Revert "Ignore 145-alloc-tracking-stress run-test failures with CC."

This reverts commit eeed32cd6b9d6949dd1e0ebbea6ea897e58bb2b8.

Bug: 27467554

Revert "Ignore 145-alloc-tracking-stress failures in interpreter with CC."

This reverts commit fee85561670331f38e0676baadc9be422d3d079c.

Bug: 27467554

Revert "Ignore 145-alloc-tracking-stress failures with JIT and CC."

This reverts commit 3296585f5879a9563214f178f97ee689fc651758.

Bug: 27467554
Change-Id: I356acd036f55605a782a83ab971d9808f10d0ed8

Merge "Do not JIT boot image during native-debugging (keep AOT code instead)."
am: e9e98f1

* commit 'e9e98f12eb5dbe81db90d56c65c8872cabaa855d':
  Do not JIT boot image during native-debugging (keep AOT code instead).

Merge "Do not JIT boot image during native-debugging (keep AOT code instead)."

Complete the 1.8 unsafe memory fences tests.

Rationale:
The tests for the fences were still TODO, and this
change list completes them. Java does not quite allow
for a weak memory order on just the signal, but this
is as close as a real unit test as I can think of.
Also added a test suggested by Hans.

BUG=26264765

Change-Id: I2dcb9b02fa63f72daf4c39b3744ff2010888713e

Merge "Revert "Revert "Fix issue with proxy invocation on default methods"""
am: 7f3397b

* commit '7f3397bec51bdeb7dfe76a17ea9d3279ddf79345':
  Revert "Revert "Fix issue with proxy invocation on default methods""

Merge "Make run-test give extra time when doing gc-stress runs."
am: 1160f2e

* commit '1160f2e14d61f95f3fb6bea03acb5451ad926acc':
  Make run-test give extra time when doing gc-stress runs.

Merge "Revert "Revert "Fix issue with proxy invocation on default methods"""

Merge "Make run-test give extra time when doing gc-stress runs."

Merge "Revert "Revert "Assembly TLAB allocation fast path for arm64."""
am: 275f97e

* commit '275f97e225f534582b446eb7b9eb00c6757255d9':
  Revert "Revert "Assembly TLAB allocation fast path for arm64.""

Merge "Revert "Revert "Assembly TLAB allocation fast path for arm64."""

Merge "resolve merge conflicts of c19c1c2 to nyc-dev-plus-aosp" into nyc-dev-plus-aosp

resolve merge conflicts of c19c1c2 to nyc-dev-plus-aosp

Change-Id: Iac3339e21d94a5d14316f5df2761ab10cdda28d0

Make run-test give extra time when doing gc-stress runs.

Some tests (such as 968-default-partial-compile-generated) are running
up against the current timeout when run with gcstress. This increases
the timeout to allow the tests to run fully.

Change-Id: I89ada3affefd09d07c770a83d849f280293cc8ed

Generate native debug stackmaps before calls as well.

The debugger looks up PC of the call instruction, so the runtime's
stackmap is not sufficient since it is at PC after the instruction.

Change-Id: I0dd06c0b52e8079ea5d064ea10beb12c93584092

Merge "Fix bug where verification could crash on unresolved types."
am: 3bff678

* commit '3bff678d255c9551782a7fdfb356e758585e477d':
  Fix bug where verification could crash on unresolved types.

Merge "Fix bug where verification could crash on unresolved types."

Revert "Revert "Fix issue with proxy invocation on default methods""

This reverts commit daf58c80d42f024aae0cb94ebd2c0bd61ebbf240.

Bug: 27621360
Change-Id: I56fccf19ae80451be4c316ba14911c667cbe9e47

Re-enable checker tests 536 and 537
am: 7d3ed11

* commit '7d3ed110398e26b36081924a93ff751d1698ab65':
  Re-enable checker tests 536 and 537

Merge "ART: Mterp arm64 2-operand double rem fix" into nyc-dev
am: fccd0ae

* commit 'fccd0ae38eca90460218e6ddf0a5b34319079f0a':
  ART: Mterp arm64 2-operand double rem fix

Merge "Revert "Fix issue with proxy invocation on default methods"" into nyc-dev
am: 09cd709

* commit '09cd7092d352dae9bc406e4451a239caf0d423e3':
  Revert "Fix issue with proxy invocation on default methods"

Add verify-art-runtime usage to dex2oat
am: c606870

* commit 'c6068709a482065d6f5426640f1c9316797508e3':
  Add verify-art-runtime usage to dex2oat

Fix issue with proxy invocation on default methods
am: 4b34bf5

* commit '4b34bf5b242dce5ee20db08b7781bbbaf0d25969':
  Fix issue with proxy invocation on default methods

Add ProfileSaver stats and dump them during SigQuit

Bug: 27516906
Change-Id: I7d2b1091c3523805ef0f87df42feed1098678aad

Merge "Dump oat file information as part of DumpForSigQuit."
am: e4914ec

* commit 'e4914ec0d83544243257e55d0f0b9da3ed7f147a':
  Dump oat file information as part of DumpForSigQuit.

Merge "Dump oat file information as part of DumpForSigQuit."

Merge "Fix load store elimination bug in the presence of null[i]."
am: 07c4134

* commit '07c4134a759500383bf5931fe12941f48fd051da':
  Fix load store elimination bug in the presence of null[i].

Merge "Fix load store elimination bug in the presence of null[i]."

Dump oat file information as part of DumpForSigQuit.

Change-Id: Ie8d256d8fa16565105cfe5c39cc96826983469da

Fix load store elimination bug in the presence of null[i].

Due to the dex specification, we can be in a state where
two array get with the same dex register inputs are typed
differently.

bug:27683874

Change-Id: Ia821fd32e86c306093372249e7686332a7584263

Do not JIT boot image during native-debugging (keep AOT code instead).

The performance impact is currently significant and being able
to debug framework is not our primary goal for native debugging.

Change-Id: I3366c2a6317004f9dd16700b271a6c9b974f1c6e

Merge "Check GetDumpNativeStackOnSigQuit() for daemons."
am: 0205b58

* commit '0205b58a0d7a9ce5832393857c19c086c78996e9':
  Check GetDumpNativeStackOnSigQuit() for daemons.

Merge "Check GetDumpNativeStackOnSigQuit() for daemons."

Check GetDumpNativeStackOnSigQuit() for daemons.

The flag is passed by run-test to avoid libunwind crash flakiness,
so we should check it when dumping threads.

Change-Id: I3a3fcfd80ab254a315d58d629c81161fea9900dc

Re-enable checker tests 536 and 537

They were disabled due to a bug in Jack that is fixed now.

Bug: 25635944

(cherry picked from commit 2385d09d254ba3aafb2ee77178f16758e9685682)

Change-Id: I3c21e1dee81a148bf5725dea451bd63c97413fd5

Merge "ART: Remember whether the cache was pruned"
am: db1f7da

* commit 'db1f7dac02f6dcecac3e032f10abbcdbf3cf4331':
  ART: Remember whether the cache was pruned

Merge "ART: Remember whether the cache was pruned"

Merge "ART: Mterp arm64 2-operand double rem fix" into nyc-dev

Merge "ART: Mterp arm64 2-operand double rem fix"
am: e848c29

* commit 'e848c29bb35d91f23bb6f52712aa7f2dff342ea0':
  ART: Mterp arm64 2-operand double rem fix

Merge "Revert "Fix issue with proxy invocation on default methods""
am: a67a089

* commit 'a67a089761ca770aa50122c47c5451a52e7eb807':
  Revert "Fix issue with proxy invocation on default methods"

Merge "ART: Mterp arm64 2-operand double rem fix"

Merge "Revert "Fix issue with proxy invocation on default methods""

Merge "Fix valgrind-test-art-host to run both 32-bit and 64-bit tests."
am: 08aa5f3

* commit '08aa5f3086a1c0f6f4839a73535ecf3a55cfae8c':
  Fix valgrind-test-art-host to run both 32-bit and 64-bit tests.

Revert "Fix issue with proxy invocation on default methods"

This reverts commit ec3b7ab8f008f36f1072d4ba03da204229b95976.

Change-Id: Idfb2c63c246cc5dae7670503e70ad3f62bb1e1eb

Merge "Revert "Fix issue with proxy invocation on default methods"" into nyc-dev

Revert "Fix issue with proxy invocation on default methods"

This reverts commit 4b34bf5b242dce5ee20db08b7781bbbaf0d25969.

Change-Id: If072fdbaafd0ee1877e8281a83ba4a598adb3355

Merge "Fix valgrind-test-art-host to run both 32-bit and 64-bit tests."

ART: Mterp arm64 2-operand double rem fix

An instruction ordering bug caused 2-operand double-precision
rem operations to be performed incorrectly on the arm64 fast
interpreter.  Also, fixes the existing omnibus-opcodes smoke test
to better catch 2-operand float and double operation problems
(the problem was masked in the existing test).

Bug: 27604215

(cherry picked from commit 908c0b28517c2d21f9ddd231e91cdd2c339aeb16)

Change-Id: I1856d914a0cb76c6034d0c0e021525b095e33452

Merge "Add verify-art-runtime usage to dex2oat"
am: 9ee48fb

* commit '9ee48fbf7395488ea34d7d268edb31d7abe4ebe9':
  Add verify-art-runtime usage to dex2oat

Merge "Add verify-art-runtime usage to dex2oat"

Add verify-art-runtime usage to dex2oat

Bug: 27688793

(cherry picked from commit c6068709a482065d6f5426640f1c9316797508e3)

Change-Id: Idcf6b8023a15c816168ce42456d2bd7b463d0951

ART: Mterp arm64 2-operand double rem fix

An instruction ordering bug caused 2-operand double-precision
rem operations to be performed incorrectly on the arm64 fast
interpreter.  Also, fixes the existing omnibus-opcodes smoke test
to better catch 2-operand float and double operation problems
(the problem was masked in the existing test).

Bug: 27604215
Change-Id: I9fac1e61d9defe7d623c611406c35921abb65b04

Fix bug where verification could crash on unresolved types.

Bug: 27626708

Change-Id: I708ee18215dccadc00ff3daa1e01c2830475ccae

Add verify-art-runtime usage to dex2oat

Bug: 27688793

Change-Id: I290f59df003ba241a6b71fb628351312b431ec4c

Merge "Fix race in AllocEntrypointsInstrumented"
am: 6f1b7ec

* commit '6f1b7ecf61a7be88bcbb259726e99a78a7235d15':
  Fix race in AllocEntrypointsInstrumented

Merge "Fix race in AllocEntrypointsInstrumented"

Fix issue with proxy invocation on default methods

Bug: 27621360

(cherry picked from commit ec3b7ab8f008f36f1072d4ba03da204229b95976)

Change-Id: I617eee243f475872235fd75b6a401e056111ea23

Revert "Revert "Assembly TLAB allocation fast path for arm64.""

This reverts commit 52fa2c698b995c21940f366cf3a44204ddf4f8e9.

Fix the mvn instructions.

Bug: 9986565
Change-Id: Ib7b2023cd54c57131442e1de85c64f40b818313d

Merge "Fix issue with proxy invocation on default methods"
am: 932e6bf

* commit '932e6bf238a00ae9813e91acb7249cb063fab205':
  Fix issue with proxy invocation on default methods

Merge "Fix issue with proxy invocation on default methods"

Fix race in AllocEntrypointsInstrumented
am: 77d9931

* commit '77d993107773b7b9bd7f07ce08d0aaac1631bf84':
  Fix race in AllocEntrypointsInstrumented

Fix race in AllocEntrypointsInstrumented

We were using the quick_alloc_entry_points_instrumentation_counter_,
this counter is updated before the threads are suspended. The
allocator could come out of a suspend point, see that threads are
supposedly still no instrumented, continue the allocation, then
suddenly quick_alloc_entry_points_instrumentation_counter_ becomes
1 and alloc_tracking_enabled_ becomes true resulting in a failing
DCHECK.

The fix is to add a boolean that is updated only when the threads
are suspended.

Bug: 27506909

(cherry picked from commit 77d993107773b7b9bd7f07ce08d0aaac1631bf84)

Change-Id: Id12983ef77c4fddb0394e4439ce0829777f1a70b

Fix race in AllocEntrypointsInstrumented

We were using the quick_alloc_entry_points_instrumentation_counter_,
this counter is updated before the threads are suspended. The
allocator could come out of a suspend point, see that threads are
supposedly still no instrumented, continue the allocation, then
suddenly quick_alloc_entry_points_instrumentation_counter_ becomes
1 and alloc_tracking_enabled_ becomes true resulting in a failing
DCHECK.

The fix is to add a boolean that is updated only when the threads
are suspended.

Bug: 27506909
Change-Id: Ic3c6716d2af9c00189ce9271f7a2825b03cdd8a0

Merge "Remove --native-debuggable compiler option."
am: 8e57978

* commit '8e57978b5f330a322f05644c2db7fa8ce89b1b46':
  Remove --native-debuggable compiler option.

Merge "Make art::HCompare support boolean, byte, short and char inputs."
am: 03fdc90

* commit '03fdc90721a1874a4aef9adfd8fc1b9b4b4bc249':
  Make art::HCompare support boolean, byte, short and char inputs.

Merge "Forgot space."
am: 1a738fc

* commit '1a738fc659a7b91456b168d2ee1edcd9eb777569':
  Forgot space.