am dc31313c: Merge "ART: Clean up arm64 kNumberOfXRegisters usage."

* commit 'dc31313c4577bb4c00d90774159c84e738ed348d':
  ART: Clean up arm64 kNumberOfXRegisters usage.

Merge "ART: Clean up arm64 kNumberOfXRegisters usage."

am 03b5a398: Merge "Fix for potential moving GC bugs around proxy class."

* commit '03b5a398a96ea29c39ddfe3d810245b868d2871f':
  Fix for potential moving GC bugs around proxy class.

am 1b742fcc: Merge "Log the process memory maps on host SIGSEGV."

* commit '1b742fccf9ca68b783fdbd559a3fcbdafab48852':
  Log the process memory maps on host SIGSEGV.

Merge "Fix for potential moving GC bugs around proxy class."

Merge "Log the process memory maps on host SIGSEGV."

Fix for potential moving GC bugs around proxy class.

- Handlerize proxy_class which is live across multiple allocation
  points in ClassLinker::CreateProxyClass().

- In ClassLinker::CreateProxyClass(), insert a proxy class into the
  class table before creating ArtFields for it (and update it later in
  LinkClass()) because the field roots (ArtField::declaring_class_)
  won't be updated by GC unless the class is in the class table. If GC
  happens before they are updated by FixupTemporaryDeclaringClass()
  from LinkClass(), FixupTemporaryDeclaringClass() may not update the
  field roots correctly because the old class may already be moved but
  the fields roots may not. Reduce a window of time where the fields
  roots could be stale.

- In ClassLinker::LinkClass(), directly wrap a new class in a handle
  to avoid a window of time where new_class may be potentially stale.

- Print more diagnostic info about the holder of the field upon a mark
  sweep invalid ref crash.

- Add an additional sanity check in Field::GetArtField().

(cherry pick commit 08d1b5f2296c0f51507b8b443f4e39dfc161572c)

Bug: 20557050

Change-Id: I9ad32d304922da96b7e1fad262d97de21cbac776

ART: Clean up arm64 kNumberOfXRegisters usage.

Avoid undefined behavior for arm64 stemming from 1u << 32 in
loops with upper bound kNumberOfXRegisters.

Create iterators for enumerating bits in an integer either
from high to low or from low to high and use them for
<arch>Context::FillCalleeSaves() on all architectures.

Refactor runtime/utils.{h,cc} by moving all bit-fiddling
functions to runtime/base/bit_utils.{h,cc} (together with
the new bit iterators) and all time-related functions to
runtime/base/time_utils.{h,cc}. Improve test coverage and
fix some corner cases for the bit-fiddling functions.

Bug: 13925192
Change-Id: I704884dab15b41ecf7a1c47d397ab1c3fc7ee0f7

am c06841da: Merge "Fix a few literals and assertions."

* commit 'c06841dac1443e99e92596f15caa82c99bfee010':
  Fix a few literals and assertions.

am 41f27292: Merge "Move the catalog after the stack maps."

* commit '41f27292e10792165bfdf67b9a35ccaea1a6872e':
  Move the catalog after the stack maps.

Merge "Fix a few literals and assertions."

Merge "Move the catalog after the stack maps."

am 9470f045: Merge "Make inlining deterministic."

* commit '9470f045c89cdb3de5175266bda523b8a1c2d96d':
  Make inlining deterministic.

Move the catalog after the stack maps.

Fetching a stack map should be cheap, and therefore we should
not compute the size of the catalog.

Change-Id: Iff6accb7832bdfa6d209a0928d6088b8b57c1aa4

Merge "Make inlining deterministic."

Fix a few literals and assertions.

Change-Id: I0a1e9db607ec7325e17568b034ba90e68d2298f9

Make inlining deterministic.

Only the case where two methods are not in the same dex
file could lead to undeterministic behavior.

bug:20037935

Change-Id: I1a7642a979302b17e76d196894437c1aacbbbe9d

am ce63cf65: Merge "ART: Update graph\'s exit block field if removed"

* commit 'ce63cf656bb5b25051d580a58086f4bf15fb878b':
  ART: Update graph's exit block field if removed

Merge "ART: Update graph's exit block field if removed"

am 092a5656: Merge "JDWP: fix breakpoint on catch statement"

* commit '092a5656937a319449346e8c356f3f4c2870d81c':
  JDWP: fix breakpoint on catch statement

Merge "JDWP: fix breakpoint on catch statement"

ART: Update graph's exit block field if removed

Running DCE on an infinite loop will delete the exit block but the
corresponding field is currently not cleared in the parent graph.
This does not cause any problems at the moment as that information is
only used in codegens to DCHECK that a block is not the exit block.
However, it will be necessary to update the inliner once we start to
inline methods with loops.

With this patch, DCE will update the HGraph::exit_block_ field. DCHECK
was also added to HGraph::InlineInto to make sure that the inlined
graph does have an exit block.

Change-Id: Ia8ddca375bbc6830cd919af6059a52cc9b73a023

JDWP: fix breakpoint on catch statement

Setting a breakpoint on a catch statement in the source actually
installs the breakpoint on a DEX move-exception instruction. At this
point, an exception is pending in the current thread.

The issue is no exception must be pending in the current thread to
report the breakpoint event. This is required to be able to call JNI
functions to create JDWP ids.

This CL fixes it by clearing the pending exception before reporting
event and restore it after reporting the event.

Bug: 21382373
Change-Id: Id3c60cca398135a3e0859a1ccb645b9c99d5ca76

am 8a8d8252: Merge "Do not use dex_compilation_unit after inlining."

* commit '8a8d8252f52bec14a8fe2b6f88e1f184c63cf351':
  Do not use dex_compilation_unit after inlining.

Merge "Do not use dex_compilation_unit after inlining."

Log the process memory maps on host SIGSEGV.

Bug: 19894268
Change-Id: Ib35f858de2519e24422a2ed19c68cfbdfd4173b3

am ca27be7a: Merge "ARM64: Move xSELF from x18 to x19."

* commit 'ca27be7a95e1b238c76a4a5fccaad2a3416766c2':
  ARM64: Move xSELF from x18 to x19.

Merge "ARM64: Move xSELF from x18 to x19."

am e7654e5a: Merge "Revert "Switch to using ELF-64 for 64-bit architectures.""

* commit 'e7654e5ab8668d14aab0d4bf1b3631ef150b36dd':
  Revert "Switch to using ELF-64 for 64-bit architectures."

Merge "Revert "Switch to using ELF-64 for 64-bit architectures.""

Revert "Switch to using ELF-64 for 64-bit architectures."

This reverts commit a3a49fe863e0bcf423ed0b4082136d1439fb2e79.

Change-Id: I0afabec15f5d9110de0d4105cbab7d040f2b86e2

am 8c83ef7a: Merge "Return an invalid StackMap when one cannot be found."

* commit '8c83ef7abff3b96d9551da6b1c0c68497617f3f7':
  Return an invalid StackMap when one cannot be found.

am bacdffbe: Merge "Revert "Revert "Introduce a NearLabel in thumb2."""

* commit 'bacdffbeed9ee3671854a1ec3d3aa169a4fb765e':
  Revert "Revert "Introduce a NearLabel in thumb2.""

Merge "Return an invalid StackMap when one cannot be found."

Merge "Revert "Revert "Introduce a NearLabel in thumb2."""

Do not use dex_compilation_unit after inlining.

It's incompatible with inlining, as inlined invokes/load class/new
can be from another dex file.

Change-Id: I8897b6a012942bc8e136f2bea70252d3fb3a7fa5

Revert "Revert "Introduce a NearLabel in thumb2.""

This reverts commit 1f277e3cef6c33cd35e91123978491d83338d2ad.

- Fix CompareAndBranch to not use cbz/cbnz with high registers.
- Add a test for CompareAndBranch with the *inc file, as the
  other assembler test infrastructure does not handle labels.

Change-Id: If552bf1112b96caa3b9bb6c73c4b40bb90a33db7

Return an invalid StackMap when one cannot be found.

This avoids aborting when handling a crash.

Change-Id: Ie5b5d48061fa9258b349b0284f7b00c5855d9fbd

ARM64: Move xSELF from x18 to x19.

This patch moves xSELF to callee saved x19 and removes support for
ETR (external thread register), previously used across native calls.

Change-Id: Icee07fbb9292425947f7de33d10a0ddf98c7899b
Signed-off-by: Serban Constantinescu <serban.constantinescu@linaro.org>

am aa49c23d: Merge "ART: Sometimes even empty methods take forever to verify"

* commit 'aa49c23d47e5fdfcf51380550ee864e9d30d082b':
  ART: Sometimes even empty methods take forever to verify

Merge "ART: Sometimes even empty methods take forever to verify"

ART: Sometimes even empty methods take forever to verify

In cases of very high load and/or bad scheduling, the verifier may
take longer than the threshold duration to verify an empty method.
The LargeMethod detection needs to accept that the code_item may
be null.

Bug: 21364300

(cherry picked from commit 8110447356f559688d6290563e912af7f8d4709b)

Change-Id: Iceff3e4688cc1a5fe7a836f7a9bf6c49a392b618

am 559b178e: Merge "Update failures"

* commit '559b178e34c5d92e7932f92e5d8a981ac334606f':
  Update failures

Merge "Update failures"

am 12beecf8: Merge "ART: Blacklist CFI test for Heap Poisoning"

* commit '12beecf84130780915c1bdc97d80f9715ce941d2':
  ART: Blacklist CFI test for Heap Poisoning

Merge "ART: Blacklist CFI test for Heap Poisoning"

ART: Blacklist CFI test for Heap Poisoning

The test expects Java frames on the stack.

Change-Id: I19e7429f06fac107c2cf9f85325e8f071e5799fb

Update failures

Some failures have been removed too eagerly, this CL reverts that.

Change-Id: I897b1b7f936d2c001c44db604905c22242797e46

am dc80984a: Merge "Write 64-bit address in DWARF if we are on 64-bit architecture."

* commit 'dc80984a9e24d6e46fab9461f01d7a27e656cf90':
  Write 64-bit address in DWARF if we are on 64-bit architecture.

Merge "Write 64-bit address in DWARF if we are on 64-bit architecture."

am 7288b382: Merge "Update libcore failures"

* commit '7288b382b03936e707f0cb2da955be4e4301a699':
  Update libcore failures

Merge "Update libcore failures"

Update libcore failures

Updated libcore failures.

Change-Id: I570846220c5b4dea43fd39a4a777c37eb3595a08

am 1d637745: Merge "Fix the size of a StackHandleScope in art::CompilerDriver."

* commit '1d637745bf1a47db6b899f0c004746ada2fd3f58':
  Fix the size of a StackHandleScope in art::CompilerDriver.

Merge "Fix the size of a StackHandleScope in art::CompilerDriver."

am d6a1eaef: Merge "Revert "Introduce a NearLabel in thumb2.""

* commit 'd6a1eaef02d0c73a8c24ff93ff68fc4f0d46925f':
  Revert "Introduce a NearLabel in thumb2."

Merge "Revert "Introduce a NearLabel in thumb2.""

Revert "Introduce a NearLabel in thumb2."

Fails some benchmarks and libcore tests.

This reverts commit db0bbab279534974dca507946c66cff2d05dc9f9.

Change-Id: I5d1afef5ede87e65d61f49529027c5c2f35b17fb

am 9758a8e4: Merge "Also encode the InvokeType in an InlineInfo."

* commit '9758a8e4f7a8823161604aa876517e9ff5b8bb12':
  Also encode the InvokeType in an InlineInfo.

am 19b71cd0: Merge "ART: Rename graph dump file name to \'graph.cfg\'"

* commit '19b71cd0af3b45a105073d80ce33ede69b14adfd':
  ART: Rename graph dump file name to 'graph.cfg'

Merge "Also encode the InvokeType in an InlineInfo."

Also encode the InvokeType in an InlineInfo.

This will be needed to recover the call stack.

Change-Id: I2fe10785eb1167939c8cce1862b2d7f4066e16ec

Merge "ART: Rename graph dump file name to 'graph.cfg'"

ART: Rename graph dump file name to 'graph.cfg'

Change-Id: I91f6a6bf5efac5b3bba92b9a89e22e1b7be6e02f

am 533d89ce: Merge "ART: Implement next-line assertions in Checker"

* commit '533d89ceea0a8b953c688a2e9c259a5361581ecf':
  ART: Implement next-line assertions in Checker

am 1fd8dae9: Merge "Introduce a NearLabel in thumb2."

* commit '1fd8dae90da7bd62c4c04cd10d4f600e8d9e4416':
  Introduce a NearLabel in thumb2.

Merge "ART: Implement next-line assertions in Checker"

Merge "Introduce a NearLabel in thumb2."

ART: Implement next-line assertions in Checker

Some tests require verifying an exact sequence of lines in the graph
dump. This was already possible by inserting 'CHECK-NOT: {{.*}}'
between the individual lines, but hardly a convenient way of doing so.
This patch introduces a new 'CHECK-NEXT' kind of assertions that
replaces the old method and will become useful for testing assembly.

Change-Id: I1bb951707bda44320166dc7ef828866a6957a113

am 01cb410f: Merge "ART: Refactor and simplify matching in Checker"

* commit '01cb410f4ad23135671d821ba36c269f8c82affa':
  ART: Refactor and simplify matching in Checker

Merge "ART: Refactor and simplify matching in Checker"

ART: Refactor and simplify matching in Checker

Change-Id: Ib8a2b51488f66a7239e799f5fa5910b4ac2dfe08

am bacadb05: Merge "Intercept JNI invocation of String.<init> methods."

* commit 'bacadb05b8b74dc91f05ee3899cbd9b2ea5dab7e':
  Intercept JNI invocation of String.<init> methods.

Merge "Intercept JNI invocation of String.<init> methods."

Intercept JNI invocation of String.<init> methods.

libmono uses JNI AllocObject and CallNonvirtualVoidMethod to create and
initialize a string instead of using the recommended NewObject. This
change adds an intercept to change the String.<init> call to a
StringFactory call instead. Then, it uses the object id of the original
string object referrer and maps it to the result of the StringFactory.

Bug: 21288130

(cherry picked from commit 15e9ad1d028d7f12cb598b075453173532a00d91)

Change-Id: I3421c43722c07397da4a398c2ca9110e1d40bcfa

am 768b19b6: Merge "ART: Use v2 version of NativeBridge struct"

* commit '768b19b6bbddaa2262f6532e7fcb96fecbfee981':
  ART: Use v2 version of NativeBridge struct

Merge "ART: Use v2 version of NativeBridge struct"

am 03231f42: Merge "Fix a cts crash around proxy class fields."

* commit '03231f42c6e887237417d2d2e8d661dcedd7f5b2':
  Fix a cts crash around proxy class fields.

ART: Use v2 version of NativeBridge struct

Update the struct to the new version, but without implementation.

Bug: 20217701
Change-Id: Ic23a60b949f119c7d8b0e7cb27a61e6c16532a23

Merge "Fix a cts crash around proxy class fields."

Fix a cts crash around proxy class fields.

org.apache.harmony.tests.java.io.SerializationStressTest4#test_writeObject_Proxy

As the static fields of proxy classes share the dex file indices, they
shouldn't be resolved in the dex cache or else Field::GetArtField()
may return a wrong art field that belong to a different proxy class.

(cherry pick commit a56ce5e267c9744ed99e40ae5cd9b527971e1d63)

Bug: 20557050

Change-Id: If672c0e67bc49e672e34d75ffbe29c65f5a423b9

am 847d902c: Merge "ART: Fix gtest dependency"

* commit '847d902cc274d0c317fd94b5b5a058cba94828c6':
  ART: Fix gtest dependency

Merge "ART: Fix gtest dependency"

ART: Fix gtest dependency

Gtests should depend on the debug version of libart-disassembler.

Bug: 21273194
Change-Id: Iaf216a1c0cff650296ebf83fe4e8352f5614cb28

am 22efeba8: Merge "ART: Blacklist CFI test for JIT"

* commit '22efeba85d4056143e34ed9a212a48a206d2f5b8':
  ART: Blacklist CFI test for JIT

Merge "ART: Blacklist CFI test for JIT"

ART: Blacklist CFI test for JIT

The test expects Java frames on the stack.

Change-Id: I485a7a2a3d15fb52c207bd8723bed3b01289c000

am d6eee932: Merge "ART: Fix RegisterNative order"

* commit 'd6eee932a4da1cf1a582eff9d75a1561c849789d':
  ART: Fix RegisterNative order

Merge "ART: Fix RegisterNative order"

am 513b6fdc: Merge "ART: Blacklist CFI test for non-compiled run-tests"

* commit '513b6fdcbee0f83f2d9157d41b8498dba96283c5':
  ART: Blacklist CFI test for non-compiled run-tests

ART: Fix RegisterNative order

First check for both direct and virtual methods in the current class,
then move to the parent.

Optimize registration by checking first whether the current method
under test is native. This slows down registering implementations
in parent classes. Add a CheckJNI warning for this.

Add a run-test to check the behavior. Fix host comparison testing.

Bug: 19569721
Change-Id: I61e77117d96310632aad123d7f1279d0f834dc99

Merge "ART: Blacklist CFI test for non-compiled run-tests"

ART: Blacklist CFI test for non-compiled run-tests

The test expects Java frames on the stack.

Change-Id: I37914ad6a3c82264ee189492611469928a786f67

am c1c5de73: Merge "ART: Refactor UnstartedRuntime for testing"

* commit 'c1c5de731e2a23646e02e91311de29ecdd167dbd':
  ART: Refactor UnstartedRuntime for testing

am 20d80bef: Merge "Switch to using ELF-64 for 64-bit architectures."

* commit '20d80bef40f63ac94eeb93749199f59e1b16adf1':
  Switch to using ELF-64 for 64-bit architectures.

Merge "ART: Refactor UnstartedRuntime for testing"

ART: Refactor UnstartedRuntime for testing

Expose the UnstartedRuntime implementation functions as private static
methods of a class. Add a gtest that can invoke these functions. Add
sample tests for String and Memory.

Bug: 21173514
Change-Id: Ib5bde6347fafaf7607c642542ea7d5938ff4b1df

Merge "Switch to using ELF-64 for 64-bit architectures."

am e128ee2e: Merge "ART: CFI Test"

* commit 'e128ee2ee9118f2673b67831e98d92b0a637b078':
  ART: CFI Test

Merge "ART: CFI Test"