am e786a7ba: am 2619371b: am 2c68641c: Merge "ART: Merge entries with same name and protect in MemMap dump."

* commit 'e786a7babc291a13e8338012bb1d503ba1be494b':
  ART: Merge entries with same name and protect in MemMap dump.

am 2619371b: am 2c68641c: Merge "ART: Merge entries with same name and protect in MemMap dump."

* commit '2619371bd4dc0eab0862b193f0fbfc90706f2278':
  ART: Merge entries with same name and protect in MemMap dump.

am 2c68641c: Merge "ART: Merge entries with same name and protect in MemMap dump."

* commit '2c68641c0f0d8fb410cf73182acb4e498c0e558f':
  ART: Merge entries with same name and protect in MemMap dump.

Merge "ART: Merge entries with same name and protect in MemMap dump."

ART: Merge entries with same name and protect in MemMap dump.

This should make the MemMap dump less chatty and allow the
logger to keep more relevant output.

Bug: 20873174
Change-Id: I09058798faa8f79d154cee5008017b03700df8db

am 00dcbc36: am 8ee43e3c: ART: Fix debug output for bad reused MemMap.

* commit '00dcbc3651b53a930e9003b3f2e6d052e4624aa2':
  ART: Fix debug output for bad reused MemMap.

am aa41220a: am 5eae4555: Support multiple instrumentation clients

* commit 'aa41220a9ebdc149b2e8ef069a9f2bcb9f7cd7e5':
  Support multiple instrumentation clients

am 8ee43e3c: ART: Fix debug output for bad reused MemMap.

* commit '8ee43e3ce18e90bfbf8cbeb05ff78ad281015d1a':
  ART: Fix debug output for bad reused MemMap.

am 5eae4555: Support multiple instrumentation clients

* commit '5eae455507399286c845ba54796c47087a72b3e2':
  Support multiple instrumentation clients

am c744a5d3: am 7381073d: am 6727a481: Merge "Skip compilation when falling-through out of method code."

* commit 'c744a5d36002614ec4337f240359fb09a23374d9':
  Skip compilation when falling-through out of method code.

am 7381073d: am 6727a481: Merge "Skip compilation when falling-through out of method code."

* commit '7381073d52bb1598674e5eb3af483780b1848dcd':
  Skip compilation when falling-through out of method code.

am 863fd4ae: am 1f927a86: am add1d9ba: Merge "Fix typo in comments."

* commit '863fd4aea0b4270e841ba867e0f7d1762e4613bb':
  Fix typo in comments.

am 5c34846a: am 0b4408d5: am 2e8da8ec: Merge "ART: Fix debug output for bad reused MemMap."

* commit '5c34846aec68e55dd7e5a77bb6f2fa8d9ab536bd':
  ART: Fix debug output for bad reused MemMap.

am 7f951ac5: am e15eea0b: am 659f4c14: Merge "ART: Refactor 082-inline-execute"

* commit '7f951ac53315f8abc477f3c7fc18750d87e95dea':
  ART: Refactor 082-inline-execute

am 6727a481: Merge "Skip compilation when falling-through out of method code."

* commit '6727a48193db2a0cf01af971cccffe1a6518c247':
  Skip compilation when falling-through out of method code.

Merge "Skip compilation when falling-through out of method code."

am 1f927a86: am add1d9ba: Merge "Fix typo in comments."

* commit '1f927a86e3e70619a0005eb976c229c9dbfda9e3':
  Fix typo in comments.

am add1d9ba: Merge "Fix typo in comments."

* commit 'add1d9ba61b6ddd52f2161587baf3ae4bc8717af':
  Fix typo in comments.

Merge "Fix typo in comments."

am 0b4408d5: am 2e8da8ec: Merge "ART: Fix debug output for bad reused MemMap."

* commit '0b4408d5587af57c76cd7f2af2d975f4eae20f8e':
  ART: Fix debug output for bad reused MemMap.

am 2e8da8ec: Merge "ART: Fix debug output for bad reused MemMap."

* commit '2e8da8ecb94134081c3ea18a217294db367888a3':
  ART: Fix debug output for bad reused MemMap.

ART: Fix debug output for bad reused MemMap.

Log the error_msg instead of a pointer to it.
Raise the log level to ERROR for the process maps.

(cherry picked from commit b5505823ad3d0bab8fc9c15c266a3d8cd96ee59e)

Change-Id: Ib0c258961e2384125907aca4bae2108f36067d5f

Merge "ART: Fix debug output for bad reused MemMap."

ART: Fix debug output for bad reused MemMap.

Log the error_msg instead of a pointer to it.
Raise the log level to ERROR for the process maps.

Change-Id: I95f82f013d1c6360f40e6315652d1a2dda0f39e2

am e15eea0b: am 659f4c14: Merge "ART: Refactor 082-inline-execute"

* commit 'e15eea0b6548fe933b5a22d5d9b2c97908399403':
  ART: Refactor 082-inline-execute

am 659f4c14: Merge "ART: Refactor 082-inline-execute"

* commit '659f4c14e8e044f8efe5405ba0da50aa894dd25f':
  ART: Refactor 082-inline-execute

Merge "ART: Refactor 082-inline-execute"

ART: Refactor 082-inline-execute

Refactor the indexOf intrinsics tests so that the optimizing
compiler would actually compile them.

Bug: 20889065
Change-Id: I69bfda7fa3eb4ce42c593203731e3ddd61f7e1ed

am 1df8987a: am c3cde2c0: JDWP: properly combine location events

* commit '1df8987ae5ac56eeaf50eaca9bf0dd284126b3dc':
  JDWP: properly combine location events

am c3cde2c0: JDWP: properly combine location events

* commit 'c3cde2c00bcbe97c9b0c119919500fcbbe824baa':
  JDWP: properly combine location events

Support multiple instrumentation clients

Changes Instrumentation::ConfigureStubs to support multiple clients
that need different levels of instrumenation. A client is identified
by a string key used to save the desired instrumentation level.

Also adds regression gtest instrumentation_test and some cleanup.

Bug: 19829329

(cherry picked from commit 0462c4c87c39db6cfcd338f323844738109ac3c9)

Change-Id: I1fc24a86fcb7cb46d4be806895376c25cc0a0b3c

JDWP: properly combine location events

This CL properly groups JDWP events at the same location: Breakpoint,
Single-step, Method Entry and Method Exit. This is necessary if the
debugger is not the only instrumentation listener. This matches the
behavior of Dalvik, especially for methods with a single return
instruction.

The interpreter was tuned so the instrumentation callbacks were
called to satisfy the debugger with the idea the debugger was the
only instrumentation listener. This is not true when method tracing
is enabled at the same time.
When tracing is enabled, there is always a listener for MethodEntry
and MethodExit events (art::Trace class). However, if the debugger
is only listening to DexPcMoved event (to manage JDWP Breakpoint
event), it will not be notified of this event.

We now properly call all the instrumentation callbacks in the
interpreter and move the logic specific to debugging into the class
DebugInstrumentationListener. This allows to properly group JDWP
location events together depending on the sequence of instrumentation
callbacks.

We add Thread::tls_32bit_sized_values::debug_method_entry_ flag to
remember we just entered a method. It replaces the local variable
notified_method_entry_event in the interpreter and simplifies the
code.

Bump oat version to force recompilation because the layout of the
Thread class is modified.

Bug: 19829329
Bug: 20205350

(cherry picked from commit 9d6bf69ad3012a9d843268fdd5325b6719b6d5f2)

Change-Id: I204af9112e37d2eebc86661fb7c961a41c74e598

am 3772670f: am 5260d145: Merge "Quick: Fix DCE to rename SSA regs for degenerate Phis." into mnc-dev

* commit '3772670f8120c602c87d039ea25cf075e8f4e343':
  Quick: Fix DCE to rename SSA regs for degenerate Phis.

am d2775e09: am f80552b7: Quick: Abolish kMirOpCheckPart2.

* commit 'd2775e092ce5cf62f51d2ec55478bf2b5e41d5e7':
  Quick: Abolish kMirOpCheckPart2.

am 5260d145: Merge "Quick: Fix DCE to rename SSA regs for degenerate Phis." into mnc-dev

* commit '5260d14566d13b1767ca63d2768d9b0e0a1bba33':
  Quick: Fix DCE to rename SSA regs for degenerate Phis.

am f80552b7: Quick: Abolish kMirOpCheckPart2.

* commit 'f80552b7e5f627a5dd07af017b7d65dec010ca48':
  Quick: Abolish kMirOpCheckPart2.

am e74d1bc1: am ea1c03e1: am 7b4de07f: Merge "Quick: Fix DCE to rename SSA regs for degenerate Phis."

* commit 'e74d1bc146b8dfee01eb7894a67151681c5b162e':
  Quick: Fix DCE to rename SSA regs for degenerate Phis.

am 183b0dff: am 9888b444: am b5784d47: Merge "Quick: Abolish kMirOpCheckPart2."

* commit '183b0dff49c3432eb68557706d72ddebd1fe8416':
  Quick: Abolish kMirOpCheckPart2.

am ea1c03e1: am 7b4de07f: Merge "Quick: Fix DCE to rename SSA regs for degenerate Phis."

* commit 'ea1c03e1ab324a43077fa22feb247f14c8699165':
  Quick: Fix DCE to rename SSA regs for degenerate Phis.

am 9888b444: am b5784d47: Merge "Quick: Abolish kMirOpCheckPart2."

* commit '9888b44423953f71b4d29843d423663a56f5901d':
  Quick: Abolish kMirOpCheckPart2.

Merge "Quick: Fix DCE to rename SSA regs for degenerate Phis." into mnc-dev

Quick: Fix DCE to rename SSA regs for degenerate Phis.

If we're inserting a new Phi and all its inputs are the same
SSA reg (for example, we allow a single-input Phi), some
subsequent insn may actually refer to that reg, so we need
to rename it to keep the graph in a consistent state.

(cherry picked from commit f60715c2c0a2398822a143ffb6b75507bffca703)

Change-Id: I46822b0db21c187601b79d2ff1a650494558772a

Fix typo in comments.

Change-Id: I6569c0fdc878f797b37509432b59b0d5c9bc5533

Quick: Abolish kMirOpCheckPart2.

The tricks played with kMirOpCheckPart2 are making the
native GC map generation unnecessarily complex. They have
caused problems in the past and now there is bad interaction
with the DCE. Rather than fixing it time and again, remove
the pseudo-insn.

(The whole purpose of those tricks seems to be to allow the
register tracking to be used for the throwing insn before
resetting the tracking for the next block. However, it's
questionable whether that's better than processing the
throwing insn with the subsequent instructions.)

Bug: 20736048

(cherry picked from commit e299f167c9559401548eab71678d4b779e46c2fb)

Change-Id: I8a60d26c5e6b6b608d68b8bb6b66d411f9a28f90

am 7b4de07f: Merge "Quick: Fix DCE to rename SSA regs for degenerate Phis."

* commit '7b4de07fffc6dbbbd986bb21b212fa0c4a2aec58':
  Quick: Fix DCE to rename SSA regs for degenerate Phis.

am b5784d47: Merge "Quick: Abolish kMirOpCheckPart2."

* commit 'b5784d4703d8f99921d69b6b97542f2fbcac3a51':
  Quick: Abolish kMirOpCheckPart2.

Quick: Abolish kMirOpCheckPart2.

The tricks played with kMirOpCheckPart2 are making the
native GC map generation unnecessarily complex. They have
caused problems in the past and now there is bad interaction
with the DCE. Rather than fixing it time and again, remove
the pseudo-insn.

(The whole purpose of those tricks seems to be to allow the
register tracking to be used for the throwing insn before
resetting the tracking for the next block. However, it's
questionable whether that's better than processing the
throwing insn with the subsequent instructions.)

Bug: 20736048

(cherry picked from commit e299f167c9559401548eab71678d4b779e46c2fb)

Change-Id: Ifae6c5bd961a2619b50fd3440261762cb9151460

Merge "Quick: Fix DCE to rename SSA regs for degenerate Phis."

Merge "Quick: Abolish kMirOpCheckPart2."

am d367191c: am 4a75e3c2: am ba9ef537: Merge "Support multiple instrumentation clients"

* commit 'd367191c05d2f61ea836342f62dadc5745de27fc':
  Support multiple instrumentation clients

am ccb11541: am a8b4dfcc: am b882da74: Merge "Remove unused InvokeWithShadowFrame"

* commit 'ccb1154107a8f50c816f59bae765ac63d74297d4':
  Remove unused InvokeWithShadowFrame

am 0c3c9c17: am f3a8c70a: am 6d2d4806: Merge "Fix broken checks in IsValidPartOfMemberNameUtf8Slow."

* commit '0c3c9c1707ccc133f7904484c34893961db78acd':
  Fix broken checks in IsValidPartOfMemberNameUtf8Slow.

am 8b2f6e19: am c42bd509: am eb7c5b19: Merge "JDWP: properly combine location events"

* commit '8b2f6e1989ae4d6c9e13d3429c3e8145ec871dd5':
  JDWP: properly combine location events

am 61ad25f7: am 205db690: am f07f71f9: Merge "Use a growable array instead of an environment during SSA."

* commit '61ad25f7b97345d8c057e6c28c36e2b5e9b86870':
  Use a growable array instead of an environment during SSA.

am 90bc2a27: am 905a88d3: am 46a4b268: Merge "Relax the only one back-edge restriction."

* commit '90bc2a277dbfd042627014f187623d13f4643178':
  Relax the only one back-edge restriction.

am 9b66b2ac: am 5199f914: am bc3b93ea: Merge "No need to relocate the core image for the art script."

* commit '9b66b2acb74805eecb807a36c4eb1e1772072532':
  No need to relocate the core image for the art script.

am 71359ff3: am 9a950dec: am a6591ef9: Merge "ART: Fix mips64 jni bugs."

* commit '71359ff31b41fabaa492abcae4bf55b89341c3d0':
  ART: Fix mips64 jni bugs.

am 8e8aee7a: am 75b27b57: am 225f679f: Merge "In quick compiler, force string init invoke to be fast path."

* commit '8e8aee7a64e9ad4fef3c0ca38dd2695daaaf089e':
  In quick compiler, force string init invoke to be fast path.

am bbbc7702: am 86b09d0f: am 8771be9a: Merge "Fix mips64 bugs."

* commit 'bbbc7702dd86a46a86a7c9ff8bdcd2d2f16e05c2':
  Fix mips64 bugs.

am f689ca39: am 3ba8671d: Fix broken checks in IsValidPartOfMemberNameUtf8Slow.

* commit 'f689ca3917d995880d8eab72f0d8760bdea9ac6c':
  Fix broken checks in IsValidPartOfMemberNameUtf8Slow.

am 97ee6ec7: am 5c8fe302: In quick compiler, force string init invoke to be fast path.

* commit '97ee6ec76d0824669fb7a37facc1b21a9a6b53aa':
  In quick compiler, force string init invoke to be fast path.

am 4a75e3c2: am ba9ef537: Merge "Support multiple instrumentation clients"

* commit '4a75e3c22ccd4680ff0416c1609dafad1130ede8':
  Support multiple instrumentation clients

am ba9ef537: Merge "Support multiple instrumentation clients"

* commit 'ba9ef5377b21a627ec502da3e475b34e45d0c435':
  Support multiple instrumentation clients

am a8b4dfcc: am b882da74: Merge "Remove unused InvokeWithShadowFrame"

* commit 'a8b4dfcc22b0fdbdeeb87ce91368e88d46ab43e6':
  Remove unused InvokeWithShadowFrame

am f3a8c70a: am 6d2d4806: Merge "Fix broken checks in IsValidPartOfMemberNameUtf8Slow."

* commit 'f3a8c70aeb1ca2f4c1eb7a65a8f6df97698dc03a':
  Fix broken checks in IsValidPartOfMemberNameUtf8Slow.

am c42bd509: am eb7c5b19: Merge "JDWP: properly combine location events"

* commit 'c42bd509e5a0e5a7634b7c8ebb18196523af08a9':
  JDWP: properly combine location events

am 205db690: am f07f71f9: Merge "Use a growable array instead of an environment during SSA."

* commit '205db6908eeca3f38c2da79cd25878d74537bf46':
  Use a growable array instead of an environment during SSA.

am 905a88d3: am 46a4b268: Merge "Relax the only one back-edge restriction."

* commit '905a88d309be4c41af4addd1ec2a821ecdcc9725':
  Relax the only one back-edge restriction.

am 5199f914: am bc3b93ea: Merge "No need to relocate the core image for the art script."

* commit '5199f91486c1dac9037456c3f940dfd9faf673aa':
  No need to relocate the core image for the art script.

am 9a950dec: am a6591ef9: Merge "ART: Fix mips64 jni bugs."

* commit '9a950decdf11303d47a450ff0d71ea64ae3d75ea':
  ART: Fix mips64 jni bugs.

am 75b27b57: am 225f679f: Merge "In quick compiler, force string init invoke to be fast path."

* commit '75b27b57ad23da4c85fc13c644a763d541b8f0dd':
  In quick compiler, force string init invoke to be fast path.

am 86b09d0f: am 8771be9a: Merge "Fix mips64 bugs."

* commit '86b09d0fecd1d01819a8680ae7be087be90489aa':
  Fix mips64 bugs.

Merge "Support multiple instrumentation clients"

Support multiple instrumentation clients

Changes Instrumentation::ConfigureStubs to support multiple clients
that need different levels of instrumenation. A client is identified
by a string key used to save the desired instrumentation level.

Also adds regression gtest instrumentation_test and some cleanup.

Bug: 19829329
Change-Id: I1fc24a86fcb7cb46d4be806895376c25cc0a0b3c

am b882da74: Merge "Remove unused InvokeWithShadowFrame"

* commit 'b882da74db950246b5731aecad9737b63bdefef7':
  Remove unused InvokeWithShadowFrame

Merge "Remove unused InvokeWithShadowFrame"

Remove unused InvokeWithShadowFrame

This is an artifact of the Portable compiler, probably left after its
removal https://android-review.googlesource.com/119049.

Change-Id: I7f6caa008e37e7a93792840c79713e90d0c7e1e1

am 6d2d4806: Merge "Fix broken checks in IsValidPartOfMemberNameUtf8Slow."

* commit '6d2d48066f503c16aaf15661a22c75fde7ab76c2':
  Fix broken checks in IsValidPartOfMemberNameUtf8Slow.

Quick: Abolish kMirOpCheckPart2.

The tricks played with kMirOpCheckPart2 are making the
native GC map generation unnecessarily complex. They have
caused problems in the past and now there is bad interaction
with the DCE. Rather than fixing it time and again, remove
the pseudo-insn.

(The whole purpose of those tricks seems to be to allow the
register tracking to be used for the throwing insn before
resetting the tracking for the next block. However, it's
questionable whether that's better than processing the
throwing insn with the subsequent instructions.)

Bug: 20736048
Change-Id: I4767e4609914d3b6990da4416e5093e4ca209780

am 3ba8671d: Fix broken checks in IsValidPartOfMemberNameUtf8Slow.

* commit '3ba8671d60061359fd833f60f7a9dca14878cc0b':
  Fix broken checks in IsValidPartOfMemberNameUtf8Slow.

Merge "Fix broken checks in IsValidPartOfMemberNameUtf8Slow."

Quick: Fix DCE to rename SSA regs for degenerate Phis.

If we're inserting a new Phi and all its inputs are the same
SSA reg (for example, we allow a single-input Phi), some
subsequent insn may actually refer to that reg, so we need
to rename it to keep the graph in a consistent state.

Change-Id: Ic6a1907c3138f4a7d3b13f9e58e9107ca2d92f17

am eb7c5b19: Merge "JDWP: properly combine location events"

* commit 'eb7c5b192257c92fc38a163c57e0a8bb771e476e':
  JDWP: properly combine location events

Merge "JDWP: properly combine location events"

Fix broken checks in IsValidPartOfMemberNameUtf8Slow.

GetUtf16FromUtf8 returns a surrogate pair only if it encounters
a 4-byte UTF sequence. Three byte UTF sequences will only return the
first or second half of a pair so we need to check for that
explicitly.

bug: 20844537

(cherry picked from commit 3ba8671d60061359fd833f60f7a9dca14878cc0b)

Change-Id: I2e2a4f9f736cd11050a2b634b3bb27b75a0ee0ba

am f07f71f9: Merge "Use a growable array instead of an environment during SSA."

* commit 'f07f71f9af8f56e738ef0451c60734ec6022d08e':
  Use a growable array instead of an environment during SSA.

Merge "Use a growable array instead of an environment during SSA."

Use a growable array instead of an environment during SSA.

Using an environment was convenient because it contains
a growable array. But there's no need for the environment
abstraction when being used as a temporary holder for values
of locals.

Change-Id: Idf2883fe4b8f97a31ee70b3627c1bdd23ebfff0e

Fix broken checks in IsValidPartOfMemberNameUtf8Slow.

GetUtf16FromUtf8 returns a surrogate pair only if it encounters
a 4-byte UTF sequence. Three byte UTF sequences will only return the
first or second half of a pair so we need to check for that
explicitly.

bug: 20844537

Change-Id: Icb660fae77ac8a852fc768e6c1cd5766117e68e4

am 46a4b268: Merge "Relax the only one back-edge restriction."

* commit '46a4b26855ae9fcd14c6e4435f37f09ceb134f61':
  Relax the only one back-edge restriction.

am bc3b93ea: Merge "No need to relocate the core image for the art script."

* commit 'bc3b93eadd155342b6124d2d5ef3806ecec5dfd6':
  No need to relocate the core image for the art script.

Merge "Relax the only one back-edge restriction."

Relax the only one back-edge restriction.

The rule is in the way for better register allocation, as
it creates an artificial join point between multiple paths.

Change-Id: Ia4392890f95bcea56d143138f28ddce6c572ad58

Merge "No need to relocate the core image for the art script."

am 5c8fe302: In quick compiler, force string init invoke to be fast path.

* commit '5c8fe3028655b2f1fcab77080272f071cc0d8bc4':
  In quick compiler, force string init invoke to be fast path.

am a6591ef9: Merge "ART: Fix mips64 jni bugs."

* commit 'a6591ef95afa6bc9d89c0c6044635ec7a13d4c55':
  ART: Fix mips64 jni bugs.

Merge "ART: Fix mips64 jni bugs."

ART: Fix mips64 jni bugs.

For mips64, 32-bits loads can be zero extended or sign extended to
64-bits. The extension type must match the data type to be loaded.

Also re-enable mips64 generic jni testing.

Change-Id: I9cabaf80b4fde63d9868fccd74593b36d1c324e8

am 225f679f: Merge "In quick compiler, force string init invoke to be fast path."

* commit '225f679f1d276e16e95960fcbe9743f95012adfa':
  In quick compiler, force string init invoke to be fast path.

Merge "In quick compiler, force string init invoke to be fast path."