Merge "Revert "Revert "Use the interpreter as a heartbeat for the JIT."""
am: 52d6810e59

* commit '52d6810e592aad55e411d9af7e645d7abd02fe36':
  Revert "Revert "Use the interpreter as a heartbeat for the JIT.""

Merge "Revert "Revert "Use the interpreter as a heartbeat for the JIT."""

Revert "Revert "Use the interpreter as a heartbeat for the JIT.""

Bug: 27398183
Bug: 23128949
Bug: 26846185

This reverts commit a96917a6983a5abbe973255a3846fda549fb1657.

Change-Id: I5c4f0d87d3293a6a7ab56a33396670704b66a347

Merge "ART: Enable JitProfiling for x86_64 Mterp"
am: 00e94fb884

* commit '00e94fb884eb0c58e75a623b0fd26330a9327f1e':
  ART: Enable JitProfiling for x86_64 Mterp

Merge "ART: Enable JitProfiling for x86_64 Mterp"

Merge "oatdump: Add option to print just the header"
am: 15427ae9ca

* commit '15427ae9ca187b6513cb72caab07c09f517a982a':
  oatdump: Add option to print just the header

Merge "oatdump: Add option to print just the header"

oatdump: Add option to print just the header

Sometimes that's all one needs.

Change-Id: I0c442ed32f1662ad0b37dc2fccedd964ed1b1466

Merge "Fix one more Checker test after switch to Jack"
am: 8c55011b89

* commit '8c55011b897817ecf3d85d3f50a387e509653073':
  Fix one more Checker test after switch to Jack

Merge "Fix one more Checker test after switch to Jack"

Fix one more Checker test after switch to Jack

Bug: 25635944
Change-Id: I61780b3505104f2b8a62bc2dee889b8cd444c696

Merge "ART: Switch Checker run-tests to Jack"
am: f9f3c5b0eb

* commit 'f9f3c5b0ebacbfed59af9c5c81fe9752e6b1258c':
  ART: Switch Checker run-tests to Jack

Merge "ART: Switch Checker run-tests to Jack"

ART: Switch Checker run-tests to Jack

Bug: 25635944
Change-Id: Ic6774028025b51f54589e9625003e8f69cf39dea

Merge "Revert "ART: Allow unwinding unattached threads""
am: 3d4770c457

* commit '3d4770c4577107caab23ba40547124c993e21c38':
  Revert "ART: Allow unwinding unattached threads"

Merge "Revert "ART: Allow unwinding unattached threads""

Revert "ART: Allow unwinding unattached threads"

This reverts commit f7286721a51c772b2abb42fef5bc0d2548df2372.

Change-Id: I6c96ece063041e94d40a96bdd054f89cd7875ade

Merge "Give the JIT its own arena pool to avoid lock contentions."
am: 8d468e351e

* commit '8d468e351e42e8424f93746d9ee183f8cff01ec2':
  Give the JIT its own arena pool to avoid lock contentions.

Merge "Give the JIT its own arena pool to avoid lock contentions."

Merge "More run-test with Jack"
am: e33785cd73

* commit 'e33785cd73b8af1e0ce3a46caf731711cac454ce':
  More run-test with Jack

Merge "More run-test with Jack"

Merge "Disable test after libunwind change."
am: 6cb0a0228e

* commit '6cb0a0228e8237e2fad4b10864c9b934e804b538':
  Disable test after libunwind change.

Merge "Disable test after libunwind change."

Disable test after libunwind change.

Change that broke it:
https://android-review.googlesource.com/205606

bug:27391690
Change-Id: I73c545619a6a1bb03e49120466ac91686363230f

ART: Enable JitProfiling for x86_64 Mterp

Adds branch profiling and enables for x86_64.
Support interpreter switching in x86_64 mterp.

Change-Id: I0cb9fcf3e2a01e411d84efc78449e86c10e6bcac
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>

Merge "ART: Allow unwinding unattached threads"
am: f484954757

* commit 'f4849547579504fc660fe601687f7c60242473bb':
  ART: Allow unwinding unattached threads

Merge "ART: Allow unwinding unattached threads"

Merge "Revert "Disable flaky 130-hprof test.""
am: c3fe420c57

* commit 'c3fe420c57689e613e80e5b4033649d19f800e21':
  Revert "Disable flaky 130-hprof test."

Merge "Revert "Disable flaky 130-hprof test.""

Revert "Disable flaky 130-hprof test."

Bug: 27337759

This reverts commit 2f0d3bc12f9ade80f4d3c21b7c4aad2cfe10f19e.

Merge "Fix allocation tracking race"
am: 78fb377733

* commit '78fb377733796e4a203bc3530ef7dfc6df104245':
  Fix allocation tracking race

Merge "Fix allocation tracking race"

Fix allocation tracking race

Check if changed from uninstrumented to instrumented during GC for
alloc. If we changed, retry the allocation with kInstrumented = true.

Added stress test.

Bug: 27337759

Change-Id: Iaad7977693c9ed927f779a66b29cd58341a837da

Merge "Lower JIT thread priority"
am: bbc300c4c5

* commit 'bbc300c4c59ebf9541f04c67bdff0c257c43805d':
  Lower JIT thread priority

Merge "Lower JIT thread priority"

Merge "ART: Fix overlapping instruction IDs in inliner"
am: 31b743106d

* commit '31b743106d8654bcdf39e4f638f4591eb5d7fb78':
  ART: Fix overlapping instruction IDs in inliner

Merge "ART: Fix overlapping instruction IDs in inliner"

ART: Fix overlapping instruction IDs in inliner

Inliner creates the inner graph so that it generates instruction IDs
higher than the outer graph. This was broken because the inliner
would create instructions in the outer graph before the inner graph
is inlined.

The bug cannot be triggered because the offending instruction would
share the same ID as the first inner HLocal, which is removed before
the inner graph is inlined. The added DCHECKs reveal the hidden problem
and make it safe for HLocals to be removed in the future.

Change-Id: I486eb0f3987e20c50cbec0fb06332229e07fbae9

Lower JIT thread priority

Modify the JIT thread to run at a lower priority in order to allow
UI-critical threads to get enough CPU time.

Bug: 27417985

(cherry picked from commit e701f088f8fe3a2c1f16e2895402f26283f4bcc7)

Change-Id: I5b962a7970ae81dac9e01a8011128c538cd78e40

ART: Allow unwinding unattached threads

Partial revert of commit ed8b723c5f3989d2593ec21c65c96d6d8bf25579.

Make it (constexpr) configurable whether we allow unwinding native
stacks of unattached threads.

Change-Id: Ibb1fd0956fb796583f78ec9c3a74521f6fbe96d9

Merge "Revert "Use the interpreter as a heartbeat for the JIT.""
am: 73e31f8ae8

* commit '73e31f8ae826be5b9b0c3c216abb904cbfa71077':
  Revert "Use the interpreter as a heartbeat for the JIT."

Merge "Revert "Fix bogus interaction between code cache and instrumentation.""
am: a8fde8e67d

* commit 'a8fde8e67dc8d15a436db16c7bbe201dbb249290':
  Revert "Fix bogus interaction between code cache and instrumentation."

Merge "Standby list for dyn bce in potentially infinite loops."
am: 06d1115174

* commit '06d111517419aabb9835f5f003352497ff16dfb9':
  Standby list for dyn bce in potentially infinite loops.

Merge "Revert "Use the interpreter as a heartbeat for the JIT.""

Revert "Use the interpreter as a heartbeat for the JIT."

Hits a DCHECK testing libcore.
    art F 11973 12675 art/runtime/jit/jit_code_cache.cc:644] Check failed: it.second->GetProfilingInfo(sizeof(void*)) != nullptr

Bug: 27398183
Bug: 23128949
Bug: 26846185

This reverts commit 7273a5d045d3ceb3ff011ad65765356b69b155e8.

Change-Id: I6614a82e775ea71aa16f041313f67546db41eac8

Merge "Revert "Fix bogus interaction between code cache and instrumentation.""

Revert "Fix bogus interaction between code cache and instrumentation."

hits a DCHECK when testing libcore:
    art F 11973 12675 art/runtime/jit/jit_code_cache.cc:644] Check failed: it.second->GetProfilingInfo(sizeof(void*)) != nullptr

This reverts commit e9924b7d4fcafe7261233d2c16ebb73d7bb5e8e8.

Change-Id: I37d05991b9506a53a7c76738a7e4a6def1010958

Merge "Standby list for dyn bce in potentially infinite loops."

Standby list for dyn bce in potentially infinite loops.

Rationale:
The old code relied on "luck" to revisit basic blocks
after dynamic bce and incorporate all bounds checks
in potentially infinite loops that were "made" finite.
Now that revisiting has been removed completely, keeping
a standby list ensures all candidates are considered.

Change-Id: Ida3cf63be1307be6c2b0258d3e64b163f12be235

Merge "Fix bogus interaction between code cache and instrumentation."
am: d0ea91213c

* commit 'd0ea91213cd821302237f620caa03d3dcfe52e4f':
  Fix bogus interaction between code cache and instrumentation.

Merge "Fix bogus interaction between code cache and instrumentation."

Fix bogus interaction between code cache and instrumentation.

Change-Id: Icc314089ec1a05248b8994476d478a543e629e4d

Merge "Revert "Revert "Fast ART x86_64 interpretator"""
am: 7191fddfe6

* commit '7191fddfe6c623976dc8a97f785e3ce848e46d39':
  Revert "Revert "Fast ART x86_64 interpretator""

Merge "Revert "Revert "Fast ART x86_64 interpretator"""

Merge "Fixed bug on incorrectly revisiting same block."
am: a2313aa470

* commit 'a2313aa47096ea59ffb661d5afc7ff792bab22d6':
  Fixed bug on incorrectly revisiting same block.

Merge "Fixed bug on incorrectly revisiting same block."

Merge "x86 Fast Interpreter: Fix CFI"
am: 4626786291

* commit '462678629165f16df563b31b232d3f4ff69a7c23':
  x86 Fast Interpreter: Fix CFI

Merge "x86 Fast Interpreter: Fix CFI"

Merge "Use the interpreter as a heartbeat for the JIT."
am: 01f4dae0b4

* commit '01f4dae0b455f03ae3ba531a37eff5e4e8e9451f':
  Use the interpreter as a heartbeat for the JIT.

Merge "Use the interpreter as a heartbeat for the JIT."

More run-test with Jack

Updates the following tests
* 004-ReferenceMap: updates expectations
* 004-StackWalk: updates expectations
* 089-many-methods: updates expectations
* 138-duplicate-classes-check2: add Jack support
* 454-get-vreg: updates expectations

Bug: 19467889
Change-Id: I086e170b1951c9fdac5bd4e0234ac504644db5a1

Merge "Fix the signature of the IndexOf entry point."
am: 4cb069fa2f

* commit '4cb069fa2fd967312d5884b86bbdac343df9eb28':
  Fix the signature of the IndexOf entry point.

Merge "Fix the signature of the IndexOf entry point."

Use the interpreter as a heartbeat for the JIT.

When doing a partial code cache collection, update all entrypoints to
interpreter, so that the next full collection will remove code
that wasn't executed during that window.

bug:27398183
bug:23128949
bug:26846185

Change-Id: I4423f5c4810dac183dc8973078bf218818745e80

Merge "Dump more information to diagnose problem."
am: 411539c65c

* commit '411539c65c2f3462812f8dbc4a29b75469dadc12':
  Dump more information to diagnose problem.

Merge "Dump more information to diagnose problem."

Merge "Optimizing: Do not depend on sharpening in test 577"
am: 09294dc760

* commit '09294dc760d77d64ea4168c1fd6d96fbfbe38018':
  Optimizing: Do not depend on sharpening in test 577

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

Give the JIT its own arena pool to avoid lock contentions.

Sharing it with the verifier and the class loader is not ideal,
especially at startup time.

bug:27398183
bug:23128949
Change-Id: I1b91663a13f6c5b33ad3b4be780d93eb7fe445b4

Dump more information to diagnose problem.

bug:27424509
Change-Id: Id502cf408a7f117a2a891534f5c999d60f9370b1

x86 Fast Interpreter: Fix CFI

With this change gdb can fall through x86 Fast Interpreter frame.

Change-Id: Ia5efe0059ecbdb09491d37ab47367e222d29f1d3
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>

Merge "Improve documentation about StringFactory.newStringFromChars."
am: 47bce25b3e

* commit '47bce25b3eaf5c0e1d46f0555eaf4676f958a7a7':
  Improve documentation about StringFactory.newStringFromChars.

Merge "Improve documentation about StringFactory.newStringFromChars."

Revert "Revert "Fast ART x86_64 interpretator""

CFI is updated. Now unwinder can fall through fast interpreter.

This reverts commit c4a575f58007036ff3408b15c2ec48108add20f3.

Change-Id: Ibb0a2ef41e4d02ab0b4ecc4f030ba0e72971aa9d
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>

Merge "Revert "Disable symbolizing traces on volantis.""
am: 757701fc80

* commit '757701fc80dfe35ff9677c858b3d85e946e1ae36':
  Revert "Disable symbolizing traces on volantis."

Merge "Revert "Disable symbolizing traces on volantis.""

Revert "Disable symbolizing traces on volantis."

Devices have been updated to M.

This reverts commit 95303c682bba971128950ef82c8f6c247917ffff.

Change-Id: Ib6620f88b939700e53af87d2003ea8bbe222e1bc

Merge "Disable flaky 130-hprof test."
am: eff87a33bf

* commit 'eff87a33bf5323b002fe58bcfd808d284f38eefc':
  Disable flaky 130-hprof test.

Merge "Disable flaky 130-hprof test."

Disable flaky 130-hprof test.

bug:27337759
Change-Id: I167bd42ef539bce46978b07f755efd57bc61b2f2

Fixed bug on incorrectly revisiting same block.

Rationale:
Aart's fuzz tester found this particular bug, where
revisiting a block (after dynamic bce) would cause
static array length based bce to feed into itself
and thus incorrectly remove a needed bounds check.

bug=27376274

Change-Id: I9163f283af355d444b4cec707f194fe2b67c2572

Merge "[MIPS] Add Fast Art interpreter for Mips32."
am: 9f03916ff7

* commit '9f03916ff79dca0d529a39c0202b67ac256cf9df':
  [MIPS] Add Fast Art interpreter for Mips32.

Merge "[MIPS] Add Fast Art interpreter for Mips32."

Merge "Revert "Revert "Improve immune spaces logic"""
am: f2493846a8

* commit 'f2493846a85b596f1994ce4dd294f80d363879d7':
  Revert "Revert "Improve immune spaces logic""

Merge "Revert "Revert "Improve immune spaces logic"""

Revert "Revert "Improve immune spaces logic""

Test was flaky due to missing guard page at the end of the region.

Bug: 27136196

This reverts commit 07dbbca0b42cb8da1811de8209b4a7d4becfc7b2.

Change-Id: I4eb345f34e66b8684cd71a7897e3a8caa2f84140

Fix the signature of the IndexOf entry point.

The IndexOf entry point was declared as taking four
arguments (void*, uint32_t, uint32_t, uint32_t) whereas all
actual implementations use three arguments (void*, uint32_t,
uint32_t).  As that fourth argument is not documented, drop
it from the intrinsic declaration to have it match the
implementations.

Change-Id: I65d747033192025ccd2b9a5e8f8ed05b77a21941

Improve documentation about StringFactory.newStringFromChars.

Make it clear that the native method requires its third
argument to be non-null, and therefore that the intrinsics
do not need a null check for it.

Bug: 27378573
Change-Id: Id2f78ceb0f7674f1066bc3f216b738358ca25542

Merge "Bug fix for polymorphic inlining."
am: c3bcf40e90

* commit 'c3bcf40e905c53ec857ec40a9ae45f4c86738929':
  Bug fix for polymorphic inlining.

Merge "Cleanup polymorphic inlining to the same target."
am: fb734b416d

* commit 'fb734b416d20e5c3d1c29576f5cf16a003960b55':
  Cleanup polymorphic inlining to the same target.

Merge "Bug fix for polymorphic inlining."

Merge "Cleanup polymorphic inlining to the same target."

Cleanup polymorphic inlining to the same target.

- Don't do expensive lookups, just fetch the ArtMethods from the
  imtable or vtable.
- Don't perform the optimization if we may end up in the conflicting
  trampoline.

bug:27398183

Change-Id: I327c9f7c3a85d570a057ff8f18602e8e52402fdc

Bug fix for polymorphic inlining.

The code used to wrongly propagate try/catch information to
new blocks. Since it has the same logic as Hraph::InlineInto,
extract the code that updates loop and try/catch information
to blocks to a shared method.

bug:27330865
bug:27372101
bug:27360329

Change-Id: I4386f724d8d412bde5bcc04fda6955bc3bacf5a9

Optimizing: Do not depend on sharpening in test 577

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

Change-Id: I01a2fc1bd2755804326a2ac284ee81e99fc91aa6

Merge "Disable symbolizing traces on volantis."
am: eebfed8b2f

* commit 'eebfed8b2fc4d6efa614f98001a0dc918bd225c0':
  Disable symbolizing traces on volantis.

Merge "Disable symbolizing traces on volantis."

Disable symbolizing traces on volantis.

Change-Id: I1ba6b5a173e77e635d721dea8433cfcb8b0b787b

Merge "Revert "Improve immune spaces logic""
am: a48224f327

* commit 'a48224f32797b234f07d78fbbede5edbb212ceab':
  Revert "Improve immune spaces logic"

Merge "Revert "Improve immune spaces logic""