am 90d7bc05: am bc576cfd: Merge "Revert "Optimizing String.Equals as an intrinsic (x86)""

* commit '90d7bc0546059e58e8ff76a3b973473b4f5b1801':
  Revert "Optimizing String.Equals as an intrinsic (x86)"

am bc576cfd: Merge "Revert "Optimizing String.Equals as an intrinsic (x86)""

* commit 'bc576cfd9c85a285f15b8c33a2cc7f947d999f49':
  Revert "Optimizing String.Equals as an intrinsic (x86)"

Merge "Revert "Optimizing String.Equals as an intrinsic (x86)""

Revert "Optimizing String.Equals as an intrinsic (x86)"

Reverted as it breaks the compilation of boot.{oat,art} on x86 (although this CL may not be the culprit, as the issue seems to come from Optimizing's register allocator).

This reverts commit 8ab7bd6c8b10ad58758c33a1dc9326212bd200e9.

Change-Id: If7c8b6258d1e690f4d2a06bcc82c92563ac6cdef

am adcc1f20: am 855223fa: Merge "Fix ambiguous phrasing in tests comments regarding $."

* commit 'adcc1f20fe01134301b13769b9595fd78e4ffc35':
  Fix ambiguous phrasing in tests comments regarding $opt$.

am 855223fa: Merge "Fix ambiguous phrasing in tests comments regarding $."

* commit '855223fa12fc1efc926c068ea5c536663f30030d':
  Fix ambiguous phrasing in tests comments regarding $opt$.

Merge "Fix ambiguous phrasing in tests comments regarding $opt$."

Fix ambiguous phrasing in tests comments regarding $opt$.

Change-Id: I78e6bccbe35288df5de30700d3877bbcdef37d65

am a63809f1: am 7a58566d: Merge "Optimizing String.Equals as an intrinsic (x86)"

* commit 'a63809f1e6d8a6333e60d35996ac16fea5645947':
  Optimizing String.Equals as an intrinsic (x86)

am 7a58566d: Merge "Optimizing String.Equals as an intrinsic (x86)"

* commit '7a58566d2b0ad051b8ccaf5614ad986979af8ef4':
  Optimizing String.Equals as an intrinsic (x86)

Merge "Optimizing String.Equals as an intrinsic (x86)"

am 70b593a4: am 7879f711: Merge "ART: Check a RETURN_VOID_NO_BARRIER like a RETURN_VOID"

* commit '70b593a4754b09df15bba2a3bf90bfa964d47c8f':
  ART: Check a RETURN_VOID_NO_BARRIER like a RETURN_VOID

am 7879f711: Merge "ART: Check a RETURN_VOID_NO_BARRIER like a RETURN_VOID"

* commit '7879f7119d7a88a71f6c31c9b8eac27cdba8a123':
  ART: Check a RETURN_VOID_NO_BARRIER like a RETURN_VOID

Merge "ART: Check a RETURN_VOID_NO_BARRIER like a RETURN_VOID"

ART: Check a RETURN_VOID_NO_BARRIER like a RETURN_VOID

RETURN_VOID_NO_BARRIER is used in all methods with return-void,
as long as they're not a constructor and have been quickened. As
such, do the regular checking of the return type.

Bug: https://code.google.com/p/android/issues/detail?id=181700
Bug: 22871329
Change-Id: I11bcf7bffd91fc01995b703a99e603ddb370016e

am 43498ce6: am 66c50ccc: Merge "runtime: cleanup class_linker out-parameters and formatting"

* commit '43498ce63eefeb2c3fad59592ce240c42d655441':
  runtime: cleanup class_linker out-parameters and formatting

am 9b0483d5: am 5cdee205: Merge "Revert "runtime: cleanup class_linker out-parameters and formatting""

* commit '9b0483d5ac74f846c0cbd45779e758c942085ee5':
  Revert "runtime: cleanup class_linker out-parameters and formatting"

am 0c607a99: am 9c977b92: Merge "runtime: cleanup class_linker out-parameters and formatting"

* commit '0c607a9956b934f2716e2687de22e52624152def':
  runtime: cleanup class_linker out-parameters and formatting

am 66c50ccc: Merge "runtime: cleanup class_linker out-parameters and formatting"

* commit '66c50cccd9dda431b2017ad04dc72b0c0f17d74e':
  runtime: cleanup class_linker out-parameters and formatting

am 5cdee205: Merge "Revert "runtime: cleanup class_linker out-parameters and formatting""

* commit '5cdee205231993c5f3a5f815f614d9d460448615':
  Revert "runtime: cleanup class_linker out-parameters and formatting"

am 9c977b92: Merge "runtime: cleanup class_linker out-parameters and formatting"

* commit '9c977b9260927d14c6d6c5bbdd7d16e1aebbf2fa':
  runtime: cleanup class_linker out-parameters and formatting

Merge "runtime: cleanup class_linker out-parameters and formatting"

runtime: cleanup class_linker out-parameters and formatting

* Use out<T> instead of parameters
* Fixes up some other signatures to be more correct
* Reformat parameters to be one per line if they can't fit on one line
* Reformat locks to be one per line if they can't fit on the decl line

Change-Id: Ib71b08707d3ed0bb85299406c0b23a1de4e92a1c

Merge "Revert "runtime: cleanup class_linker out-parameters and formatting""

Revert "runtime: cleanup class_linker out-parameters and formatting"

This reverts commit 0de694ed3efe8412b83622a24074831edcc7e384.

Change-Id: I4b185584d6e47918179b16d76709cc3366872f3c

Merge "runtime: cleanup class_linker out-parameters and formatting"

Optimizing String.Equals as an intrinsic (x86)

The third implementation of String.Equals.  I added an intrinsic
in x86 which is similar to the original java implementation of
String.equals: an instanceof check, null check, length check, and
reference equality check followed by a loop comparing strings
character by character.

Interesting Benchmarking Values:

Optimizing Compiler on Nexus Player
Intrinsic 15-30 Character Strings: 177 ns
Original 15-30 Character Strings: 275 ns
Intrinsic Null Argument: 59 ns
Original Null Argument: 137 ns
Intrinsic 100-1000 Character Strings: 1812 ns
Original 100-1000 Character Strings: 6334 ns

Bug: 21481923
Change-Id: Ia386e19b9dbfe0dac688b20ec93d8f90f67af47e

am f48bd295: am 0d72ce0b: Merge "Revert "ART: Do not allow RETURN_VOID_NO_BARRIER outside constructors""

* commit 'f48bd295919676d53e206f477e87d9ab38d9192e':
  Revert "ART: Do not allow RETURN_VOID_NO_BARRIER outside constructors"

runtime: cleanup class_linker out-parameters and formatting

* Use out<T> instead of parameters
* Fixes up some other signatures to be more correct
* Reformat parameters to be one per line if they can't fit on one line
* Reformat locks to be one per line if they can't fit on the decl line

Change-Id: I1031cf4edc41f95b89edb343d43ef93e15384206

am 0d72ce0b: Merge "Revert "ART: Do not allow RETURN_VOID_NO_BARRIER outside constructors""

* commit '0d72ce0b4f9eaf0f60c70a2901198099086795e0':
  Revert "ART: Do not allow RETURN_VOID_NO_BARRIER outside constructors"

Merge "Revert "ART: Do not allow RETURN_VOID_NO_BARRIER outside constructors""

Revert "ART: Do not allow RETURN_VOID_NO_BARRIER outside constructors"

This reverts commit e93505f599682a2e95c4dac1c8633dec79c602bc.

This isn't the right thing to do, actually.

Bug: 22871329
Change-Id: I0a0a5a86c7c8d8afdf417444ca7d73e56e040515

am e344f24b: am 7423f0b6: Merge "ART: Do not allow RETURN_VOID_NO_BARRIER outside constructors"

* commit 'e344f24bed7d3379c4e327987329fbed13404dde':
  ART: Do not allow RETURN_VOID_NO_BARRIER outside constructors

am 7423f0b6: Merge "ART: Do not allow RETURN_VOID_NO_BARRIER outside constructors"

* commit '7423f0b66060711cddcdbc7ba5b0dd3f2df36718':
  ART: Do not allow RETURN_VOID_NO_BARRIER outside constructors

Merge "ART: Do not allow RETURN_VOID_NO_BARRIER outside constructors"

ART: Do not allow RETURN_VOID_NO_BARRIER outside constructors

Let the verifier fail non-constructor methods using the no-barrier
variant of return. This is a quickened return purely to avoid the
constructor barrier for final fields, and cannot be used as a stand-in
for a regular return.

Bug: https://code.google.com/p/android/issues/detail?id=181700
Bug: 22871329
Change-Id: I9559c6a1599c20ce526582527ebed73b25242767

am ebf2ade8: Merge branch \'stage-aosp-master\' of https://googleplex-android.googlesource.com/_direct/platform/art into stage-aosp-master

* commit 'ebf2ade8226fe9bcdd4ef6f7560aa329c458d2ea':
  ART: Remove unique-numbered labels from arm64 assembly

am c548ae75: ART: Remove unique-numbered labels from arm64 assembly

* commit 'c548ae756a3131b1a7c9abc387697218837e29d3':
  ART: Remove unique-numbered labels from arm64 assembly

Merge branch 'stage-aosp-master' of https://googleplex-android.googlesource.com/_direct/platform/art into stage-aosp-master

ART: Remove unique-numbered labels from arm64 assembly

Use local labels like 1, 2, 3 in macros. Clang does not support
the unique counter variable \@.

Bug: 22799850
Change-Id: I9ef1859be40b875ef4d7ae74ec1af52ad05f7352

am 8d72e5d9: Merge "ART: Remove unique-numbered labels from arm64 assembly"

* commit '8d72e5d9cb61c9517d2e96776e4443427c66fd0d':
  ART: Remove unique-numbered labels from arm64 assembly

am dee5a875: am 68c5640a: Merge "Added repe_cmpsl instruction to x86, x86_64 assemblers"

* commit 'dee5a875cf48d4eb6579ddcf9b7bd13919cd3bd2':
  Added repe_cmpsl instruction to x86, x86_64 assemblers

am 68c5640a: Merge "Added repe_cmpsl instruction to x86, x86_64 assemblers"

* commit '68c5640a71cfb37f8741eeafbfc13b2a40ccefa1':
  Added repe_cmpsl instruction to x86, x86_64 assemblers

Merge "ART: Remove unique-numbered labels from arm64 assembly"

ART: Remove unique-numbered labels from arm64 assembly

Use local labels like 1, 2, 3 in macros. Clang does not support
the unique counter variable \@.

Bug: 22799850
Change-Id: I9ef1859be40b875ef4d7ae74ec1af52ad05f7352

Merge "Added repe_cmpsl instruction to x86, x86_64 assemblers"

Added repe_cmpsl instruction to x86, x86_64 assemblers

Support for this instruction has already been added to the disassembler
in commit 124b392d.

Change-Id: I6e8401a7b814618758427f5cc6b4992e265f937c

am daa4eb26: am 5a898e67: Merge "Optimizing: Replace x86 xchg use with xor sequence"

* commit 'daa4eb267379e23d752b869777476a92689d9092':
  Optimizing: Replace x86 xchg use with xor sequence

am 5a898e67: Merge "Optimizing: Replace x86 xchg use with xor sequence"

* commit '5a898e67bb0dec1fc7f2d9b5b32eb8c956a84804':
  Optimizing: Replace x86 xchg use with xor sequence

Merge "Optimizing: Replace x86 xchg use with xor sequence"

am 6c835368: am b669c6ee: Merge "ART: Change stream output kNone intrinsic"

* commit '6c835368072c115bc07cf6c495396c998061bbfa':
  ART: Change stream output kNone intrinsic

am b669c6ee: Merge "ART: Change stream output kNone intrinsic"

* commit 'b669c6ee7a942bdf103be8eae7e19590a4419db4':
  ART: Change stream output kNone intrinsic

Merge "ART: Change stream output kNone intrinsic"

ART: Change stream output kNone intrinsic

Name of intrinsics is dumped with C1visualizer and checked with
Checker whose attributes should not contain whitespace. This patch
changes the output printed for non-intrinsified invokes.

Change-Id: I3e565e8c9e26eb61026e7a13823eab20409dd63a

am 43cb410c: am cb34698f: Merge "Do not use -DUSE_*MALLOC for host build"

* commit '43cb410c355ec9f637b5cae09891f519cc01ec70':
  Do not use -DUSE_*MALLOC for host build

am cb34698f: Merge "Do not use -DUSE_*MALLOC for host build"

* commit 'cb34698f42ed688ec98bf3c9562075237ee72490':
  Do not use -DUSE_*MALLOC for host build

Merge "Do not use -DUSE_*MALLOC for host build"

am 9176fac9: am 8e568d07: Merge "Added disassembler support for repe_cmpsw instruction in x86, x86_64"

* commit '9176fac916dd06c7f98a951c8114caec28d363ce':
  Added disassembler support for repe_cmpsw instruction in x86, x86_64

am 8a0da1d8: am ad697bda: Merge "base: replace raw pointers for out-parameters with safer out<T>"

* commit '8a0da1d8f43d4106d20d25e598fc6b5d87024013':
  base: replace raw pointers for out-parameters with safer out<T>

am 8e568d07: Merge "Added disassembler support for repe_cmpsw instruction in x86, x86_64"

* commit '8e568d079b052db118d83e6e89ea36ab4fa4bfb1':
  Added disassembler support for repe_cmpsw instruction in x86, x86_64

am ad697bda: Merge "base: replace raw pointers for out-parameters with safer out<T>"

* commit 'ad697bda14eae8615e2b6110aec0118625983acd':
  base: replace raw pointers for out-parameters with safer out<T>

Merge "Added disassembler support for repe_cmpsw instruction in x86, x86_64"

Merge "base: replace raw pointers for out-parameters with safer out<T>"

am 10ee11ab: am 59c59d07: Merge "Don\'t dump stack when deleting an invalid local ref"

* commit '10ee11ab3b269e300229f658764c7038a1aec08e':
  Don't dump stack when deleting an invalid local ref

am 59c59d07: Merge "Don\'t dump stack when deleting an invalid local ref"

* commit '59c59d07900e639ec7eeffe9e3f54ec30a2fb43f':
  Don't dump stack when deleting an invalid local ref

Merge "Don't dump stack when deleting an invalid local ref"

am 2ada67b3: Don\'t dump stack when deleting an invalid local ref

* commit '2ada67b3145d38b352297bdb9d0546f58882e208':
  Don't dump stack when deleting an invalid local ref

Don't dump stack when deleting an invalid local ref

Bug: 21724724

(cherry picked from commit 2ada67b3145d38b352297bdb9d0546f58882e208)

Change-Id: Ifa99df9ee9f3265038a0c0f7944e834e42c498d4

base: replace raw pointers for out-parameters with safer out<T>

Add a zero-cost type-safe abstraction for representing "out" parameters
(i.e. when the calling function has to return multiple results out
by-reference into the argument slots instead of using the return slot).

Change-Id: I33a941e4863b6bed71d2bfa43d7f48e9b111f83f

Added disassembler support for repe_cmpsw instruction in x86, x86_64

Also included support for repe_cmpsl instruction. This is a follow up to
commit 71311f868e2 which added support for repe_cmpsw in the x86 and
x86_64 assemblers.

Change-Id: I2beac05a57341539acf96cdf77062facd031a864

Don't dump stack when deleting an invalid local ref

Bug: 21724724
Change-Id: Ifa99df9ee9f3265038a0c0f7944e834e42c498d4

am 57d92494: am 8433bb8a: Merge "ART: Fix gtest after GraphChecker CL"

* commit '57d924942f5cac9d7d15ecabe634970ca3ae244a':
  ART: Fix gtest after GraphChecker CL

am 8433bb8a: Merge "ART: Fix gtest after GraphChecker CL"

* commit '8433bb8a3120d064de97a692d98341d9bc29a985':
  ART: Fix gtest after GraphChecker CL

Merge "ART: Fix gtest after GraphChecker CL"

ART: Fix gtest after GraphChecker CL

Change-Id: Ia85cb9622b3927c401325d87f2d7a52096b7fe5d

Merge "resolved conflicts for merge of 8f96df84 to mnc-dev-plus-aosp" into mnc-dev-plus-aosp

am 430dd633: am 5d2ed003: Merge "ART: Use __ANDROID__ instead of HAVE_ANDROID_OS"

* commit '430dd633e3334c6d684c17218435a881ef4510c4':
  ART: Use __ANDROID__ instead of HAVE_ANDROID_OS

am 5fa76856: am a302e913: Merge "ART: Store and check exceptional predecessors"

* commit '5fa76856e271ea6820013a4f6e67324069474e0e':
  ART: Store and check exceptional predecessors

resolved conflicts for merge of 8f96df84 to mnc-dev-plus-aosp

Change-Id: Ib474b16ad1ddf3e823405df977d8341f9cdd3774

am 5d2ed003: Merge "ART: Use __ANDROID__ instead of HAVE_ANDROID_OS"

* commit '5d2ed003020feee437683b84e4ea6b8c6a5753e0':
  ART: Use __ANDROID__ instead of HAVE_ANDROID_OS

am a302e913: Merge "ART: Store and check exceptional predecessors"

* commit 'a302e9133b659d9b96066aa53f9d22d35d2dba13':
  ART: Store and check exceptional predecessors

Merge "ART: Use __ANDROID__ instead of HAVE_ANDROID_OS"

Merge "ART: Store and check exceptional predecessors"

ART: Store and check exceptional predecessors

Future CL on register allocation for try/catch will require the
knowledge of instructions which throw into a catch block. This patch
stores that information with the basic block and verifies it in the
graph checker.

More checks on try catch also added to the graph checker and an order
of exception handlers is enforced in TryBoundary successors.

Change-Id: I3034c610791ea51d96724bcca97f49ec6ecf2af3

ART: Use __ANDROID__ instead of HAVE_ANDROID_OS

Use the proper define.

Change-Id: I71e291ac25f5d5f0187ac9b6ef2d6872f19e6085

Optimizing: Replace x86 xchg use with xor sequence

On some x86 processors, xchg is serializing even when exchanging two
registers.  Replace the xchgl use with the 3 xor sequence to swap to
registers.  This is generally faster and doesn't serialize the machine.

Change-Id: Iea2cd993d3b70a103bbdd1dbf7818e26ae29387c
Signed-off-by: Mark Mendell <mark.p.mendell@intel.com>

Allow for fine tuning the inliner.

Bug: 21868508

(cherry picked and squashed from commits
ec74835a7e4f2660250a2f3f9508cbbe5269e49a and
0941b9d48a9a8c6d80a1af7a0d0fc9f80fe2b9a1)

Change-Id: I1750e6bea20321d04680132281a6c2924531c5ae

Do not use -DUSE_*MALLOC for host build

When we use ninja and switch a lunch target to another, they
may cause unnecessary rebuild because ninja detects command
line changes. They are used in #ifdef HAVE_ANDROID_OS so there
should be no semantics changes.

Change-Id: I629c94c872c8fc8ba4b4c59b565338e14570c3fe

am 4111d09c: am 7b926cda: Merge "Clear temporary class arrays before linking the new class"

* commit '4111d09c87d72075518c1a11b8b5c5bfe7c984aa':
  Clear temporary class arrays before linking the new class

am 7b926cda: Merge "Clear temporary class arrays before linking the new class"

* commit '7b926cdacc2b67241bc9cb5f2d4b04b13ca79d0e':
  Clear temporary class arrays before linking the new class

Merge "Clear temporary class arrays before linking the new class"

Clear temporary class arrays before linking the new class

Fixes DCHECK failure from remembered sets where two classes had the
same field array which caused the remembered set to incorrectly
remove a card with a reference to the target space.

Change-Id: If43875616fb750e20667212381bc7e359c4214a5

am 47f6198e: am 3247ce51: Merge "Register signal handler to kernel if not claimed"

* commit '47f6198ee2f1ffb9fc495ea490f0d1d7dbdeae5a':
  Register signal handler to kernel if not claimed

am c1ba0605: am 1569aad2: Merge "ART: Relax IsInstantiable check"

* commit 'c1ba06054736f990c743ac53c826ca4bc3c2f9a4':
  ART: Relax IsInstantiable check

am 3247ce51: Merge "Register signal handler to kernel if not claimed"

* commit '3247ce5195f4233c484254b41a1fcc6cd1e6db0a':
  Register signal handler to kernel if not claimed

Merge "Register signal handler to kernel if not claimed"

am 1569aad2: Merge "ART: Relax IsInstantiable check"

* commit '1569aad28638334019ecd3d649ff2f28f2921d2c':
  ART: Relax IsInstantiable check

Merge "ART: Relax IsInstantiable check"

am 60ee2840: am 36c51fc2: Merge "Take into account that we might end up with unresolved classes."

* commit '60ee28406932387c715adce35d4c6cc60a52f607':
  Take into account that we might end up with unresolved classes.

am 36c51fc2: Merge "Take into account that we might end up with unresolved classes."

* commit '36c51fc23f7d7330440ecf8de54e69bb68969c05':
  Take into account that we might end up with unresolved classes.