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.

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

Merge "Quick: Abolish kMirOpCheckPart2."

am ba9ef537: Merge "Support multiple instrumentation clients"

* commit 'ba9ef5377b21a627ec502da3e475b34e45d0c435':
  Support multiple instrumentation clients

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

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

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

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

Was running into an issue where a secondary dex file was falling back on
the quick compiler, and IsFastInvoke returned 0 because the
referrer_class was nullptr, causing the string init to be slow path. The
string init technically does not have to be fast path, but there's no
reason to duplicate the logic to call off a thread pointer on the slow
path.

Bug: 20870917

(cherry picked from commit I47e1524d939eb7e9b1da8186092fafc6e925009e)

Change-Id: Ieacd965943759de2b075c30e03cf0a71863f499e

am 8771be9a: Merge "Fix mips64 bugs."

* commit '8771be9a5826ebe12cea0c2feb0fa085df5d60a2':
  Fix mips64 bugs.

Merge "Fix mips64 bugs."

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

Change-Id: Idd540175e197602e318eeb3764f552372aadce1f

am 23d358b6: Merge "Quick: Inline synthetic accessors generated by jack."

* commit '23d358b621dbe11d6da918ab310ed3f1a5fc0558':
  Quick: Inline synthetic accessors generated by jack.

Merge "Quick: Inline synthetic accessors generated by jack."

Quick: Inline synthetic accessors generated by jack.

While javac names synthetic accessors "access$nnn", jack
names them "-getN", "-putN" and "-wrapN". For simplicity,
treat all methods starting with "-" as synthetic accessors.

Bug: 20873367
Change-Id: I67990a6c2ef1d25dafe460ef70143a113bb5b7da

am dceab011: Merge "Bump jdwp timeout to 12 minutes."

* commit 'dceab011395333520959cf743d680bcf5dc78797':
  Bump jdwp timeout to 12 minutes.

Merge "Bump jdwp timeout to 12 minutes."

Bump jdwp timeout to 12 minutes.

A run on volantis now is around 10 minutes.

Change-Id: I6e1f436d71da6eec55e984f7af99ba34438e49fb

am 9cce14f5: Merge "Re-enable JDWP tests on volantis."

* commit '9cce14f56a59bbce52254ffc50ea6817fd94d8dd':
  Re-enable JDWP tests on volantis.

am 644fa22a: Merge "Fix instrumentation exit stub for arm64"

* commit '644fa22a432ad0f717878331e745d6c25cc57d5d':
  Fix instrumentation exit stub for arm64

Merge "Re-enable JDWP tests on volantis."

Re-enable JDWP tests on volantis.

Change-Id: Ibf1d16c9830155526cee38e242a4de9a3d5ed943

Merge "Fix instrumentation exit stub for arm64"

Fix instrumentation exit stub for arm64

Fixes bad offset to restore register x21.

Bug: 20798393

(cherry picked from commit a538effb3f848bfec232c55f167b9f35b416948f)

Change-Id: I030d848ab0f46043e86f53abae204463f0aaccbe

am 3acc27f4: Merge "Fix Mips64 utils.h typo"

* commit '3acc27f454e33f4650867a3ba0ecb8d3011e1048':
  Fix Mips64 utils.h typo

Merge "Fix Mips64 utils.h typo"

Fix Mips64 utils.h typo

Include of art's utils.h was mistyped as util.h, and instead
brought in external/icu/icu4c/source/common/util.h,
which caused art build failures from -Wdeprecated warnings
when Mips64 libart is compiled with clang.

Change-Id: I9370c4dc041eaecc5d5a1490ec23034ece4322a9

am 35b6625c: Merge "Temporarily suppress 098-ddmc for the read barrier config."

* commit '35b6625c21e9304dd79700f4ff1c26a7e30d2b2d':
  Temporarily suppress 098-ddmc for the read barrier config.

Merge "Temporarily suppress 098-ddmc for the read barrier config."

Temporarily suppress 098-ddmc for the read barrier config.

Bug: 20720510
Change-Id: Ie8c159767654837c02f0484f4dda569bb0118a03

am 2b689e37: Merge "Revert "Revert "Revert "Revert "[optimizing] Improve x86 shifts"""""

* commit '2b689e370be9f32a88b419edf2d78eb6befade9c':
  Revert "Revert "Revert "Revert "[optimizing] Improve x86 shifts""""

Merge "Revert "Revert "Revert "Revert "[optimizing] Improve x86 shifts"""""

am 898fa9b9: Merge "Find better split positions in the register allocator."

* commit '898fa9b96a579715d124671102886242bd62f393':
  Find better split positions in the register allocator.

Merge "Find better split positions in the register allocator."

Find better split positions in the register allocator.

In a standard if/else control flow graph, this avoids
doing a move in one branch if the other branch decided
to move an interval.

This also needs a new register hint kind, which is what
was the location of the interval at the predecessor block.

Change-Id: I18b78264587b4d693540fbb5e014d12df2add3e2

am 008b17ae: Merge "ART: Fix double-free in oatdump"

* commit '008b17ae313d033537a3792faf937134315f03bc':
  ART: Fix double-free in oatdump

Merge "ART: Fix double-free in oatdump"

ART: Fix double-free in oatdump

The ImageDumper doesn't own the OatDumperOptions.

Bug: 20818721
Change-Id: Iea11ada8fb6418b226d39b33c252a834c42e655e

am c2baa5e3: Merge "ART: Emit debug info when app is debuggable"

* commit 'c2baa5e36f615947affc899b063ceadbd2879d17':
  ART: Emit debug info when app is debuggable

am dd13f157: Merge "Fix mismatched new[]/delete."

* commit 'dd13f1579006f0562a1b3548dc0a6d80cf62a002':
  Fix mismatched new[]/delete.

am d864f501: Merge "Delete invalid DCHECK"

* commit 'd864f5010de581b15cea1e8178871e658024a6d9':
  Delete invalid DCHECK

Merge "ART: Emit debug info when app is debuggable"

ART: Emit debug info when app is debuggable

Emit debug symbols and CFI when dex2oat is compiling with the
--debuggable flag.

Bug: 17011129

(cherry picked from commit 8484d23186c229e2927441f85ae819abb4996072)

Change-Id: I0afab1e46abd819eded2b823580e766619497393

Merge "Fix mismatched new[]/delete."

Merge "Delete invalid DCHECK"

Fix mismatched new[]/delete.

Another two cases where a new[] is used but only a delete occurs.

Bug: 18202869
Change-Id: If68264807150f3a9783e44ef8823cc366bff8df2

Delete invalid DCHECK

We can have large objects mapped at address 0xebad____.

Change-Id: I9e27d4948e7e853b4af6eb1b36dbfec6d4d7a99a

am a93b104b: Merge "Improve JNI registration failure logging"

* commit 'a93b104b47da67eaa50f8a6b7280cc6c438ef37b':
  Improve JNI registration failure logging

Merge "Improve JNI registration failure logging"

Improve JNI registration failure logging

Print INTERNAL_FATAL instead of ERROR, print the
"Failed to register native method" message last.

Bug: 13885477
Change-Id: I9f1a7ea45bd926f3984b8008d84c75e676a1ed10

am a6f2a7e0: Merge "Fix NanoSleep and add test"

* commit 'a6f2a7e01433926339866afc2b413770774c6d9f':
  Fix NanoSleep and add test

Merge "Fix NanoSleep and add test"

Fix NanoSleep and add test

Fixed a bug where tv_nsec was't between 0 and 999,999,999.

Bug: 14450052
Change-Id: I30b29a716bfa63c6b57d589dd1102d2ca934c061

Revert "Revert "Revert "Revert "[optimizing] Improve x86 shifts""""

This reverts commit 2a7a1d7808f003bea908023ebd11eb442d2fca39.

Fix the problem that a long long >> 63 got the wrong answer.  The
problem was that a shr was used instead of a sar.

Change-Id: I0327f79c718016ddec9272a605fc50ec15ec4566

am 8a30bf23: Merge "Add some more DISALLOW_COPY_AND_ASSIGN"

* commit '8a30bf23bcb1ad8d4ed9060ddbb27edbfd57a897':
  Add some more DISALLOW_COPY_AND_ASSIGN

Merge "Add some more DISALLOW_COPY_AND_ASSIGN"

Add some more DISALLOW_COPY_AND_ASSIGN

May help prevent bugs maybe.

Change-Id: Ie73d469dfcd078492ecb3aa28682b42707221202

am 25dcb259: Merge "Let the map function compute the relative pc."

* commit '25dcb2594a21a565ef82faa66a79bcf595c3a780':
  Let the map function compute the relative pc.

Merge "Let the map function compute the relative pc."

Let the map function compute the relative pc.

The new packed relocations linker option creates an elf with a non-zero
load base. It's no longer an easy task to compute the relative pc yourself
so call the helper function to do it for us.

Bug: 20687795

(cherry picked from commit 332ef327c2b1a5e1774ba0d1da23e07303002af9)

Change-Id: I3d61c767f21a3f240a14f9e8bc6bee53b6cdb60d

am 43e1bc66: Merge "Request full/partial GC type from VMRuntime.requestConcurrentGC()."

* commit '43e1bc667e66386443aee6477d767d9b8ef5aa79':
  Request full/partial GC type from VMRuntime.requestConcurrentGC().

Merge "Request full/partial GC type from VMRuntime.requestConcurrentGC()."

am 376d941d: Merge "ART: Fix max build"

* commit '376d941d5d8ad23ca2880aecffbe8381be963f35':
  ART: Fix max build

Merge "ART: Fix max build"

ART: Fix max build

Change-Id: Id1b9c0ee05b61195d1c1f09a8938df3110c2f2b3

am b0d9f0dd: Merge "Re-enable clang for Mips/Mips64 targets."

* commit 'b0d9f0ddd6189564f359f989d9ff606d6c691ed0':
  Re-enable clang for Mips/Mips64 targets.

Merge "Re-enable clang for Mips/Mips64 targets."

Request full/partial GC type from VMRuntime.requestConcurrentGC().

Bug: 20554807
Change-Id: I1cd855507c3f4e191035dbc2c429e1b8b344d81b

Fix mips64 bugs.

There are 2 bugs here. First is Method* is computed twice. The
second is the wrong runtime callee save frame offset is being
used in computing Method*.

Change-Id: Ica23585a08be480a54d0cdd17eeca8027061b3f3

am 1db3554c: Merge "Print memory maps on GC crash."

* commit '1db3554cf9751d4328d5514b6bee15b6a92b82c4':
  Print memory maps on GC crash.

am 772b617c: Merge "Add binary search table to .eh_frame_hdr"

* commit '772b617ce920ea1924b0ad2fd429c99274f0ccd5':
  Add binary search table to .eh_frame_hdr

Re-enable clang for Mips/Mips64 targets.

The build-time errors described in
  https://android-review.googlesource.com/#/c/120512
are no longer happening.

BUG: 18807290
BUG: 18789639

Change-Id: I195bbb7d102fecd6a6e6a3357b0863dd6fddb01f

Merge "Print memory maps on GC crash."

Merge "Add binary search table to .eh_frame_hdr"

am 485971cf: Merge "Do not write CFI and symbols for deduplicate methods."

* commit '485971cf4105886773a3fd10a0021738d4cd4de7':
  Do not write CFI and symbols for deduplicate methods.

am f274519c: Merge "Change invalid root from LOG(ERROR) to LOG(INTERNAL_FATAL)"

* commit 'f274519ce9fc3a9df477ffb320bcdcddab505b58':
  Change invalid root from LOG(ERROR) to LOG(INTERNAL_FATAL)

Add binary search table to .eh_frame_hdr

The addresses in the search table must be relative to the start
of .eh_frame_hdr which results in yet another pointer type to patch.
I have moved the Patch method back to elf_writer_quick where
it can be specialized to one of the three pointer types.

The .eh_frame_hdr takes around 17% of .eh_frame now.

Change-Id: I4770d1355ded6cdff9489c33380f6b06c4e3f9fe

Merge "Do not write CFI and symbols for deduplicate methods."