Merge "ART: Make verifier fail log level adjustable" am: fedcd0a
am: 226f52d

* commit '226f52d3c487ce56aa11565966b1e96ffffe86c6':
  ART: Make verifier fail log level adjustable

Change-Id: I2c70e3fa252951d1dde9ca5aff49c5cb6dbd6eb3

Merge "ART: Make verifier fail log level adjustable"
am: fedcd0a

* commit 'fedcd0a6870cf4d940d231f6725ca95c590d04bd':
  ART: Make verifier fail log level adjustable

Change-Id: Ia860a719def57f0ff1a914a93947d11b6fd58f49

Merge "ART: Make verifier fail log level adjustable"

Merge "ART: Clean up verifier" am: ef94b54
am: 361aa38

* commit '361aa3820e18657f5aca9da7a662497d22343168':
  ART: Clean up verifier

Change-Id: I934d1c10c93e589beafa15c865b386dbb7a30b4f

Merge "ART: Clean up verifier"
am: ef94b54

* commit 'ef94b5484a75902404e975cba55a7e825941b2fa':
  ART: Clean up verifier

Change-Id: I975423bbf31948bacee17a077d54001b60f23870

Merge "ART: Clean up verifier"

ART: Make verifier fail log level adjustable

To help for build failures.

Change-Id: I6c94dfb50177daa7d89902d78715ccec31bf0e45

Add support for Dex version 37 in Runtime.
am: 2190d92

* commit '2190d929695c31ad7195e2c366f4102836a7d827':
  Add support for Dex version 37 in Runtime.

ART: Clean up verifier

Clean up verifier post-Quick.

Change-Id: I0b05e10dd06edd228fe2068c8afffc4b7d7fdffa

Merge "Add support for Dex version 37 in Runtime."
am: ec89839

* commit 'ec898396658bc6a36e2b3c32569fbcb034d60e22':
  Add support for Dex version 37 in Runtime.

Add support for Dex version 37 in Runtime.

We are skipping version 36 of the dex file format due to a bug in
Dalvik dating back to ICS where dex files marked version 036 would
erroneously be accepted.

Bug: 27538761
Bug: 27809626

(cherry picked from commit c49618160d5fa64ca4883d2e09fa34b83491c8ea)

Change-Id: Ic053f7e25f5a8c3df83ff34b6656528824b2df12

Merge "Add support for Dex version 37 in Runtime."

Do not profile fully compiled oat files.
am: 6044fa7

* commit '6044fa747867413912d0de3049dc570c769d3cf8':
  Do not profile fully compiled oat files.

Merge "Fix bug not accounting for type in induction comparison. Extended unit tests."
am: e28ad4b

* commit 'e28ad4b91591c226ed404a2b01104bb99bfeb28f':
  Fix bug not accounting for type in induction comparison. Extended unit tests.

Merge "Fix bug not accounting for type in induction comparison. Extended unit tests."

Merge "ART: Relax verifier checks"
am: 782d046

* commit '782d0464ddc92fd5d55a330068d2d255336c16ba':
  ART: Relax verifier checks

Merge "ART: Relax verifier checks"

ART: Relax verifier checks

Only bail for unresolved merge types in a couple of cases. Follow-up
to bb18a03c44d99b3cae232d445d56e80d8388cca9.

Bug: 27799205
Change-Id: If448e14036bc5e245677e0cd8cdddee02b320525

Fix bug not accounting for type in induction comparison.
Extended unit tests.

Rationale:
Now that inductions are typed, comparisons should take
the type into account as well.

Change-Id: Ie533bb728d2006390646657573fa743e6fc37ff2

Merge "Allow x86+arm multilib build."
am: 91ac272

* commit '91ac272c11775630735dde3190de9f2da8f3f3eb':
  Allow x86+arm multilib build.

Merge "Allow x86+arm multilib build."

Allow x86+arm multilib build.

The explicit dependency of dex2oat on installed file of libart-compiler
isn't necessary: the build sets up the dependency automatically when
dex2oat links liblibart-compiler.
Also the way of the explicit dependency is incorrect if multilib is
involved: a 64-bit dex2oat may depend on 32-bit libart-compiler.

Bug: 27526885
Change-Id: I4a9d72b49f5712695ff5a32b88df5eab9df7c1e4
(cherry-pick from commit 5eec66ea27a2fa5bfa1b4f9bd203c76e2990d7a2)

Merge "Avoid removing new-instance instruction twice. Includes a fail-before/pass-after regression test."
am: b0b3859

* commit 'b0b38596a4f28725b71f17e13c7b544d926f0d3e':
  Avoid removing new-instance instruction twice. Includes a fail-before/pass-after regression test.

Merge "ART: Allow array-ness for unresolved merge types"
am: 0a4d6b9

* commit '0a4d6b9faaaa2652c42679df2a72c78485ed9c64':
  ART: Allow array-ness for unresolved merge types

Merge "Avoid removing new-instance instruction twice. Includes a fail-before/pass-after regression test."

Do not profile fully compiled oat files.

- don't waste a thread if we're not gonna use the profiles.
- don't put foreign dex use markers since they will be ignored.

Bug: 26080105
Change-Id: I68a8aee32e7ef40457a621cfa3f5800208c5e086

Merge "ART: Allow array-ness for unresolved merge types"

ART: Allow array-ness for unresolved merge types

In case component types aren't resolvable, array types won't be
either. They then may be merged. The merge is still potentially
an array type.

Ensure that merging an unresolved array type with a primitive
array type or a non-array type will be resolved to java.lang.Object.

Added tests.

Bug: 27799205
Change-Id: I9beff75318814dddd842abd64ef9a5d2644d801e

Avoid removing new-instance instruction twice.
Includes a fail-before/pass-after regression test.

Rationale:
In some corner cases, one new instance reached more than one
java.lang.String.<init> method call. As a result, the new
instance instruction appeared multiple times in the vector
uninitialized_strings_. A second removal crashes the compiler.
This change list avoid the crash by simply skipping instructions
that are already removed.

BUG=27847265

Change-Id: I7f8a4f84ea3c184e1529ec3e90bd6749c83c445b

Merge "Discard corrupted or out of date profiles" into nyc-dev
am: 197a6a9

* commit '197a6a9a58b2e01aa89fba7741e9a01c2f4b7992':
  Discard corrupted or out of date profiles

Merge "Discard corrupted or out of date profiles" into nyc-dev

Discard corrupted or out of date profiles

Until now we bailed out if the previous profile contained invalid data.
This CLs forces the save and clears any data in a profile that has the
wrong version or contains bad data.

Bug: 27081617
Change-Id: I0424025e37c7b4b817da791996436dd4a932c2cd

Merge "Revert "ART: Improve JitProfile perf in arm/arm64 mterp""
am: 5d87b29

* commit '5d87b29339c5301bea0bf2c3f47e520e3d7b0d16':
  Revert "ART: Improve JitProfile perf in arm/arm64 mterp"

Merge "Revert "ART: Ignore hotness updates if jit inactive""
am: 38be6ba

* commit '38be6bad7a9015b1633751608e833b40dc7a0012':
  Revert "ART: Ignore hotness updates if jit inactive"

Merge "Revert "ART: Improve JitProfile perf in arm/arm64 mterp""

Merge "Revert "ART: Ignore hotness updates if jit inactive""

Revert "ART: Improve JitProfile perf in arm/arm64 mterp"

This reverts commit c1d6b341eed646e5adafc6c4fd4e3748f0292368.

Revert "ART: Ignore hotness updates if jit inactive"

This reverts commit 17bda200dc7e36ba4178501274f10df4c82bea11.

Merge "Only call jit_load after the zygote fork"
am: 2295f2e

* commit '2295f2eff470ebb38317d1496b17439e6db8a10a':
  Only call jit_load after the zygote fork

Merge "Only call jit_load after the zygote fork"

Only call jit_load after the zygote fork

Otherwise we always get that enable_debug_features is false since
that flag passed as a compiler option post zygote fork.

No significant change to libart-compiler PSS on calculator.

Bug: 27810774

(cherry picked from commit b10cef442594dd0d6c5f1a5784643931a25cc431)

Change-Id: Ie50668ba05ad07a5cf046497959f6d27e09f8fe2

Only call jit_load after the zygote fork
am: b10cef4

* commit 'b10cef442594dd0d6c5f1a5784643931a25cc431':
  Only call jit_load after the zygote fork

Only call jit_load after the zygote fork

Otherwise we always get that enable_debug_features is false since
that flag passed as a compiler option post zygote fork.

No significant change to libart-compiler PSS on calculator.

Bug: 27810774
Change-Id: I9854716902c79e16bdfd64754cc1ed05e852a5de

Merge "ART: Ignore hotness updates if jit inactive"
am: af8e71d

* commit 'af8e71d59b0ab5e79f53a608b62b3618e36e6a98':
  ART: Ignore hotness updates if jit inactive

Add support for Dex version 37 in Runtime.

We are skipping version 36 of the dex file format due to a bug in
Dalvik dating back to ICS where dex files marked version 036 would
erroneously be accepted.

Bug: 27538761
Bug: 27809626

Change-Id: Ic053f7e25f5a8c3df83ff34b6656528824b2df12

Merge "ART: Ignore hotness updates if jit inactive"

Merge "Load jit compiler pre zygote fork"
am: ee91f08

* commit 'ee91f081f4ea8aa9747893dfaba49dad498d1a4a':
  Load jit compiler pre zygote fork

ART: Ignore hotness updates if jit inactive

The interpreters are hard-wired to collect hotness data for the
jit.  If the jit is in the process of shutting down, discard the
updates.

Change-Id: I0baf73bb2c591e9571964435f6cc45c062b85911

Merge "Load jit compiler pre zygote fork"

Merge "Load jit compiler pre zygote fork" into nyc-dev
am: d0482a8

* commit 'd0482a8cb4c43b4f440813fe11ae33856a0f100c':
  Load jit compiler pre zygote fork

Load jit compiler pre zygote fork

Reduces PSS for libart-compiler.so from 115KB to 2KB per app.

Also some clean-up.

Bug: 27810774

(cherry picked from commit ff2376770c1c6d301778c41e5c27c808a38e6968)

Change-Id: Ic0077a07c21ed9f1952a6c29863a5c0e5bacc01a

Merge "Load jit compiler pre zygote fork" into nyc-dev

Load jit compiler pre zygote fork

Reduces PSS for libart-compiler.so from 115KB to 2KB per app.

Also some clean-up.

Bug: 27810774

Change-Id: I9938b022e90c58e7978c5038338d0bcbe2a1dc2b

Merge "ART: Improve JitProfile perf in arm/arm64 mterp"
am: 42cd3cb

* commit '42cd3cb6379fe3faf078dcce98610e90155bc8e0':
  ART: Improve JitProfile perf in arm/arm64 mterp

Merge "Extend profman to be able to dump profile content." into nyc-dev
am: 55dde80

* commit '55dde80ab2bca9f07424aab92ded226761517876':
  Extend profman to be able to dump profile content.

Merge "Extend profman to be able to dump profile content." into nyc-dev

Extend profman to be able to dump profile content.

With the move to binary format it gets harder to debug profiles. This
adds a simple dumping mechanism to make profiles human readable.

Bug: 27600652
Change-Id: Idc968601cdf6e371822aa98e316605e30749e2e8

Merge "Update and improve profiles format" into nyc-dev
am: 21fae84

* commit '21fae845af390fb86831a55db79672587bb17761':
  Update and improve profiles format

Merge "ART: Improve JitProfile perf in arm/arm64 mterp"

Merge "Update and improve profiles format" into nyc-dev

Merge "ART: Fix order of operations in HBasicBlock::DisconnectAndDelete"
am: 62e78a7

* commit '62e78a7ea8221f3fbe848fa883fb86c9d87ba50b':
  ART: Fix order of operations in HBasicBlock::DisconnectAndDelete

Merge "ART: Fix order of operations in HBasicBlock::DisconnectAndDelete"

ART: Fix order of operations in HBasicBlock::DisconnectAndDelete

The method would remove predecessors before successors. As a result,
instructions used by dead loop phis would see dangling uses, causing
a DCHECK to fail.

Steps were reordered to remove dependencies in post order.

Bug: 27683071
Change-Id: I8e0e976443fb410908321a065276f1340b757c41

Merge "ART: Fix run-test"
am: 308640f

* commit '308640fd439f6e0867770d74600ad4d8edda7093':
  ART: Fix run-test

Update and improve profiles format

- add profile version in the header
- change the profile format to a binary format (saves more than 50% of
space)
- improve error handling during parsing
- improve corner cases handling (all methods/classes in the
file)
- improve error reporting

Bug: 27600652
Bug: 27081617

Change-Id: I54dfbf73fbc943ff6fc3d67f389c658d833cded9

Merge "ART: Fix run-test"

ART: Fix run-test

Follow-up to Ic61ea2193765b4578550538e965ca4f80fa4b287

Bug: 27625564
Change-Id: Ia8b128a05b473896ba9a55df0d2aaf34ae6d06f3

Merge "Fix stub_test after semantic conflicting merge."
am: 1345aa3

* commit '1345aa365e8baae25d2b3a859a4858fffbe6b11f':
  Fix stub_test after semantic conflicting merge.

Merge "Fix stub_test after semantic conflicting merge."

Fix stub_test after semantic conflicting merge.

https://android-review.googlesource.com/#/c/209881/ does
not set the ImtConflictTable when we're a AOT runtime, which the
stub test pretends to be.

Change-Id: I70e00b265e982b46b577ff19ac9272cfee45940d

Merge "ART: Loosen a GraphChecker rule on Boolean inputs"
am: 594973c

* commit '594973c83bb9187e392a7daf61b570dfe0aa3e08':
  ART: Loosen a GraphChecker rule on Boolean inputs

Merge "ART: Loosen a GraphChecker rule on Boolean inputs"

ART: Loosen a GraphChecker rule on Boolean inputs

GraphChecker tries to verify that Boolean inputs are properly typed.
This is non-trivial in the presence of simplifying optimizations
which capitalize on the fact that a Boolean value is internally
represented as an integer.

This patch removes the test from GraphChecker.

Bug: 27625564
Change-Id: Ic61ea2193765b4578550538e965ca4f80fa4b287

Merge "Post-Quick cleanup: Remove CompilationUnit."
am: 2e38434

* commit '2e384346d93a747e19fb74f32eba2d1e9403721f':
  Post-Quick cleanup: Remove CompilationUnit.

Merge "Re-enable IMT stub test."
am: a872368

* commit 'a872368b065af7cd42a1be02742a60877e096bae':
  Re-enable IMT stub test.

Merge "Post-Quick cleanup: Remove CompilationUnit."

Merge "Re-enable IMT stub test."

Merge "Clean up art::HConstant predicates."
am: 0ced281

* commit '0ced281ae6216c29f57ca0f8b7388a722e8da97b':
  Clean up art::HConstant predicates.

Merge "Clean up art::HConstant predicates."

Merge "Do a null check on the sibling in the register allocator."
am: 843a655

* commit '843a65556616183a36792bbcc1632c6d8d0e78b2':
  Do a null check on the sibling in the register allocator.

Merge "Do a null check on the sibling in the register allocator."

Do a null check on the sibling in the register allocator.

There may be a lifetime hole between the intervals, which means there is
no interval for the given position.

Litle sister of https://android-review.googlesource.com/#/c/209336/.

bug:27626705
Change-Id: I8082aa5ae2dc37d8fa5d4c430b69e6defa495439

Merge "Disable test on arm due to timeouts."
am: c2b4c6d

* commit 'c2b4c6df0c2c0fc21950c87ec9071518d43fd9c5':
  Disable test on arm due to timeouts.

Merge "Optimizing: Do not insert suspend checks on back-edges."
am: 63ec322

* commit '63ec32274066da114d733d8d3c5ff389883e403c':
  Optimizing: Do not insert suspend checks on back-edges.

Merge "Disable test on arm due to timeouts."

Merge "Optimizing: Do not insert suspend checks on back-edges."

Post-Quick cleanup: Remove CompilationUnit.

Change-Id: I309411b0fffaaed1e218e2c34394bdf6e2f75b48

Disable test on arm due to timeouts.

bug:27824283
Change-Id: I0e4d9b37dd3ac2d2390456a791ff28d9ea0b3510

Merge "Fix potential race condition with threads array."
am: d6026cd

* commit 'd6026cde1ef7f2b0abb7a3561510fe83a7dd9de2':
  Fix potential race condition with threads array.

Merge "Fix potential race condition with threads array."

Merge "ART: Update DexFile for compiler-filter pass-down" into nyc-dev
am: 5a61468

* commit '5a614682b7b293138769341aa59414a54ccdf5c8':
  ART: Update DexFile for compiler-filter pass-down

Merge "ART: Update DexFile for compiler-filter pass-down" into nyc-dev

Merge "Add RosAlloc stats dump."
am: 1fd223d

* commit '1fd223d781dfb53234cc926e7134b08545d6419c':
  Add RosAlloc stats dump.

Merge "Add RosAlloc stats dump."

Merge "Add RosAlloc stats dump." into nyc-dev
am: 4eff6fd

* commit '4eff6fde9cc17e63f1c6c15bc1cdff9410b4f7a2':
  Add RosAlloc stats dump.

Merge "Add RosAlloc stats dump." into nyc-dev

Merge "Use smaller rosalloc run sizes."
am: 760efa3

* commit '760efa3d01257250972ea47a00c0edb01d608c3a':
  Use smaller rosalloc run sizes.

Add RosAlloc stats dump.

For better understanding of the RosAlloc space.

(cherrypick commit 565c2d9bce43c430d4267c82f5702160d971e712)

Bug: 27744947
Bug: 9986565

Change-Id: I8309761a68fbc143bbcd8458a9194085aace7c3e

Merge "Use smaller rosalloc run sizes."

Merge "Use smaller rosalloc run sizes." into nyc-dev
am: c867a27

* commit 'c867a275aa7a132ec1fd4f3b8c27812bda61ea73':
  Use smaller rosalloc run sizes.