am 2c525d96: am 16dd4dfd: Merge "ART: Relax GetInstructionSetFromString" into lmp-dev

* commit '2c525d964b2331e47927c9d42bdde1086cf53379':
  ART: Relax GetInstructionSetFromString

am 92502ab7: Merge "ART: Relax GetInstructionSetFromString"

* commit '92502ab7570dfa28c98ab6781a1c5b9f8b8748bf':
  ART: Relax GetInstructionSetFromString

resolved conflicts for merge of 9285fd6d to lmp-dev-plus-aosp

Change-Id: Ic97a532af6371724c7427947d26e44dc84cd0631

Merge "ART: Relax GetInstructionSetFromString"

am e7f5428f: Merge "Make common BitVector operations inline-able."

* commit 'e7f5428fd80669d5e8a67962ef63dc80b4f628c4':
  Make common BitVector operations inline-able.

am 69d848e9: Merge "bionic has always had gettid."

* commit '69d848e9653ecc6eb3dbef6bcec4b4ddc617bef9':
  bionic has always had gettid.

am 16dd4dfd: Merge "ART: Relax GetInstructionSetFromString" into lmp-dev

* commit '16dd4dfd01c0ac432a32d081a15205222ffb1f1c':
  ART: Relax GetInstructionSetFromString

ART: Relax GetInstructionSetFromString

Do not abort on an unknown instruction set string. Instead return
kNone and let the caller handle this.

Also simplify the patchoat tool to use this.

Bug: 17136416

(cherry picked from commit aabbb2066a715b3fd8e752291f74c6d77b970450)

Change-Id: I24131914bcf91c04ae93179bf809a2907f1f2b7a

Merge "Make common BitVector operations inline-able."

Make common BitVector operations inline-able.

Change-Id: Ie25de4fae56c6712539f04172c42e3eff57df7ca

am 192da567: Merge "Add null check to CheckVirtualMethod"

* commit '192da5675b2e219f26a107bd7b60c755cd46ba74':
  Add null check to CheckVirtualMethod

am 7b6f8d21: Merge "Workaround problem reading main stack on intel devices."

* commit '7b6f8d21637e8a44d76bdf941895f1c951555f43':
  Workaround problem reading main stack on intel devices.

am 07277a70: Merge "Enable large object space for zygote."

* commit '07277a700cdc7de2f40773a05c7a791b71abd581':
  Enable large object space for zygote.

am 41cba7c6: Merge "Quick compiler: fix Arm64 inline locking code"

* commit '41cba7c66cbc441b00fca48dfb2501181b1f2a53':
  Quick compiler: fix Arm64 inline locking code

am 4efefc3b: Merge "Improve GVN performance when merging null-checked values."

* commit '4efefc3b900dfc83ffd098b0c296c835b6837925':
  Improve GVN performance when merging null-checked values.

am 72b51c66: Merge "X86: Fix alias info in GenInlinedIndexOf()."

* commit '72b51c6600e75f59c3c3dca2493fd9ea22f48b9e':
  X86: Fix alias info in GenInlinedIndexOf().

am 418a6980: Merge "Fix bug introduced in https://android-review.googlesource.com/102610."

* commit '418a6980fe461f57bea1e3bc431264c81ae890aa':
  Fix bug introduced in https://android-review.googlesource.com/102610.

am 10dac8ee: (-s ours) Merge "Merge stack usage reduction for overflow checks"

* commit '10dac8ee833a297ad0ffc5877305f2d132259478':
  Merge stack usage reduction for overflow checks

am 167350d9: Add null check to CheckVirtualMethod

* commit '167350d9c781c5e3893714bb26ab5cb1c4abf6b4':
  Add null check to CheckVirtualMethod

Merge "bionic has always had gettid."

bionic has always had gettid.

dex2oat calls GetTid for every line it outputs via Message, which is why
this showed up so much in strace.

Bug: 17049821
Change-Id: Ib51c7f88410b291692874023902cf43f4b9a9d1c

Add null check to CheckVirtualMethod

There was a runtime SIGSEGV that should have been a check jni
failure.

Also added regression test.

Bug: 16320699
Change-Id: If6c8e73959cefb24e4703f1562cdddb343d86630

am 0fcd6873: am a846c1ac: Merge "X86: Fix alias info in GenInlinedIndexOf()." into lmp-dev

* commit '0fcd6873941d55d69c7cd9bb3b458d2230a9bd0b':
  X86: Fix alias info in GenInlinedIndexOf().

am a846c1ac: Merge "X86: Fix alias info in GenInlinedIndexOf()." into lmp-dev

* commit 'a846c1ac65b197844380b8d28377a850dfa1fc94':
  X86: Fix alias info in GenInlinedIndexOf().

am 2239f5e7: am c6a62592: Merge "Quick compiler: fix Arm64 inline locking code" into lmp-dev

* commit '2239f5e7542e62a78c814e0b1306537480d0e298':
  Quick compiler: fix Arm64 inline locking code

am c6a62592: Merge "Quick compiler: fix Arm64 inline locking code" into lmp-dev

* commit 'c6a62592836612b09ff1a8021e49b1328d85493b':
  Quick compiler: fix Arm64 inline locking code

Merge "Quick compiler: fix Arm64 inline locking code"

Quick compiler: fix Arm64 inline locking code

The Arm64 monitor enter code was incorrectly comparing the
thread ID to zero rather than the lock value to determine if the
inline fast path was doable.  As a result, we ended up always
using the out-of-line case.

internal b/16707253

Change-Id: I499530da84beecf1f3f5c5ebb666e3eb12599ac0

Merge "Enable large object space for zygote."

Enable large object space for zygote.

We now enable the large object space before the zygote fork.
This reduces the size of the zygote and removes the need for
excessive explicit GCs during phone booting.

Changed the card set mod union table to support forgetting cards.
If a card has no non null references which are in another space
then it is removed from the set.

Added logging of the zygote size when you do a SIGQUIT.

Dalvik PSS is the same or slightly lower (1-3%).

Zygote space size:
Before: 15MB
After: 8MB (+ some large objects).

TODO: Combine remembered sets and mod union tables into a single
interface.

Bug: 16398684
Change-Id: Ie48cdf35004a0a37eedb1ccc1bf214b1fa9e0cca

am a99a95b5: (-s ours) am d6ee5033: Merge "Make apps able to run with a failing patchoat" into lmp-dev

* commit 'a99a95b5df2d34ed5747fbb6085382b90d7f8737':
  Make apps able to run with a failing patchoat

Merge "Workaround problem reading main stack on intel devices."

Workaround problem reading main stack on intel devices.

This works around a problem where ART can't read the lowest page
of the main stack.  The workaround is to add 4K to the stack
start address on the main stack on Intel.

Please see https://b2.corp.google.com/issues/17111575 for
underlying cause.

Cherry picked from d970bac690baa6f735b0cd187440546869088a0f

Bug: 17031544
Change-Id: Ifc3216e10160bb1eec9d989fd57b10345bc89537

Merge "Merge stack usage reduction for overflow checks"

Merge "Quick compiler: fix Arm64 inline locking code" into lmp-dev

Quick compiler: fix Arm64 inline locking code

The Arm64 monitor enter code was incorrectly comparing the
thread ID to zero rather than the lock value to determine if the
inline fast path was doable.  As a result, we ended up always
using the out-of-line case.

This fix delivers a 6% improvement to DeltaBlue.

b/16707253

Change-Id: I499530da84beecf1f3f5c5ebb666e3eb12599ac0

am d6ee5033: Merge "Make apps able to run with a failing patchoat" into lmp-dev

* commit 'd6ee5033433619a893a8f20319efcf8c8b1d1016':
  Make apps able to run with a failing patchoat

am daf2d8da: am 20e4afeb: Merge "Workaround problem reading main stack on intel devices." into lmp-dev

* commit 'daf2d8daee2a6766860874402f9980dc25a11fce':
  Workaround problem reading main stack on intel devices.

am 20e4afeb: Merge "Workaround problem reading main stack on intel devices." into lmp-dev

* commit '20e4afeb1ad9d142fe9e46efe812f0d562c28f1f':
  Workaround problem reading main stack on intel devices.

am ed9fe4ff: Merge "Avoid handle-less methods in ClassLinker::LinkInterfaceMethods()."

* commit 'ed9fe4ffd9516cfb01b53a3a2c81d3e223ee352c':
  Avoid handle-less methods in ClassLinker::LinkInterfaceMethods().

am 64a88b25: am 38bd1e1b: ART: Fix ARM invoke stub, run Generic JNI in jni_compiler_test

* commit '64a88b25482180fdae5cba4b0feeaa176356b424':
  ART: Fix ARM invoke stub, run Generic JNI in jni_compiler_test

am 38bd1e1b: ART: Fix ARM invoke stub, run Generic JNI in jni_compiler_test

* commit '38bd1e1b8aa39bdc6e67c5dc205cc8d294d985c5':
  ART: Fix ARM invoke stub, run Generic JNI in jni_compiler_test

am 40be3b81: Merge "Make apps able to run with a failing patchoat"

* commit '40be3b816c860a42e2761e1a614762ea24d12a32':
  Make apps able to run with a failing patchoat

Merge "Workaround problem reading main stack on intel devices." into lmp-dev

am 57101be6: Merge "ART: Fix ARM invoke stub, run Generic JNI in jni_compiler_test"

* commit '57101be6db093d9d27776f77eb8880ae5bae9913':
  ART: Fix ARM invoke stub, run Generic JNI in jni_compiler_test

ART: Fix ARM invoke stub, run Generic JNI in jni_compiler_test

On devices which support it, run all jni compiler tests also with
the Generic JNI infrastructure.

Fix art_quick_invoke_stub for ARM32 to enforce 16B stack alignment.
As the native convention does not require as much, the previous
code would keep it unaligned.

Bug: 17107929

(cherry picked from commit 6e498695b60f1532d2264ec6badb1cd6e10ecaa9)

Change-Id: I1481500ff7c627ab4e5bfbfd7245099a639aa049

Workaround problem reading main stack on intel devices.

This works around a problem where ART can't read the lowest page
of the main stack.  The workaround is to add 4K to the stack
start address on the main stack on Intel.

Please see https://b2.corp.google.com/issues/17111575 for
underlying cause.

Bug: 17031544
Change-Id: Ifc3216e10160bb1eec9d989fd57b10345bc89537

am e30100f1: Merge "ART: Set default visibility to protected"

* commit 'e30100f164f8ac46f0428c28be87ad289adf4ac2':
  ART: Set default visibility to protected

am 1a8f106d: am e22c03ce: Merge "ART: Set default visibility to protected" into lmp-dev

* commit '1a8f106d36539781f286b56fdecdc0de706b251a':
  ART: Set default visibility to protected

am e22c03ce: Merge "ART: Set default visibility to protected" into lmp-dev

* commit 'e22c03ceaa847e3b6c8fcb7ee9d838f61945cda7':
  ART: Set default visibility to protected

am fd5a3f79: Merge "ART: More failure output on run-test"

* commit 'fd5a3f796797c32c0346d13589f25cfd8bc4e465':
  ART: More failure output on run-test

am 82771c03: Merge "Check for constructor first when installing tracing stubs."

* commit '82771c03ac7e5ed01aec036ff894235bc4dd0cd4':
  Check for constructor first when installing tracing stubs.

am 614fbb86: am 5afe466c: Check for constructor first when installing tracing stubs.

* commit '614fbb8630351c8a69345fd7bc6bef08cefe3658':
  Check for constructor first when installing tracing stubs.

am 5afe466c: Check for constructor first when installing tracing stubs.

* commit '5afe466c027c6ac821bf7c9a88cd7de5260a639b':
  Check for constructor first when installing tracing stubs.

Check for constructor first when installing tracing stubs.

Addresses comments in
https://android-review.googlesource.com/#/c/104383/ made after the commit.

Bug: 16386215
Change-Id: I0e59b8507f601fd6508fbda8d24dd3c98ead1537

Merge stack usage reduction for overflow checks

Cherry pick from: b038ba66a166fb264ca121632f447712e0973b5b

Change-Id: I19999e72ff731d4fc73d91b9ff767de5858c15ee

Merge "ART: Fix ARM invoke stub, run Generic JNI in jni_compiler_test"

ART: Fix ARM invoke stub, run Generic JNI in jni_compiler_test

On devices which support it, run all jni compiler tests also with
the Generic JNI infrastructure.

Fix art_quick_invoke_stub for ARM32 to enforce 16B stack alignment.
As the native convention does not require as much, the previous
code would keep it unaligned.

Bug: 17107929
Change-Id: Ibce423949846e447d2b0246d07973b6b717691ac

Merge "ART: Set default visibility to protected" into lmp-dev

am 75dd9883: Merge "Fix JDWP event races"

* commit '75dd98835859ca98c9f98339da8b64640d09a175':
  Fix JDWP event races

am 9be4be58: Merge "AArch64: Add tbz/tbnz and tst."

* commit '9be4be5882a05b3aa6ec822901ff4092203d8dc5':
  AArch64: Add tbz/tbnz and tst.

am b71adf18: am 214f4f15: Merge "Fix JDWP event races" into lmp-dev

* commit 'b71adf18363d852f568b1ca125f5fdd468760e1f':
  Fix JDWP event races

am 214f4f15: Merge "Fix JDWP event races" into lmp-dev

* commit '214f4f15fe0137c285a05b3f70e52a211aecfdb5':
  Fix JDWP event races

Merge "ART: Relax GetInstructionSetFromString" into lmp-dev

ART: Relax GetInstructionSetFromString

Do not abort on an unknown instruction set string. Instead return
kNone and let the caller handle this.

Also simplify the patchoat tool to use this.

Bug: 17136416
Change-Id: I24131914bcf91c04ae93179bf809a2907f1f2b7a

Merge "Make apps able to run with a failing patchoat" into lmp-dev

Make apps able to run with a failing patchoat

Bug: 17000769

(cherry picked from commit 9dcc4572949f6a8231a1b4ed859676ba6f411726)

Change-Id: I0a1a4dc7f5d4bb268530840302ecfb1555231e05

am 562f0ef6: Merge "Fix heap corruption and verification."

* commit '562f0ef60bb1498fdf3c43fe747ba067ffe6ebed':
  Fix heap corruption and verification.

am 5638b7ee: am 5a001b3e: Merge "AttachCurrentThread should only warn on missing thread name if CheckJNI is enabled." into lmp-dev

* commit '5638b7ee01bb7e64662fcf4beffd14523488d6bd':
  AttachCurrentThread should only warn on missing thread name if CheckJNI is enabled.

am 5a001b3e: Merge "AttachCurrentThread should only warn on missing thread name if CheckJNI is enabled." into lmp-dev

* commit '5a001b3e4a4b4d066fd987071eb38b2b97493807':
  AttachCurrentThread should only warn on missing thread name if CheckJNI is enabled.

am 7df1754a: Merge "Really fix target gtest dependencies"

* commit '7df1754ae17acd891995159fc50f3bb176b770dd':
  Really fix target gtest dependencies

am 7968bfc8: Merge "AttachCurrentThread should only warn on missing thread name if CheckJNI is enabled."

* commit '7968bfc840c926fbb1e4955b165e6eeb67bcc3f5':
  AttachCurrentThread should only warn on missing thread name if CheckJNI is enabled.

am 84438f89: am 6724d864: Fix heap corruption and verification.

* commit '84438f891c83a064b243d89e57b958770f86323d':
  Fix heap corruption and verification.

am 6724d864: Fix heap corruption and verification.

* commit '6724d8649ab73e4fb86c8014bda51b13bddf2f3f':
  Fix heap corruption and verification.

Merge "Really fix target gtest dependencies"

Fix heap corruption and verification.

There was a faulty assumption that space End() was always aligned to
card boundaries. This was true for all spaces other than the image
and resulted in heap corruption when the last object of the image
space contained a reference to an object in another space.

Also fixed an error where we called the pre GC verification post GC.

Bug: 17080623

Change-Id: I041ee564518f53b79c6e8dc2ad782a3152577a4e

Really fix target gtest dependencies

In 64006834b26f7138131fac87c3b0973cda70e560 we started depending on
TARGET_CORE_JARS which are phony targets and undefined when using
"mm test-art-target".

Change-Id: I671c5d6068ff999782ad5f388f3077f4c1594884

AttachCurrentThread should only warn on missing thread name if CheckJNI is enabled.

Bug: 17108693
Change-Id: I346cc090e091605c552e5954d7ea1ca4874cea5f

Merge "Make apps able to run with a failing patchoat"

Make apps able to run with a failing patchoat

Bug: 17000769

Change-Id: I0a1a4dc7f5d4bb268530840302ecfb1555231e05

Merge "ART: Set default visibility to protected"

ART: Set default visibility to protected

Set default visibility of symbols to protected. This allows the
linker to optimize internal calls and helps avoid plt calls.

Make almost all assembly stubs hidden, as -fvisibility does not
seem to apply to them. Change the assembly tests accordingly. Also
allows to clean up previous hacks to avoid plt calls.

Bug: 16974467

(cherry picked from commit 235e77bd9f19e4faefda109be40f8744f3a66f40)

Change-Id: I9030dcf6116251f434f94a2b08e56e12085af652

Merge "ART: More failure output on run-test"

ART: More failure output on run-test

Change-Id: Ifdf1516e950117069b3cfa58c4bda03281dad661

Merge "Check for constructor first when installing tracing stubs."

Check for constructor first when installing tracing stubs.

Addresses comments in
https://android-review.googlesource.com/#/c/104383/ made after the commit.

Bug: 16386215

(cherry picked from commit 5afe466c027c6ac821bf7c9a88cd7de5260a639b)

Change-Id: Ic81161d0b96a0633c4c534a93172cff59312fa61

Merge "Fix heap corruption and verification."

Fix heap corruption and verification.

There was a faulty assumption that space End() was always aligned to
card boundaries. This was true for all spaces other than the image
and resulted in heap corruption when the last object of the image
space contained a reference to an object in another space.

Also fixed an error where we called the pre GC verification post GC.

Bug: 17080623

(cherry picked from commit 6724d8649ab73e4fb86c8014bda51b13bddf2f3f)

Change-Id: I931149421c738653863eb6105dfdf7ecd750601a

Merge "AttachCurrentThread should only warn on missing thread name if CheckJNI is enabled."

am 76bc718d: Merge "[MIPS] Fix the code generated by GenSelectConst32"

* commit '76bc718dc2cf0279ec5afd08c3a82b6ad61756d8':
  [MIPS] Fix the code generated by GenSelectConst32

am 92fd388b: Merge "Check that we don\'t accidentally invoke compiled code when -Xint."

* commit '92fd388b9bb433a54e9e7a4e6ab9415f59d8956c':
  Check that we don't accidentally invoke compiled code when -Xint.

am 506a79fd: Merge "Optimized instrumentation listener handling"

* commit '506a79fd8a698911b5a82575eed59b9320ae97cd':
  Optimized instrumentation listener handling

am 90b180ee: Merge "ART: Do not recursively abort when visiting locks in a bad state"

* commit '90b180ee1edd17b8dca34ea9e58624818d2ee15a':
  ART: Do not recursively abort when visiting locks in a bad state

am 9d352c06: Merge "ART: Avoid expensive class lookup in compiler driver"

* commit '9d352c06d8b6e9ea6343cc4f361a3309c2ce7ebe':
  ART: Avoid expensive class lookup in compiler driver

am 2311e3db: am dfa7a6c8: Disable load/store elimination. DO NOT MERGE

* commit '2311e3db454dcad1546a1b67f0443b3b742b5d7f':
  Disable load/store elimination. DO NOT MERGE

am dfa7a6c8: Disable load/store elimination. DO NOT MERGE

* commit 'dfa7a6c8711623bf20aad58f57ae980b497ef4d7':
  Disable load/store elimination. DO NOT MERGE

Disable load/store elimination. DO NOT MERGE

The optimization has several issues and it has been
effectively disabled by
    https://android-review.googlesource.com/103511
Disable it completely to avoid wasting compilation time.

Bug: 16862804
Change-Id: I42f69ae64170dc44f5fd6455b6a51c9bd3f93672

am 76288481: Merge "Update ziparchive usage to the new API."

* commit '7628848182492bdcdcd7e4161680a4e24ce9486e':
  Update ziparchive usage to the new API.

am d75383af: am 5e8a63ae: Merge "ART: Do not recursively abort when visiting locks in a bad state" into lmp-dev

* commit 'd75383af2f0843ecd2f493fd64b4eb38630c6071':
  ART: Do not recursively abort when visiting locks in a bad state

am 5e8a63ae: Merge "ART: Do not recursively abort when visiting locks in a bad state" into lmp-dev

* commit '5e8a63ae39cb5db996229c6c4a1a21a4b8dec5a2':
  ART: Do not recursively abort when visiting locks in a bad state