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

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

Merge "Change intern table to not use WaitHoldingLocks"

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 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 511bb656: Merge "Fix FieldGap priority queue ordering bug."

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

Merge "Fix FieldGap priority queue ordering bug."

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 33e50d85: Merge "Reduce space filter threshold to 128."

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

Merge "Reduce space filter threshold to 128."

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

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

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

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

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"

ART: Fix System.arraycopy

We cannot use the same code for float+int and long+double. In debug
mode, this will fail.

Change-Id: Icf263626896a7b53e59685c474e77b4c3128ecd5

am 6d3d1e3d: Merge "Disable test while investigating."

* commit '6d3d1e3d866a880b4df95ba96ed126c1723e3dd6':
  Disable test while investigating.

Merge "Disable test while investigating."

Disable test while investigating.

Change-Id: Iea1a5b59582d2d12aba8ca5eb5eab3db3cc9dd59

Use the quickened metadata to compile -QUICK opcodes.

Change-Id: I28a8d68921698bea81f54c95cc6e4c6d2c03f2b4

am cced6ad1: Merge "Use (D)CHECK_ALIGNED more."

* commit 'cced6ad152cf8ec13f38f2210c1b8445919232d3':
  Use (D)CHECK_ALIGNED more.

am 530f3185: Merge "Make oatdump see the new vmap table generated by dextodex."

* commit '530f31858cfe019e48b5f0b2ecea2ad528e6f6c6':
  Make oatdump see the new vmap table generated by dextodex.

Merge "Use (D)CHECK_ALIGNED more."

Merge "Make oatdump see the new vmap table generated by dextodex."

Make oatdump see the new vmap table generated by dextodex.

Change-Id: I48f3fb7320e39c864f8fbc478f970b18358408ec

am bca4d299: Merge "ART: Boolean simplifier fix"

* commit 'bca4d29975396b08f1d7357912065660b8007dd7':
  ART: Boolean simplifier fix

am 8f861e39: Merge "ART: DCE should know that array-length can throw NPE"

* commit '8f861e399c96abd8f1bc7c8c585d53b0904f371a':
  ART: DCE should know that array-length can throw NPE

Merge "ART: Boolean simplifier fix"

Merge "ART: DCE should know that array-length can throw NPE"

ART: DCE should know that array-length can throw NPE

array-length can throw NPE so it should be taking into account.

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

am fec9d09b: Merge "Fix the documentation of all-run-test-names."

* commit 'fec9d09bf7d2d54048b1df8e3c24a210e58dbc84':
  Fix the documentation of all-run-test-names.

am 00b5c617: Merge "Use art::MemoryRegion::{Load,Store}Bits more in art::CodeInfo."

* commit '00b5c6175210e3372920086818a9d2aa15216eda':
  Use art::MemoryRegion::{Load,Store}Bits more in art::CodeInfo.

ART: Boolean simplifier fix

Boolean simplifier should not remove condition instruction
if it has side effects.

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

Use (D)CHECK_ALIGNED more.

Change-Id: I9d740f6a88d01e028d4ddc3e4e62b0a73ea050af

Merge "Fix the documentation of all-run-test-names."

Merge "Use art::MemoryRegion::{Load,Store}Bits more in art::CodeInfo."

am a14e2e78: Merge "Move MarkObject to mark_sweep.cc"

* commit 'a14e2e780230b0c2f8e94445a4c8d21f19d447cf':
  Move MarkObject to mark_sweep.cc

am 2e7ccf9a: Merge "Enable ASan annotations in ART based on macro setting."

* commit '2e7ccf9aeb7212c96e2081b66f85b67856b8825a':
  Enable ASan annotations in ART based on macro setting.

Merge "Move MarkObject to mark_sweep.cc"

Move MarkObject to mark_sweep.cc

Maybe fixes mac build.

Change-Id: I7c018922be8e159001498d30682435643bed1dfd

Merge "Enable ASan annotations in ART based on macro setting."

Enable ASan annotations in ART based on macro setting.

This is a quick hack to remove ASan annotations from SANITIZE_TARGET
build, because they are breaking everything - it looks like we forget
to clean redzones from a page before releasing it to the system, and
some unrelated code runs into them later.

We will add some way to control this from the build system later,
or just fix the underlying issue and re-enable by default.

Bug: 21785137
Change-Id: If79e2455e246dc39355f2758323b249ac20b6144

am 34ac23dc: Merge "Address some GC comments"

* commit '34ac23dc965bf6ca24dd2e9bb115f66ff1fc191e':
  Address some GC comments

Merge "Address some GC comments"

Address some GC comments

Follow-up from:
https://android-review.googlesource.com/#/c/159650/

Change-Id: Id14f29b4ce5b70b63fcb3e74f8503ae60a3ae444

am e5abf070: Merge "Use SuspendAllInternal for FlipThreadRoots"

* commit 'e5abf0702555f9f4284a96e923f5c331561974e4':
  Use SuspendAllInternal for FlipThreadRoots

Merge "Use SuspendAllInternal for FlipThreadRoots"

Use SuspendAllInternal for FlipThreadRoots

Fix FlipThreadRoots to wait until all the threads are suspended by
using SuspendAllInternal. Since running threads no longer hold the
mutator lock as shared held, doing an exclusive lock will not block
if the threads are not suspended. Also implemented MarkHeapReference
which is used to preserve soft references.

Fixes CC tests.

Change-Id: I4b059238f4249cf297e21ae918becd029fe26527

Fix the documentation of all-run-test-names.

Change-Id: I160f57bb2244a79e5eaf5e87b9f0ac94747adcd1

am 12cfc600: Merge "Clean up after verifier changes wrt. conflict/undefined reg."

* commit '12cfc600d21ebf97373be59f1bd538b68687b5bf':
  Clean up after verifier changes wrt. conflict/undefined reg.

Merge "Clean up after verifier changes wrt. conflict/undefined reg."

Clean up after verifier changes wrt. conflict/undefined reg.

Remove unused path in RegType::Merge() and compare undefined
type as pointers. Reduce the number of edges to merge in
b_22331663.smali to just those that we really want to test.

Bug: 22331663
Change-Id: I16b83c4c97fd40be9dd246ef13ccda6e924eef60

Use art::MemoryRegion::{Load,Store}Bits more in art::CodeInfo.

Change-Id: Id50fc029e83bf2c16bd991ea82acd3b355c29127

Fix FieldGap priority queue ordering bug.

The priority queue for keeping track of gaps when packing fields in a
class object had the order reversed, giving priority to smaller gaps
instead of priority to larger gaps. This led to cases where fields
were not placed in gaps when they could be.

Bug: 22460222
Change-Id: I062e772e030c034adc227d75deed31c3322e203e

am 5c03f369: Merge "ART: Build SSA form when try/catch is present"

* commit '5c03f3698f5e2e0d762e2e15ec00310b6cdab4fe':
  ART: Build SSA form when try/catch is present

Merge "ART: Build SSA form when try/catch is present"

ART: Build SSA form when try/catch is present

This patch implements support for try/catch in the SsaBuilder.
Values of locals are propagated from throwing sites inside try
blocks to their respective catch blocks and phis ("catch phis")
are created when necessary.

Change-Id: I0736565c2c4ff3f9f0924b6e3a785a50023f875a

am 045da88b: Merge "Revert "Revert "Revert "Revert "Make dex2dex return a CompiledMethod after quickening."""""

* commit '045da88bd0230ddf48aaf037b3d6c16fbb97ba65':
  Revert "Revert "Revert "Revert "Make dex2dex return a CompiledMethod after quickening.""""

am 172c8f71: Merge "Avoid unexpected conversions to ArrayRef."

* commit '172c8f71a866931728c389a46ae9e69bc4c04a13':
  Avoid unexpected conversions to ArrayRef.

Merge "Revert "Revert "Revert "Revert "Make dex2dex return a CompiledMethod after quickening."""""

Revert "Revert "Revert "Revert "Make dex2dex return a CompiledMethod after quickening.""""

This reverts commit ed6195a514e3253576af27ea9ba13038509d29ac.

Change-Id: Icb58854301e8982147cdebe3edf2e0d9e0a63a56

Merge "Avoid unexpected conversions to ArrayRef."

am 76ca8206: Merge "ART: Fix mac build"

* commit '76ca82063ac62488fefb0b12d40026f4ebb2e198':
  ART: Fix mac build

Merge "ART: Fix mac build"

am 19315c12: Merge "Clean up GC callbacks to be virtual methods"

* commit '19315c121047edcedf2828fea50f3208d799eca2':
  Clean up GC callbacks to be virtual methods

ART: Fix mac build

Fix Clang assembler bugs introduced in commit
3031c8da0c5009183f770b005c245f9bf2a4d01b.

Change-Id: I460c7c1b8f4380244925d248b90c88239540527a

Merge "Clean up GC callbacks to be virtual methods"

am 70451aea: Merge "Change phony targets to set dalvik.vm.usejit properly"

* commit '70451aea314de4743d2d15a48d33201a897c190d':
  Change phony targets to set dalvik.vm.usejit properly

Merge "Change phony targets to set dalvik.vm.usejit properly"

Change phony targets to set dalvik.vm.usejit properly

Bug: 22408082
Change-Id: I87d75902fef9d695c034c27ea4e25448c23a6924

Clean up GC callbacks to be virtual methods

Change-Id: Ia08034a4e5931c4fcb329c3bd3c4b1f301135735

am 8ae35889: Merge "Add ART_TEST_DEBUG_GC to help with debugging gc."

* commit '8ae3588989ea99d8c60f885e3d830e6e0c87ff5f':
  Add ART_TEST_DEBUG_GC to help with debugging gc.

Merge "Add ART_TEST_DEBUG_GC to help with debugging gc."

Add ART_TEST_DEBUG_GC to help with debugging gc.

ART_TEST_DEBUG_GC is meant to be used to easily set the GC
configuration to the one that currently needs debugging in a
continuous test.

Bug: 19894268
Change-Id: I566e44360fc4cd47b3448e194cc8c16c9f498b7f

am 34fcff08: Merge "ART: Remove art_quick_invoke_interface_trampoline"

* commit '34fcff08a9ecc981f5967b9cc8111a5fac555282':
  ART: Remove art_quick_invoke_interface_trampoline

Avoid unexpected conversions to ArrayRef.

Make single-parameter constructors explicit, except
for the default copy constructor.

Change-Id: If7c8db643b22a4c36e244830154fc699b39c9016

Merge "ART: Remove art_quick_invoke_interface_trampoline"

ART: Remove art_quick_invoke_interface_trampoline

The function has only been used by the IMT conflict resolution
trampoline for a while. Merge the two, which saves a branch.

Change-Id: I2f8c9204adf839ddc5459cc04e70d98f858110a1

am 01c2b053: Merge "Fix mac build"

* commit '01c2b053aa74d9bd461a29e0c252f7a8ce3f115a':
  Fix mac build

am 68953ebf: Merge "Add an option to run art tests with strace."

* commit '68953ebf53e3ef7e0fd2c5beadfa7a2437eebc60':
  Add an option to run art tests with strace.

Merge "Fix mac build"

Merge "Add an option to run art tests with strace."

Fix mac build

Untested.

Change-Id: I1b7803f2fdcd7f509614aa9d3dce414e50a6119d

am d1665b7a: Merge "Revert "Revert "Fuse long and FP compare & condition on ARM64 in Optimizing."""

* commit 'd1665b7a689086ec5b33a69a05313c46ea1a95e4':
  Revert "Revert "Fuse long and FP compare & condition on ARM64 in Optimizing.""

Merge "Revert "Revert "Fuse long and FP compare & condition on ARM64 in Optimizing."""