am d72bef09: am c962dd74: Merge "Prefer 32-bit host executables on Mac."

* commit 'd72bef09d6e77158a7b2fa4bfc9440f82a9f57d9':
  Prefer 32-bit host executables on Mac.

am 0a72d65a: am 56ebee4d: Merge "Add more checks to diagnose flaky oat file assistant test."

* commit '0a72d65ab483085c6de22bf76176873cddd5a30f':
  Add more checks to diagnose flaky oat file assistant test.

am 9aa5c9be: am 2bb897ea: Merge "Refine the canary pattern in RedZone"

* commit '9aa5c9bedecc2def743a428ca0f03384ac42a1fd':
  Refine the canary pattern in RedZone

am c962dd74: Merge "Prefer 32-bit host executables on Mac."

* commit 'c962dd742ea2c514cd36f02381c0ad33fc89e8b3':
  Prefer 32-bit host executables on Mac.

Merge "Prefer 32-bit host executables on Mac."

am be86eabe: Merge "Don\'t check code pointer for proxies in ArtMethod::GetQuickFrameInfo" into mnc-dev

* commit 'be86eabe578df433f46055bae08e77314a661d49':
  Don't check code pointer for proxies in ArtMethod::GetQuickFrameInfo

am 3387f397: Get non proxy method for StackDumpVisitor

* commit '3387f39750942b0442eec351757c7d1f9b1405ca':
  Get non proxy method for StackDumpVisitor

am 56ebee4d: Merge "Add more checks to diagnose flaky oat file assistant test."

* commit '56ebee4d2b94bcf4040f49eddaa35ad7e43cc53d':
  Add more checks to diagnose flaky oat file assistant test.

Merge "Don't check code pointer for proxies in ArtMethod::GetQuickFrameInfo" into mnc-dev

Don't check code pointer for proxies in ArtMethod::GetQuickFrameInfo

(cherry picked from commit 7c0fe5e99d1d150b3072a3ae4cd0f0362fe32834)

Bug: 22541058

Change-Id: Id084a69ccd9835147ec71df6bb759e0f1b5e6fbf

Merge "Add more checks to diagnose flaky oat file assistant test."

Prefer 32-bit host executables on Mac.

Bug: 22527295
Change-Id: I2cb1d3086b65b8227e6bdac9c0485fdc452f1e9c

am e3bab35d: am b203b332: Merge "ART: Fix UninitializedReference handling"

* commit 'e3bab35d4ff256e9c9ba7510123ccca5e8799232':
  ART: Fix UninitializedReference handling

am 75c82d00: am 364fa537: Merge "Fix bug in OatFileAssistant::GetBestOatFile."

* commit '75c82d009fb6aa31c81f5f8b8d5abb2eadd395b0':
  Fix bug in OatFileAssistant::GetBestOatFile.

am 7f474a0d: am 411b96d6: Merge "ART: Remove some of the Mac craziness"

* commit '7f474a0dfefb45021303de4702f6a4409536cfa6':
  ART: Remove some of the Mac craziness

am 2bb897ea: Merge "Refine the canary pattern in RedZone"

* commit '2bb897ea8551ff8f486a59c7bc8368f4388b7760':
  Refine the canary pattern in RedZone

Merge "Refine the canary pattern in RedZone"

Get non proxy method for StackDumpVisitor

Fixes failing DCHECK.

(cherry picked from commit ef76c3fa57ddf3f7ca32e118f599da8cb4eb1560)

Bug: 22553459
Change-Id: I6fe1012f9a5f8d29cb98bf25e3a279da51c5e1c0

am b203b332: Merge "ART: Fix UninitializedReference handling"

* commit 'b203b332be18c7bf1e4b3be3e0ddd345174e0517':
  ART: Fix UninitializedReference handling

Merge "ART: Fix UninitializedReference handling"

ART: Fix UninitializedReference handling

The merge rules in the verifier allowed Object to be successfully
merged with uninitialized references. This is invalid and should
result in a conflict. Fix by moving UninitializedReference rules
earlier.

Also add a test that forward merging is correctly allowed, both
with a valid result as well as a conflict.

Also add tests that backwards branches have the expected behavior.

Bug: 22411633
Change-Id: If837376c15f0b3550d6ce1721a3cde5901c80c7f

am 364fa537: Merge "Fix bug in OatFileAssistant::GetBestOatFile."

* commit '364fa537a78ef67825a3f7aa6b4bf8ff32c39266':
  Fix bug in OatFileAssistant::GetBestOatFile.

Merge "Fix bug in OatFileAssistant::GetBestOatFile."

am 411b96d6: Merge "ART: Remove some of the Mac craziness"

* commit '411b96d641a9db09c5b6bf3dcccd7198cab688ae':
  ART: Remove some of the Mac craziness

Merge "ART: Remove some of the Mac craziness"

ART: Remove some of the Mac craziness

We rely on new-enough Clang/LLVM builds nowadays. The integrated
assembler supports named parameters. Throw away most of the
old duplication (effectively cutting support for older Clang
versions). The only required duplications are:

1) Clang as does not support .altmacro. However, the Clang
preprocessor works different wrt/ the GCC preprocessor
and does not give us trouble with inserted spaces.

2) On the Mac, symbols are prefixed with an underscore.

This should help to avoid breaking the Mac build when changing
the assembly code, and prepare for a complete Clang-only build
for x86 and x86-64. Switching to the integrated assembler for
the host build may be done in a follow-up CL.

Bug: 17443165
Change-Id: I1a077d4b612abc2b1b851c1bdabb5008a52e5aa6

am 86aee341: am 9ed82069: Merge "Don\'t check code pointer for proxies in ArtMethod::GetQuickFrameInfo"

* commit '86aee341a23398696516b843f7c35d26056c9b50':
  Don't check code pointer for proxies in ArtMethod::GetQuickFrameInfo

am c0023fec: am 84bfbe14: Merge "Fix broken tests."

* commit 'c0023fec9f5eed1cf926ca44d53cacbc8218319d':
  Fix broken tests.

am 3495f1ea: am af0365ac: Merge "runtime: Add lambda box/unbox object equality"

* commit '3495f1ea7aeca69befe9cafd3fd1a0e71dbbaf22':
  runtime: Add lambda box/unbox object equality

am 9ed82069: Merge "Don\'t check code pointer for proxies in ArtMethod::GetQuickFrameInfo"

* commit '9ed820697b75179e236fe5a07981b198a36400b3':
  Don't check code pointer for proxies in ArtMethod::GetQuickFrameInfo

Merge "Don't check code pointer for proxies in ArtMethod::GetQuickFrameInfo"

am 84bfbe14: Merge "Fix broken tests."

* commit '84bfbe14f708cdbc2ef310a5b8973be6911602b5':
  Fix broken tests.

Merge "Fix broken tests."

Fix broken tests.

Rationale:
  (1) volatile field write/read need to apply to all
      to comply with Java memory model
  (2) clinit only needs only the write
  (3) added conservative assumptions to memory barrier
      (nothing broke, but this seems better)

Change-Id: I37787ec8f3f2c8d6166a94c57193fa4544ad3372

am af0365ac: Merge "runtime: Add lambda box/unbox object equality"

* commit 'af0365ac6a4c7d242df6f50c4da69ba0ff67e3f1':
  runtime: Add lambda box/unbox object equality

Merge "runtime: Add lambda box/unbox object equality"

runtime: Add lambda box/unbox object equality

A lambda that is boxed with box-lambda is now stored as a weak reference
in a global runtime table (lambda::BoxTable). Repeatedly boxing the same
lambda closure value will always return the same java.lang.Object back.

Since there is no way to observe the address of an object, a GC can
happen and clean up the table of any dead boxed lambdas, which can also
shrink the table to prevent the memory use from growing too much.

(Note that a lambda closure is immutable, so hashing over it is
guaranteed safe.)

Change-Id: I786c1323ff14eed937936b303d511875f9642524

am f2d108e1: am e07300b7: Merge "Improved side effect analysis (field/array write/read)."

* commit 'f2d108e1aea5cd5f8bce392a78b9021410ef5aaa':
  Improved side effect analysis (field/array write/read).

am e07300b7: Merge "Improved side effect analysis (field/array write/read)."

* commit 'e07300b712f44397ecbfed1ef7f880686e46274a':
  Improved side effect analysis (field/array write/read).

Merge "Improved side effect analysis (field/array write/read)."

Improved side effect analysis (field/array write/read).

Rationale:
Types (int, float etc.) and access type (field vs. array)
can be used to disambiguate write/read side-effects analysis.
This directly improves e.g. dead code elimination and licm.

Change-Id: I371f6909a3f42bda13190a03f04c4a867bde1d06

Add more checks to diagnose flaky oat file assistant test.

Bug: 22599792
Change-Id: I00574444133fb1cb2571f6ae71ac39c30a36275f

am 936c7ed9: am 16a150b8: Merge "ART: Fix arm32 quick_invoke_stub CFI"

* commit '936c7ed92fd3ae6c6e478dd6999326d6e087b3bf':
  ART: Fix arm32 quick_invoke_stub CFI

Fix bug in OatFileAssistant::GetBestOatFile.

Previously, GetBestOatFile would fail to return an oat file in the
case where a non-executable oat file was requested and the only
problem with the oat file was that it wasn't relocated.

Bug: 22561444
Change-Id: I6446bf474afaf6c97861e7a89bd74a07c5a52a21

am 16a150b8: Merge "ART: Fix arm32 quick_invoke_stub CFI"

* commit '16a150b8a4c5c8c9e7159aff3b2ffbd918494abd':
  ART: Fix arm32 quick_invoke_stub CFI

Merge "ART: Fix arm32 quick_invoke_stub CFI"

am f9f0b860: am 6db13772: Merge "Improve documentation of memory layouts in stack_map.h."

* commit 'f9f0b860c1b295ca5bc931c45d5595c1827cb7ec':
  Improve documentation of memory layouts in stack_map.h.

am 6db13772: Merge "Improve documentation of memory layouts in stack_map.h."

* commit '6db13772db22489e61e7934dee43144ee5362d8e':
  Improve documentation of memory layouts in stack_map.h.

Merge "Improve documentation of memory layouts in stack_map.h."

Don't check code pointer for proxies in ArtMethod::GetQuickFrameInfo

Bug: 22541058

Change-Id: Id084a69ccd9835147ec71df6bb759e0f1b5e6fbf

ART: Fix arm32 quick_invoke_stub CFI

Follow-up to 4808846b2a8647a448aaa05d561a4f60b190196b.

Change-Id: I534ba39a4f59eaea9cfec8e81c944675284fc427

am cf6fee10: am b2295afd: Merge "Get non proxy method for StackDumpVisitor"

* commit 'cf6fee1040853f02f04639f6bee077be846eaf7e':
  Get non proxy method for StackDumpVisitor

am b2295afd: Merge "Get non proxy method for StackDumpVisitor"

* commit 'b2295afd038c921ac4219c3d09774579bd577c87':
  Get non proxy method for StackDumpVisitor

am 3b34cc63: am 11237947: Merge "Change intern table to not use WaitHoldingLocks"

* commit '3b34cc634ad00ecf5bef4913dda427e7c09661d6':
  Change intern table to not use WaitHoldingLocks

Merge "Get non proxy method for StackDumpVisitor"

am 11237947: Merge "Change intern table to not use WaitHoldingLocks"

* commit '11237947eefd3ec3167d9df41f2f3a041e717b6f':
  Change intern table to not use WaitHoldingLocks

Get non proxy method for StackDumpVisitor

Fixes failing DCHECK.

Bug: 22553459
Change-Id: I6fe1012f9a5f8d29cb98bf25e3a279da51c5e1c0

Merge "Change intern table to not use WaitHoldingLocks"

am 8cd99c8f: am 7745dc34: Merge "runtime: Add -Xverify:softfail and ART_TEST_INTERPRETER_ACCESS_CHECKS"

* commit '8cd99c8f2d7c7268afc95a66f7e64ab0fb6b9e40':
  runtime: Add -Xverify:softfail and ART_TEST_INTERPRETER_ACCESS_CHECKS

am 7745dc34: Merge "runtime: Add -Xverify:softfail and ART_TEST_INTERPRETER_ACCESS_CHECKS"

* commit '7745dc34f0b40c8c2ef2ef1a540c84c305cd987a':
  runtime: Add -Xverify:softfail and ART_TEST_INTERPRETER_ACCESS_CHECKS

Merge "runtime: Add -Xverify:softfail and ART_TEST_INTERPRETER_ACCESS_CHECKS"

runtime: Add -Xverify:softfail and ART_TEST_INTERPRETER_ACCESS_CHECKS

Use ART_TEST_INTERPRETER_ACCESS_CHECKS=true to run all the tests through
the interpreter with access checks enabled. The normal interpreter tests
do not currently enable access checks, which means that a large part of
the interpreter codebase is untested.

The verifier will force every class into a soft fail mode if
-Xverify:softfail is used, thereby ensuring that if used along with the
interpreter (-Xint) that the interpret is always in access checks mode.

This is used alongside with --compile-filter=verify-at-runtime to
prevent the AOT compiler from putting down any code.

Change-Id: I35a10ed8c43d76fa96133cf01fdad497da387200

Change intern table to not use WaitHoldingLocks

Bug: 22423014
Change-Id: I9e16b8cb4def72fff73f1783a182877105feb7aa

am 73aca5e9: am 479ebe07: Merge "Extended test with "cross-over arrays"."

* commit '73aca5e9f39632b6039520b0413f79f55d9a37b2':
  Extended test with "cross-over arrays".

am 479ebe07: Merge "Extended test with "cross-over arrays"."

* commit '479ebe076561e4a48129ab0c78cf8ca93c44baf6':
  Extended test with "cross-over arrays".

Merge "Extended test with "cross-over arrays"."

Extended test with "cross-over arrays".

Rationale:
Since HIR is mostly sloppy with array references types, it is good
to extend the tests with some false invariant array references.

Bug: 22538329

Change-Id: I2ef22cbdc999c0d627590f76e7bac61984ac6ad6

am 74c32d6a: am 511bb656: Merge "Fix FieldGap priority queue ordering bug."

* commit '74c32d6adc3cc92f8b4e237089908cff695f2720':
  Fix FieldGap priority queue ordering bug.

am 511bb656: Merge "Fix FieldGap priority queue ordering bug."

* commit '511bb656522e22f0570a9edff8dd295e0de6c145':
  Fix FieldGap priority queue ordering bug.

Merge "Fix FieldGap priority queue ordering bug."

am 91825544: am 2661ffc6: Merge "Improve Thumb2 branch/load-literal fixup performance."

* commit '9182554402d024912256a4e1bf4560f7c477f200':
  Improve Thumb2 branch/load-literal fixup performance.

am eb4b5b09: am 75fd056e: Merge "Revert "Revert "Use the quickened metadata to compile -QUICK opcodes."""

* commit 'eb4b5b09d00475cef167a2b314757f755aaa9c9e':
  Revert "Revert "Use the quickened metadata to compile -QUICK opcodes.""

am 2661ffc6: Merge "Improve Thumb2 branch/load-literal fixup performance."

* commit '2661ffc6f739f2006bfa5c70bbb2f60badfaf321':
  Improve Thumb2 branch/load-literal fixup performance.

am 75fd056e: Merge "Revert "Revert "Use the quickened metadata to compile -QUICK opcodes."""

* commit '75fd056e2ac95492bb9a044fc77ef47e413a3ea1':
  Revert "Revert "Use the quickened metadata to compile -QUICK opcodes.""

Merge "Improve Thumb2 branch/load-literal fixup performance."

Merge "Revert "Revert "Use the quickened metadata to compile -QUICK opcodes."""

Improve Thumb2 branch/load-literal fixup performance.

Replace per-Fixup dependents vectors with a single array
held by the assembler and referenced by the Fixups to avoid
the cost of many allocations with the default allocator.

This improves the compilation time of the boot.oat on N5,
AOSP ToT, by about ~3% as measured by the "Compile Time"
reported with --dump-timing (~2% of the "dex2oat took...").

Change-Id: I7121cdef32d9edc6d287e602d774ffe03f530d18

Revert "Revert "Use the quickened metadata to compile -QUICK opcodes.""

This reverts commit 8ab9a9010f3929acbb6c671008b8885b762b01e1.

Change-Id: I1e654d5010ea2112982c3055fcb8c8f9c10e9ac8

am 030f2af2: Reduce space filter threshold to 128.

* commit '030f2af20dcae2c44bcf1b7b265cf3db4e3c23f8':
  Reduce space filter threshold to 128.

am 06fc780f: am 33e50d85: Merge "Reduce space filter threshold to 128."

* commit '06fc780faeaef3406aab181ba4301273af7f9579':
  Reduce space filter threshold to 128.

am 33e50d85: Merge "Reduce space filter threshold to 128."

* commit '33e50d85a3e43072fad4f58369b71da73b8f966c':
  Reduce space filter threshold to 128.

Reduce space filter threshold to 128.

In order to get closer to our code increase limit.

Bug:21868508

(cherry picked from commit 432bf3d525dd0736c91238d69e29b4db6a2f7b80)

Change-Id: Iab19fe99a8a869b0fb841bac317016e4a3dc00e2

Merge "Reduce space filter threshold to 128."

am 4c310c4c: am 32f59e58: Merge "Revert "Use the quickened metadata to compile -QUICK opcodes.""

* commit '4c310c4c31a4f54dcd6c0b53cf74158d18750be9':
  Revert "Use the quickened metadata to compile -QUICK opcodes."

am 32f59e58: Merge "Revert "Use the quickened metadata to compile -QUICK opcodes.""

* commit '32f59e585a1f8932f2a4ab3f934a0a83e29b3d19':
  Revert "Use the quickened metadata to compile -QUICK opcodes."

Reduce space filter threshold to 128.

In order to get closer to our code increase limit.

Bug:21868508

Change-Id: I05f76bfd97f03bda3741eca4e6a678c536ad0702

am 38216947: am c3c0a06b: Merge "Use the quickened metadata to compile -QUICK opcodes."

* commit '3821694760a428ffb7239ab95d3c632e07e74da7':
  Use the quickened metadata to compile -QUICK opcodes.

resolved conflicts for merge of a2407de1 to mnc-dev-plus-aosp

Change-Id: Iaf366cec9606c8fed79ec00ac2573f2298de199c

Merge "Revert "Use the quickened metadata to compile -QUICK opcodes.""

Revert "Use the quickened metadata to compile -QUICK opcodes."

Broke sdk build. Investigating.

This reverts commit 4515a67434c9e3e1627586629a1f2aba1a99aa6a.

Change-Id: I6c1107ab626758570a59dc6bf3e4228c70a57196

am c3c0a06b: Merge "Use the quickened metadata to compile -QUICK opcodes."

* commit 'c3c0a06bea879741972add3dfc8099de729856a1':
  Use the quickened metadata to compile -QUICK opcodes.

Merge "Use the quickened metadata to compile -QUICK opcodes."

Merge "ART: Boolean simplifier fix" into mnc-dev

am 2cf3fc24: am 913a737b: Merge "Test on (in)variant static and instance field arrays."

* commit '2cf3fc24395e2a166001144bf28f0cf87943fe43':
  Test on (in)variant static and instance field arrays.

am 913a737b: Merge "Test on (in)variant static and instance field arrays."

* commit '913a737ba5774b3fec944190c13fcdf5b2257b6a':
  Test on (in)variant static and instance field arrays.

am 69f77ac2: am caa2fe33: Merge "ART: Fix System.arraycopy"

* commit '69f77ac22c33c128cb76a73fd09f6d627d141dca':
  ART: Fix System.arraycopy

Merge "Test on (in)variant static and instance field arrays."

am caa2fe33: Merge "ART: Fix System.arraycopy"

* commit 'caa2fe33a8c4b3a5f7ffd0a6d6adb68337f8655f':
  ART: Fix System.arraycopy

Test on (in)variant static and instance field arrays.

Rationale:
Ensure upcoming improved side effects analysis
results in valid optimizations only.

Bug: 22538329
Change-Id: I87e699853e914a0e42adfd1c4e4f9789781b65df

Merge "ART: Fix System.arraycopy"